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

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

ただいまの
回答率

88.09%

JavaScript ハンバーガーメニューの表示が消えたまま再表示されない

解決済

回答 2

投稿 編集

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

score 12

今HPでハンバーガーメニューの実装をしているのですがうまくいきません。
JQueryはまだ勉強していないのでJavaScriptで実装しようとしています。
JavaScriptはProGateでやっただけで知識も理解も不足しているのですが、メニューの表示までは何とか形になりました。
メニューボタンをクリックすると関数toggleNavが実行されhamburger-menuクラスにactiveクラスが追加されることでメニューを表示するようにコードを書きました。
その後が問題で、ページ内リンクへジャンプすると同時にactiveクラスを削除することでメニューの表示を消すようにコードを書いたのですが、ページが消えた後再びメニューをクリックするとメニューが表示されません。(ページをリロードしてクリックすれば表示される。)
見様見真似でコードをあれこれ書いてみましたがどんなに調べても結局どこが悪いのかわかりません。エラーを見るとクラスを削除したことで関数が読み込めていないような感じですがさっぱり理解できません。
ここのサイトを見て同じような感じでコードを書いたつもりです。
https://tech-dig.jp/hamburger-global-nav/

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

hamberger.js:16 Uncaught ReferenceError: toggleNav is not defined
at hamberger.js:16
10hamberger.js:5 Uncaught TypeError: Cannot read property 'classList' of null
at HTMLAnchorElement.<anonymous> (hamberger.js:5)

該当のソースコード

<!DOCTYPE html>
<html lang="ja" dir="ltr">

<head>
  <meta charset="utf-8">
  <meta name="viewport" content="width=device-width, initial-scale=1">
  <title>shiro</title>
  <link rel="stylesheet" href="style.css">
</head>

<body>
  <header class="header">
    <div class="header_nav">
      <div class="u-content-wrapper header_nav_wrapper">
        <div class="header_nav_logo">Shiro</div>
        <a href="#" class="hamburger-menu" id="hamburger-menu-Button">
          <div class="hamburger-menu_line"></div>
          <div class="hamburger-menu_line"></div>
          <div class="hamburger-menu_line"></div>
        </a>
      </div>
      <ul class="header-menu" id="header-menu">
        <li class="header_menu_item" ><a href=#section-concept id="header_menu_item">Concept</a></li>
        <li class="header_menu_item" ><a href=#work id="header_menu_item">Work</a></li>
        <li class="header_menu_item" ><a href=#florist id="header_menu_item">Florist</a></li>
        <li class="header_menu_item" ><a href=#shop_Information id="header_menu_item">Shop information</a></li>
        <li class="header_menu_item" ><a href=#contact_us id="header_menu_item">Contact us</a></li>
      </ul>
    </div>
    <div class="u-content-wrapper">
      <h1 class="header_title">Shiro</h1>
      <p class="header_title_sub">White flower shop</p>
    </div>
  </header>
/* header */
.header {
  color: white;
  width: 100%;
  height: 620px;
  background-image: url(./header/header0.png);
  background-size: cover;
  text-align: center;
  font-family: myriad-pro, sans-serif;
  position: relative;
}

.header_nav {
  position: fixed;
  top: 0;
  left: 0;
  width: 100%;
  height: 80px;
  background-color: rgba(84, 131, 126,0.5);
  z-index: 100;
}

/* ハンバーガーメニュー */
.hamburger-menu  {
  display: block;
  width: 50px;
  height: 30px;
  background-color: none;
  top:0;
  right: 0;
  position: relative;
}

.hamburger-menu_line {
  display: block;
  width: 35px;
  height: .8px;
  background-color: white;
  position: absolute;
  top: 50%;
  left: 50%;
  transform: translateX(-50%);
  transition: .3s;
}


.hamburger-menu_line:nth-of-type(1) {
  transform: translate(-50%,-8px);
}

.hamburger-menu.active div:nth-of-type(1) {
  transform: rotate(45deg) translate(-50%,0px);
  transform-origin: 0% 50%;
}

.hamburger-menu.active div:nth-of-type(2) {
  opacity: 0;
}

.hamburger-menu_line:nth-of-type(3) {
  transform: translate(-50%,8px);
}

.hamburger-menu.active div:nth-of-type(3) {
  transform: rotate(-45deg) translate(-50%,0px);
  transform-origin: 0% 50%;
}

/* 普段は非表示 */

.header-menu {
  opacity: 0;
  visibility: hidden;
  transition: .3s;
 position: absolute;
 width: 100%;
 z-index: 100;
}

/* クリックされたときにメニュー表示 */

.header-menu.active {
  opacity: 1;
  visibility: visible;

}

.header_menu_item {
  background-color: rgba(222, 234, 238,0.8);
  width: 100%;
  height: 135px;
  border-bottom: 1px solid #ccc;
  line-height: 135px;
}
document.addEventListener('DOMContentLoaded', function toggleNav() {
  document.getElementById('hamburger-menu-Button').addEventListener("click",
    function() {
      this.classList.toggle("active");
      document.getElementById('header-menu').classList.toggle("active");
    })
});

document.addEventListener('DOMContentLoaded', function() {
  document.getElementById('header_menu_item').addEventListener("click",
    function() {
        document.getElementById('header-menu').remove("active");
    })
});

toggleNav();

試したこと

こちらのコードを試してみたが結果は変わらなかったです。

document.addEventListener('DOMContentLoaded', function() {
  document.getElementById('hamburger-menu-Button').addEventListener("click",function() {
    this.classList.toggle("active");
    document.getElementById('header-menu').classList.toggle("active");
  });
  document.getElementById('header_menu_item').addEventListener("click", function() {
    document.getElementById('header-menu').remove("active");
  });
});


上記にあるサイトを参考にこちらもためしてみましたがactiveクラスすら追加されませんでした。検証でConsoleを見てみると、エラーは出てませんでした。もう訳が分からなくて困り果てております。
いろいろなサイトを見ているのですがJQueryばかりで参考にならないです。

function toggleNav() {
  var body = document.body;
  var hamburger_menu = document.getElementById('hamburger-menu-Button');
  var header_menu = document.getElementById("header-menu");

  hamburger_menu.addEventListener('click', function() {
    body.classList.toggle('active');
  });
  header_menu.addEventListener('click', function() {
    body.classList.remove('active');
  });
}

toggleNav();

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

ここにより詳細な情報を記載してください。

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

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

  • kei344

    2020/03/07 02:15

    (質問文は編集できます)質問文のコードはそれぞれコードブロックで囲んでいただけませんか? ```(バッククオート3つ)で囲み、前後に改行をいれるか、コードを選択して「<code>」ボタンを押すとコードブロックになります。

    キャンセル

  • Kiyobun510

    2020/03/07 03:20

    すみません、すぐに変更いたしました。
    ご指摘ありがとうございました。

    キャンセル

回答 2

check解決した方法

0

コードを書き換えたところやっと解決致しました。
thisを使いactiveクラスを削除したままだったのが悪かったみたいです。
最後にまたactiveクラスを追加するコードを書いたところうまく動きました。
夜分遅くに対応してくださった方、ありがとうございました。

document.addEventListener('DOMContentLoaded', function() {
  document.getElementById('hamburger-menu-Button').addEventListener("click",function() {
    this.classList.toggle("active");
    document.getElementById('header_menu').classList.toggle("active");
  });
  document.getElementById('header_menu').addEventListener("click", function() {
   ** this.classList.remove("active");**
   document.getElementById('hamburger-menu-Button').classList.toggle("active");
  });
});

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

0

document.addEventListener('DOMContentLoaded', function toggleNav() {
document.getElementById('hamburger-menu-Button').addEventListener("click",
function() {
this.classList.toggle("active");
document.getElementById('header-menu').classList.toggle("active");
})
});


の部分について以下の部分の記載のような感じでないと反応しないです。

対象要素.addEventListener(種類, sampleEvent, false);

function sampleEvent() {

  //ここに処理を記述する

}

上のような書き方ではなくても良い場合、下のjsで動作します。

document.addEventListener('DOMContentLoaded', function() {
  document.getElementById('hamburger-menu-Button').addEventListener("click",function() {
    this.classList.toggle("active");
    document.getElementById('header-menu').classList.toggle("active");
  });
  document.getElementById('header_menu_item').addEventListener("click", function() {
    document.getElementById('header-menu').remove("active");
  });
});

addEventListenerについて詳しい説明のurlについても一応載せておきます。

  • https://qiita.com/_ayk_study/items/519bb661a746f96727ce

  • https://www.sejuku.net/blog/57625

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2020/03/07 03:27

    拙い文章にご回答ありがとうございます。addEventListenerについてはだいぶ理解できました。

    試させていただいたのですがうまくいきませんでした。
    やはり一度はうまくいくのですが、その後クリックするとメニューが表示されなくなってしまします。

    ちなみにこのようなエラーを確認できました。
    ```
    Uncaught TypeError: Cannot read property 'classList' of null
    at HTMLAnchorElement.<anonymous> (hamberger.js:4)
    ```

    ご教授いただければ幸いです。

    キャンセル

  • 2020/03/07 04:08

    いただいたコードでこちらの方では問題なく実行できるので、動作するもののcodepenも共有いたします。
    https://codepen.io/miwashutaro0611/pen/eYNeBWL

    キャンセル

  • 2020/03/07 04:26

    ありがとうございます。このようなものがあるのですね。
    いただいたcodepenを確認したところ一回目は成功するのですが二回目からは相変わらずメニューは表示されません。
    つまるところ動作環境の違いなんでしょうか?

    キャンセル

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

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

関連した質問

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