addEventListenerでセットした子要素でイベントが発火するのを防ぐには

Question

以下のコードでid=koreにクリックイベントをセットします。
しかし、ボタンに表示されている画像の部分をクリックするとコールバックのe.targetがbuttonでなくimgになってしまいます。
e.currentTargetを使えばbutton要素が取得出来るのは知っているのですが、button要素を取得する方法ではなくimg要素でイベントが発生しないようにする方法を知りたいです。

ボタンの中の画像をクリックしてe.targetがimgの時と、ボタンの中の画像が無い所をクリックしてe.targetがbuttonの時で、画面の表示が微妙に異なり、e.targetがbuttonの時の表示に固定したいのです。

createEvent→initMouseEvent でイベントを作成しても挙動が違いました。

document.querySelector("#kore").addEventListener("click",function(e){
  //e.target or e.currentTarget
});

<button id="kore">
  <img src="xxx">
</button>
```、

Answer 1

バブリングフェーズ、キャプチャリングフェーズを強制終了

イベントの発火を防止するのは原理的に不可能だと思います。
バブリングを停止することで上位要素のバブリングフェーズにおいてイベントが発火していないかのように見せかける事は可能です。

'use strict';
document.querySelector('#kore *').addEventListener('click', function (event) {
  event.preventDefault();           // デフォルトアクションを抑止
  event.stopImmediatePropagation(); // バブリングを停止
}, true);

document.getElementById('kore').addEventListener('click', function (event) {  // // #kore のみ発火する
  console.log(event.type, 'bubbling-phase', '/ target = #' + event.target.id + ' / currentTarget = #' + event.currentTarget.id);
}, false);

document.getElementById('kore').addEventListener('click', function (event) {  // // #kore 以外も発火する
  console.log(event.type, 'capturing-phase', '/ target = #' + event.target.id + ' / currentTarget = #' + event.currentTarget.id);
}, true);

addEventListener('click', function (event) {  // #kore 以外も発火する
  console.log(event.type, 'capturing-phase', '/ target = #' + event.target.id + ' / currentTarget = window');
}, true);

キャプチャリングフェーズまで対応するなら最上位ノードである window.addEventListener を監視するしかありません。

'use strict';
addEventListener('click', function (event) {  // #kore 以外も発火する
  console.log(event.type, 'capturing-phase', '/ target = #' + event.target.id + ' / currentTarget = window');
}, true);

addEventListener('click', function (event) {
  for (var target = event.target, i = 0, elements = event.currentTarget.document.querySelectorAll('#kore *'), l = elements.length; i < l; ++i) {
    if (target === elements[i]) {
      event.preventDefault();           // デフォルトアクションを抑止
      event.stopImmediatePropagation(); // キャプチャリングフェーズを強制終了
      return;
    }
  }
}, true);

document.getElementById('kore').addEventListener('click', function (event) {  // #kore のみ発火する
  console.log(event.type, 'bubbling-phase', '/ target = #' + event.target.id + ' / currentTarget = #' + event.currentTarget.id);
}, false);

document.getElementById('kore').addEventListener('click', function (event) {  // #kore のみ発火する
  console.log(event.type, 'capturing-phase', '/ target = #' + event.target.id + ' / currentTarget = #' + event.currentTarget.id);
}, true);

addEventListener('click', function (event) {  // #kore のみ発火する
  console.log(event.type, 'capturing-phase', '/ target = #' + event.target.id + ' / currentTarget = window');
}, true);

当然ながら event.stopImmediatePropagation() 実行前のリスナーは無効化できませんので、event.stopImmediatePropagation() は一番初めに実行する必要があります。

event.target で分岐処理

Fushihara さんが期待する動作ではないかもしれませんが、個人的には次のように書きます。

document.getElementById('kore').addEventListener('click', function (event) {
  if (event.target !== event.currentTarget) { // event.target が #kore でないなら強制終了
    return;
  }

  console.log(event.type, '#' + event.target.id);
}, false);

Re: Fushihara さん

Answer 2

イベント自体を止めることは無理ですが、無視することは出来ます。

<div id="kore">
  <img src="https://placehold.jp/8/cc9999/993333/50x30.png">
  てきすと
</div>
```として
```JavaScript
document.querySelector( '#kore' ).addEventListener( 'click', function( e ) {
//	if ( e.target.tagName === 'img' ) { // 修正前（コメント欄参照）
	if ( e.target.tagName === 'IMG' ) { // 修正後
		e.stopPropagation();
		return;
	}
	/* do something. */
}, false );
```か
```JavaScript
document.querySelector( '#kore img' ).addEventListener( 'click', function( e ) {
	e.stopPropagation();
}, false );
document.querySelector( '#kore' ).addEventListener( 'click', function( e ) {
	/* do something. */
} );

ではどうでしょう。
※ buttonタグの場合上記記述では使え無いと思います。

動くサンプル：https://jsfiddle.net/pj75tsbg/

Comments

[think49]

> if ( e.target.tagName === 'img' ) {
XHTMLを想定されているのでしょうか。
HTML では tagName は常に大文字ですので e.target.tagName === 'IMG' もしくは String#toLowerCase を併用する必要があります。

[kei344]

指摘ありがとうございます！
手元の環境で試したときには toLowerCase していたのに失敗しました･･･。
本文を修正しておきます。

Answer 3

querySelector()はあまり使ったことがないですが、ブラウザによって挙動が違うようですね
いずれ仕様が統一されるような気もしますが、今回の件に関していえば場当たり的ですが
子要素を掴んでいるのであれば親に遡ればよいのではないでしょうか？

function(e){
  var t=e.target;
  while(t){
    if(t.nodeName=="BUTTON") break;
    t=t.parentNode;
  }
  console.log(t.nodeName);
});

Comments

[yambejp]

失礼しました
e.currentTargetで取れるって書いてありましたね