Removing element attributes when clicking on it using Vanilla JS

Question

How can I remove the attribute of an element that is on a click outside or on another of divthe same type? Here is my code:

HTML:

<div id="container">
  <div data-something></div>
  <div data-something></div>
</div>

JavaScript:

var elements = document.querySelectorAll("[data-something]");    

Array.prototype.forEach.call(elements, function(element) {

  // Adding
  element.onclick = function() {
    this.setAttribute("data-adding", "");
  };

  // Removing -- Example
  element.onclickoutside = function() {
     this.removeAttribute("data-adding");
  };

});

Answer 1

I would probably use a click handler in the document, and then remove the attribute from any element that had it out of the way of the bubble.

document.addEventListener("click", function(e) {
    Array.prototype.forEach.call(document.querySelectorAll("*[data-adding][data-something]"), function(element) {
        var node, found = false;
        for (node = e.target; !found && node; node = node.parentNode) {
            if (node === element) {
                found = true;
            }
        }
        if (!found) {
            element.removeAttribute("data-adding");
        }
    });
}, false);

... or something like that.

Live example:

    document.addEventListener("click", function(e) {
      Array.prototype.forEach.call(document.querySelectorAll("*[data-adding]"), function(element) {
        var node, found = false;
        for (node = e.target; !found && node; node = node.parentNode) {
          if (node === element) {
            found = true;
          }
        }
        if (!found) {
          element.removeAttribute("data-adding");
        }
      });
    }, false);

*[data-adding] {
  color: red;
}

<div data-adding data-something>One</div>
<div data-adding data-something>Two</div>

Run code

Answer 2

You can use Node.contains()inside the global click event handler to check if the click is outside the element and handle the event accordingly:

box = document.getElementById('box');
lastEvent = document.getElementById('event');

box.addEventListener('click', function(event) {
  // click inside box
  // (do stuff here...)
  lastEvent.textContent = 'Inside';
});

window.addEventListener('click', function(event) {
  if (!box.contains(event.target)) {
    // click outside box
    // (do stuff here...)
    lastEvent.textContent = 'Outside';
  }
});

#box {
  width: 200px;
  height: 50px;
  background-color: #ffaaaa;
}

<div id="box">Click inside or outside me</div>

<div>Last event: <span id="event">(none)</span>
</div>

Run code