質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

87.34%

addEventListenerのクリックイベントでディスプレイの表示・非表示の切り替えをしたい

解決済

回答 2

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 868

score 23

前提・実現したいこと

現在、webページの模写をしています。
addEventListenerのクリックイベントでディスプレイを表示させ、その表示させたディスプレイ以外の場所(bodyタグ)をクリックするとディスプレイが消えるようにしたいです。

*上記のディスプレイとは<div class="guide">と<div class="guide-2">のことです

htmlは見やすいように、表示・非表示の切り替えをしたいディスプレイとクリックする場所以外は省略してあります。

発生している問題・エラーメッセージ

エラーメッセージは出ませんが、ディスプレイ自体を表示することがそもそもできません。

該当のソースコード

<body>
<header>
<div class="hd-link" id="hd-margin">
        <p>ホストをはじめる</p>
      </div>
</header>
<main>
<div class="guide close">
      <div class="sur">
      <h4>宿泊先</h4>
      <p>お部屋を掲載</p>
      <h2>横浜市でホスティング、月最大¥101,538の収入に。</h2>
      <p>お家のホスティングについて</p>
      </div>
    </div>
<div class="guide-2 close">
      <div class="sur">
      <h4>体験</h4>
      <p>体験をホストする</p>
      <h2>好きなことを世界とシェアして、収入を得よう</h2>
      </div>
    </div>
</main>
<footer>
</footer>
</body>
.sur {
  width: 230px;
  margin: 0 auto;
  background: rgb(255, 255, 255);
}
.guide {
  height: 140px;
  width: 300px;
  margin: 0 auto;
  background: rgb(255, 255, 255);
  position: absolute;
  right: 300px;
  border: 1px solid rgb(180, 180, 180);
  box-sizing: border-box;
  border-bottom: none;
}
.guide-2 {
  width: 300px;
  margin: 0 auto;
  background: rgb(255, 255, 255);
  position: absolute;
  right: 300px;
  top: 220px;
  border: 1px solid rgb(180, 180, 180);
  box-sizing: border-box;
}
.close {
  display: none;
}
let guide = document.getElementsByClassName("guide")[0];
let guide2 = document.getElementsByClassName("guide-2")[0];
let nav = document.getElementById("hd-margin");
let body =document.getElementsByTagName("body")[0];

let nav2 = function() {
  nav.addEventListener("click", function() {
    guide.classList.remove("close");
    guide2.classList.remove("close");
    body.addEventListener("click", function() {
      guide.classList.add("close");
      guide2.classList.add("close");
      nav2();
    });
  });
};

nav2();

試したこと

試しに全く同じ場所をクリックして、ディスプレイの表示・非表示の切り替えができるか試したら、それは実装可能でした。
その際はJavaScriptのbody.addEventListenerの箇所をnav.addEventListenerに切り替えました。

またこの問題を解決したとしても次は、表示させたディスプレイ以外のbodyタグ内をクリックした時のみ、ディスプレイを非表示にしたいという問題が出てくると思いますが、そこに関しての解決法は全く分かりません。

補足情報(FW/ツールのバージョンなど)

Atomエディタ version 1.40.1

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • kei344

    2019/09/23 15:46

    「ディスプレイを表示させ」とありますが、「ディスプレイ」とは具体的にどこを指しているのかが提示されていません。

    キャンセル

  • Naoki_Pro

    2019/09/23 16:00

    分かりにくくてすみません。質問を修正いたしました。

    キャンセル

回答 2

checkベストアンサー

+3

まずは、addEventListener は既にあるイベントを上書きせずに追加していくので、クリックイベント内にaddEventListenerを記述すると、クリックするたびにイベントリステナーが追加されてしまうので、おかしな動作になります。

ドキュメント全体で、特定の要素をクリックしたか、それ以外の要素をクリックしたかを判定するには、下記のようにdocument.addEventListenerのイベントで場合分けしたらどうでしょうか。

document.addEventListener('click', function(e){
  let guide = e.target.closest(".guide");
  let guide2 = e.target.closest(".guide-2");

  if(guide){
    console.log("guideをクリック");
  } else if(guide2) {
    console.log("guide-2をクリック");
  }else {
    console.log("guide, guide-2以外をクリック");
  };;
}, false);

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/09/23 17:41 編集

    重なり合う要素をクリックした場合に、どちらの要素がクリックされたのかを特定する方法があるのは知りませんでした。
    大変勉強になります。

    キャンセル

  • 2019/09/27 10:58

    回答して頂きありがとうございます。
    返信が遅くなってしまいすみません。

    hatena19さんの方法を試したら、上手く稼働させることができました。
    ありがとうございます。

    キャンセル

+1

2点注意を

let nav2 = function() {
  nav.addEventListener("click", function() {
    guide.classList.remove("close");
    guide2.classList.remove("close");
    body.addEventListener("click", function() {
      guide.classList.add("close");
      guide2.classList.add("close");
      nav2(); // ←nav2の中でnav2が呼び出されている
    });
  });
};

何か特別な理由がない限りこういうことはやめた方がいいです。
処理が複雑になり過ぎます。
計算処理ならまだしも、domの操作で再起を使うくらいなら、代替手段を考えましょう。

navをクリックしたとき、bodyもクリックしたことになっているので、'close'クラスが削除(remove)された瞬間に再度追加(add)される結果となっているように見えます。

そのため、結果として見た目に変更は生じません。

Na_Yさんのおっしゃる「ディスプレイ」を消す手段は、bodyをクリックしたときではなく、navを含まない何か別の要素をクリックしたときにする必要があるように思います。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/09/27 10:59

    返信が遅くなってしまいすみません。
    アドバイスして頂きありがとうございます。

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 87.34%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る