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

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

ただいまの
回答率

87.79%

ハンバーガーメニューをクリックで閉じたいです

解決済

回答 1

投稿 編集

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

実行したいこと

レスポンシブにしてメニューボタンをクリックし、バツになった時にメニューを閉じたいのですが閉じられません。
コードは以下のサイトを参考にしています。

最近jsやjQを勉強し始めた初心者です。
単純な質問で恐れ入りますが、以下のコードをどう書いたら実行されるのかお手上げ状態になりました。どなたかご教示いただけましたら幸いです。
よろしくお願いいたします。

参考にしたサイト様

モーダルの動き
https://haniwaman.com/responsive-menu/#i-9

メニューサンプル(2番目のボタンです)
https://125naroom.com/web/2958

<!DOCTYPE html>
<html lang="ja">
<head>
<meta charset="utf-8">
<meta name="viewport" content="width=device-width">
<meta http-equiv="X-UA-Compatible" content="IE=edge">
<title>ハンバーガーメニュー</title>
<link rel="stylesheet" href="css/reset.css">
<link rel="stylesheet" href="css/layout03.css">
</head>

<body>
    <div class="icon-hamburger"><span></span></div>
    <div class="btn-trigger" id="btn02">
      <span></span>
      <span></span>
      <span></span>
    </div>

    <nav class="menu-container">
      <ul class="menu">
        <li class="menu-item"><a href="#">ホーム</a></li>
        <li class="menu-item"><a href="#">ページ A</a></li>
        <li class="menu-item"><a href="#">ページ B</a></li>
        <li class="menu-item"><a href="#">ページ C</a></li>
        <li class="menu-item"><a href="#">ページ D</a></li>
        <li class="menu-item"><a href="#">ページ E</a></li>
      </ul>
    </nav>
<script src="js/jquery-1.12.4.min.js"></script>    
<script src="js/index03.js"></script>
</body>
</html>
/*PC版メニューのデザイン*/
.menu-container { margin: 0; }
.menu-container .menu { display: flex; justify-content: space-between; list-style: none; margin: 0; padding: 0; }
.menu-container .menu .menu-item { flex: 1; }
.menu-container .menu .menu-item a { border-right: 1px solid #ccc; color: #333; display: block; padding: 0.6em 0.4em; text-align: center; text-decoration: none; }
.menu-container .menu .menu-item a:hover { background: #ccc; color: #ffffff; }
.menu-container .menu .menu-item:last-child a { border-right: none; }

#modal-overlay { background-color: rgba(0, 0, 0, 0.6); display: none; height: 120%; left: 0; position: fixed; top: 0; width: 100%; }

@media screen and (max-width: 768px) {
    /*ハンバーガーアイコン*/
    .btn-trigger { position: relative; border: 1px solid #ccc; width: 50px; height: 50px; cursor: pointer; margin: 10px 0 0 10px; z-index: 10; }
    .btn-trigger span { position: absolute; width: 30px; height: 2px; background-color: #ccc; border-radius: 4px; cursor: pointer; left: 18%; }
    .btn-trigger, .btn-trigger span { display: inline-block; transition: all .5s; box-sizing: border-box }
    .btn-trigger span:nth-of-type(1) { top: 28%; }
    .btn-trigger span:nth-of-type(2) { top: 50%; }
    .btn-trigger span:nth-of-type(3) { bottom: 25%; }

    /*閉じるアイコン*/
    .btn-trigger.active span:nth-of-type(1) { -webkit-transform: translateY(11px) rotate(45deg); transform: translateY(11px) rotate(45deg); }
    .btn-trigger.active span:nth-of-type(2) { opacity: 0; }
    .btn-trigger.active span:nth-of-type(3) { -webkit-transform: translateY(-10px) rotate(-45deg); transform: translateY(-10px) rotate(-45deg); }

    /*メニューの中身*/
    .menu-container .menu { -webkit-transform: translate(-50%, -50%); display: none; height: auto; left: 50%; position: fixed; top: 50%; transform: translate(-50%, -50%); width: 100%; z-index: 100; }
    .menu-container .menu .menu-item a { color:#fff; border-right: none; }
    .menu-container .menu .menu-item a:hover { background: none; color: #ccc; }
    .menu-container .menu .menu-item:last-child a { border-bottom: none; }
}
$(function(){
    $('.btn-trigger').on('click', function() {
        $('body').append('<div id="modal-overlay"></div>');
        $('#modal-overlay').fadeIn('1500');
        $('.menu-container .menu').fadeIn('1500');
        $(this).toggleClass('active');
    });
});

追記__コメント欄記載の現象【1】【2】発生後の検証コードです(2020'06'09追記)

【現象2】に関しては、メディアクエリ内の/*メニューの中身*/<.menu-container .menu>のdisplayをblockに変更し、画面幅に併せて表示/非表示になりましたが、そうすることにより、レスポンシブ化した際のメニューが表示されたままになります。(当然ですよね・・・・)
そこで、<.menu-container .menu.close>を作成し、透明0/ボタンを押せなくしてjsに当てるといけるかも?と思い、試してみましたが、書き方がよろしくないのか思うようにいきません。
コードは以下です。ご検証のほどよろしくお願い申し上げます。

<!DOCTYPE html>
<html lang="ja">
<head>
<meta charset="utf-8">
<meta name="viewport" content="width=device-width">
<meta http-equiv="X-UA-Compatible" content="IE=edge">
<title>ハンバーガーメニュー</title>
<link rel="stylesheet" href="css/reset.css">
<script src="https://ajax.googleapis.com/ajax/libs/jquery/1.12.4/jquery.min.js"></script>
<style>
/*PC版メニューのデザイン*/
body { background-color: #e9967a; }    
.menu-container { margin: 0; }
.menu-container .menu { display: flex; justify-content: space-between; list-style: none; margin: 0; padding: 0; }
.menu-container .menu .menu-item { flex: 1; }
.menu-container .menu .menu-item a { border-right: 1px solid #ccc; color: #333; display: block; padding: 0.6em 0.4em; text-align: center; text-decoration: none; }
.menu-container .menu .menu-item a:hover { background: #ccc; color: #ffffff; }
.menu-container .menu .menu-item:last-child a { border-right: none; }


@media screen and (max-width: 768px) {
    /*ハンバーガーアイコン*/
    .btn-trigger { position: relative; border: 1px solid #ccc; width: 50px; height: 50px; cursor: pointer; margin: 10px 0 0 10px; z-index: 10; }
    .btn-trigger span { position: absolute; width: 30px; height: 2px; background-color: #ccc; border-radius: 4px; cursor: pointer; left: 18%; }
    .btn-trigger, .btn-trigger span { display: inline-block; transition: all .5s; box-sizing: border-box }
    .btn-trigger span:nth-of-type(1) { top: 28%; }
    .btn-trigger span:nth-of-type(2) { top: 50%; }
    .btn-trigger span:nth-of-type(3) { bottom: 25%; }

    /*閉じるアイコン*/
    .btn-trigger.active span:nth-of-type(1) {-webkit-transform: translateY(11px) rotate(45deg); transform: translateY(11px) rotate(45deg); }
    .btn-trigger.active span:nth-of-type(2) { opacity: 0; }
    .btn-trigger.active span:nth-of-type(3) { -webkit-transform: translateY(-10px) rotate(-45deg); transform: translateY(-10px) rotate(-45deg); }

    /*メニューの中身*/
    .menu-container .menu { -webkit-transform: translate(-50%, -50%); display: block;/*noneから変更してみました*/ height: auto; left: 50%; position: fixed; top: 50%; transform: translate(-50%, -50%); width: 100%; z-index: 100; }
    .menu-container .menu .menu-item a { color:#fff; border-right: none; }
    .menu-container .menu .menu-item a:hover { background: none; color: #ccc; }
    .menu-container .menu .menu-item:last-child a { border-bottom: none; }

    /*オーバーレイ →こちらに移動しました*/
    .modal-overlay { background-color: rgba(0, 0, 0, 0.6); display: none; height: 120%; left: 0; position: fixed; top: 0; width: 100%; }

    .menu-container .menu.close { opacity: 0; visibility: hidden; } /*追記してみました*/

}
</style>
<script>
    $(function(){
    $('.btn-trigger').on('click', function() {
        if ($('header').hasClass('modal-overlay') == false) {
            $('header').addClass('modal-overlay');
            $('header').fadeIn('1500');
            $('.menu-container .menu').fadeIn('1500');
            $(this).toggleClass('active');
        } else {
            $('header').removeClass('modal-overlay');
            $(this).toggleClass('active');
            $(this).toggleClass('.menu-container .menu.close'); /*追記してみました*/
        }
    });
});
</script>
</head>

<body>
    <header>
        <div class="icon-hamburger"><span></span></div>
        <div class="btn-trigger" id="btn02">
            <span></span>
            <span></span>
            <span></span>
        </div>

        <nav class="menu-container">
            <ul class="menu">
                <li class="menu-item"><a href="#">ホーム</a></li>
                <li class="menu-item"><a href="#">ページ A</a></li>
                <li class="menu-item"><a href="#">ページ B</a></li>
                <li class="menu-item"><a href="#">ページ C</a></li>
                <li class="menu-item"><a href="#">ページ D</a></li>
                <li class="menu-item"><a href="#">ページ E</a></li>
            </ul>
        </nav>
    </header>
</body>
</html>
  • 気になる質問をクリップする

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 1

checkベストアンサー

+1

今日、早出なのでぶっつけ本番のやっつけ仕事になってしまい申し訳ありませんが、一応動くようにはなりました。
ただ、ソースが汚いかもなので参考にしてご自身のコードを修正してください。
jQueryはheadの中で先に読み込んでおいた方がいいですよ。ソースもいちいち用意せずgoogleからロードすればいいです。

フォルダ構造やファイル名の指定など時間がなかったので、HTML・CSS・JSが1つのコードになってます。
適宜、切り出して別ファイルにしてください。

追記:改修後の問題の出てるソースを修正したものに書き換えました。これで動作確認お願いします。

<!DOCTYPE html>
<html lang="ja">
<head>
<meta charset="utf-8">
<meta name="viewport" content="width=device-width">
<meta http-equiv="X-UA-Compatible" content="IE=edge">
<title>ハンバーガーメニュー</title>
<link rel="stylesheet" href="css/reset.css">
<script src="https://ajax.googleapis.com/ajax/libs/jquery/1.12.4/jquery.min.js"></script>
<style>
/*PC版メニューのデザイン*/
body { background-color: #e9967a; }    
.menu-container { margin: 0; }
.menu-container .menu { display: flex; justify-content: space-between; list-style: none; margin: 0; padding: 0; }
.menu-container .menu .menu-item { flex: 1; }
.menu-container .menu .menu-item a { border-right: 1px solid #ccc; color: #333; display: block; padding: 0.6em 0.4em; text-align: center; text-decoration: none; }
.menu-container .menu .menu-item a:hover { background: #ccc; color: #ffffff; }
.menu-container .menu .menu-item:last-child a { border-right: none; }


@media screen and (max-width: 768px) {
    /*ハンバーガーアイコン*/
    .btn-trigger { position: relative; border: 1px solid #ccc; width: 50px; height: 50px; cursor: pointer; margin: 10px 0 0 10px; z-index: 10; }
    .btn-trigger span { position: absolute; width: 30px; height: 2px; background-color: #ccc; border-radius: 4px; cursor: pointer; left: 18%; }
    .btn-trigger, .btn-trigger span { display: inline-block; transition: all .5s; box-sizing: border-box }
    .btn-trigger span:nth-of-type(1) { top: 28%; }
    .btn-trigger span:nth-of-type(2) { top: 50%; }
    .btn-trigger span:nth-of-type(3) { bottom: 25%; }

    /*閉じるアイコン*/
    .btn-trigger.active span:nth-of-type(1) {-webkit-transform: translateY(11px) rotate(45deg); transform: translateY(11px) rotate(45deg); }
    .btn-trigger.active span:nth-of-type(2) { opacity: 0; }
    .btn-trigger.active span:nth-of-type(3) { -webkit-transform: translateY(-10px) rotate(-45deg); transform: translateY(-10px) rotate(-45deg); }

    /*メニューの中身*/
    .menu-container .menu { -webkit-transform: translate(-50%, -50%); display: block;/*noneから変更してみました*/ height: auto; left: 50%; position: fixed; top: 50%; transform: translate(-50%, -50%); width: 100%; z-index: 100; }
    .menu-container .menu .menu-item a { color:#fff; border-right: none; }
    .menu-container .menu .menu-item a:hover { background: none; color: #ccc; }
    .menu-container .menu .menu-item:last-child a { border-bottom: none; }

    /*オーバーレイ →こちらに移動しました*/
    .modal-overlay { background-color: rgba(0, 0, 0, 0.6); display: none; height: 120%; left: 0; position: fixed; top: 0; width: 100%; }

    .menu-container { display: none;} 

    .menu-container.active { display: block;}
}
</style>
<script>
    $(function(){
    $('.btn-trigger').on('click', function() {
        if ($('header').hasClass('modal-overlay') == false) {
            $('header').addClass('modal-overlay');
            $('header').fadeIn('1500');
            $('.menu-container .menu').fadeIn('1500');
            $(this).toggleClass('active');
            $('.menu-container').toggleClass('active');

        } else {
            $('header').removeClass('modal-overlay');
            $(this).toggleClass('active');
            $('.menu-container').toggleClass('active');
        }
    });
});
</script>
</head>

<body>
    <header>
        <div class="icon-hamburger"><span></span></div>
        <div class="btn-trigger" id="btn02">
            <span></span>
            <span></span>
            <span></span>
        </div>

        <nav class="menu-container">
            <ul class="menu">
                <li class="menu-item"><a href="#">ホーム</a></li>
                <li class="menu-item"><a href="#">ページ A</a></li>
                <li class="menu-item"><a href="#">ページ B</a></li>
                <li class="menu-item"><a href="#">ページ C</a></li>
                <li class="menu-item"><a href="#">ページ D</a></li>
                <li class="menu-item"><a href="#">ページ E</a></li>
            </ul>
        </nav>
    </header>
</body>
</html>

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2020/06/09 11:02

    お仕事の合間ということで、重ねて御礼を申し上げます。
    はじめての質問のため使い方に慣れておらずに大変お恥ずかしい限りです・・・。
    最新コード追記いたしました。
    緊急ではございませんので、お時間の許す際にご確認いただけますと幸いでございます。
    どうぞよろしくお願いいたします!

    キャンセル

  • 2020/06/09 16:59 編集

    いただいた最新のソースから修正したものを 改めてソースとして差し替えて記載してあります。
    動作確認は一応、こちらでは問題ありませんでした。そちらでもチェックお願いします。
    ちなみに当方 nekora(ねこ+コアラ)です。まぁどうでもいいですね、失礼しました。

    キャンセル

  • 2020/06/09 17:50

    本当にありがとうございます!!
    不足分を追加し紐解きながら、おかげさまで何を指示しているのかが少しずつ理解を深めることができました。自力で書けるにはまだまだ程遠いのですが、動くと感動しますね♪
    また、お名前間違いを大変失礼いたしました!頭がてんぱっている証拠ですね。。気をつけます。

    今後も初歩的な質問を投げると思いますが、お目に留まりました際にご指導いただけましたら幸いです。
    どうぞ引き続きよろしくお願いいたします。
    この度は本当にありがとうございました!

    キャンセル

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

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

関連した質問

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