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

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

ただいまの
回答率

90.35%

  • JavaScript

    22144questions

    JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

  • HTML

    12619questions

    HTMLとは、ウェブ上の文書を記述・作成するためのマークアップ言語のことです。文章の中に記述することで、文書の論理構造などを設定することができます。ハイパーリンクを設定できるハイパーテキストであり、画像・リスト・表などのデータファイルをリンクする情報に結びつけて情報を整理します。現在あるネットワーク上のほとんどのウェブページはHTMLで作成されています。

  • jQuery

    8777questions

    jQueryは、JavaScriptライブラリのひとつです。 簡単な記述で、JavaScriptコードを実行できるように設計されています。 2006年1月に、ジョン・レシグが発表しました。 jQueryは独特の記述法を用いており、機能のほとんどは「$関数」や「jQueryオブジェクト」のメソッドとして定義されています。

ドロップダウンメニュー、ページ内リンクで飛んでもドロップダウンが閉まらない。

解決済

回答 1

投稿

  • 評価
  • クリップ 0
  • VIEW 1,119

satoxmato

score 43

分からないこと

ドロップダウンメニュー内に、ページ内リンクを設置しましたがクリックしても
ドロップダウンメニューが閉まらないままページ内リンクの指定した場所へ飛びます。
手動でドロップダウンメニューを閉じると先ほどクリックしたページ内リンクのところが出てきます。
javascriptに何か指定すれば解決するかと思うのですがその指定するコードがわからずこちらへ質問に参りました。

実装したいこと

ドロップダウンメニュー内でページ内リンクをクリックするとドロップダウンメニューがしまわれる

下記コードです。
http://example#sample のURLはダミーです。

$(function() {
  $(".menu-trigger").click(function() {
    $("#panel").slideToggle(350);
    $(".menu-trigger").toggleClass('active');
    $(".menu-trigger2").removeClass('active');
    $("#panel2").slideUp(350);
    return false;
  });
});

$(function(){
    $(".spmenuin_li a").click(function(){
        $(this).next("ul").slideToggle();
        $(this).toggleClass("open");
    });
});

$(function() {
  $(".menu-trigger2").click(function() {
    $("#panel2").slideToggle(350);
    $(".menu-trigger2").toggleClass('active');
    $(".menu-trigger").removeClass('active');
    $("#panel").slideUp(350);
    return false;
  });
});
<div id="spbatsu" class="pcnone"><!-- 三本線のメニュー、cssアニメーションで×になる1 -->
    <a class="menu-trigger" href="#">
        <span></span>
        <span></span>
        <span></span>
    </a>
</div>
<!-- ドロップダウンで表示される内容1 -->
<div id="panel">
    <div id="panel_in">
        <ul id="panel_in_ul">
            <li class="spmenuin_li"><a href="#">トップページ</a></li>
            <li class="spmenuin_li spyazi"><a href="#" class="panel_in_li">店舗一覧</a>
                <ul class="spmenuin_li_in">
                    <li><a href="http://example#sample" class="spmenuin_li_in_li">●●店</a></li>
                    <li><a href="http://example#sample2" class="spmenuin_li_in_li">●●店</a></li>
                </ul>
            </li>
            <li class="spmenuin_li spyazi"><a href="#" class="panel_in_li">メニュー</a>
                <ul class="spmenuin_li_in">
                    <li><a href="#" class="spmenuin_li_in_li">ランチメニュー/a></li>
                    <li><a href="#" class="spmenuin_li_in_li">ディナーメニュー</a></li>
                </ul>
            </li>
        </ul>
    </div>
</div>
<div id="spbatsu2" class="pcnone"><!-- 三本線のメニュー、cssアニメーションで×になる2 -->
    <a class="menu-trigger2" href="#">
        <span></span>
        <span></span>
        <span></span>
    </a>
</div>
<!-- ドロップダウンで表示される内容2 -->
<div id="panel2">
    <div id="panel_in2">
        <p class="yoyakup">レストラン</p>
        <div class="spbtm"><a href="#">予約</a></div>
        <p><a href="tel:0123456789" class="sp_tel_ablack">0123456789</a></p>
        <p>定休日:なし</p>
        <hr />
        <p class="yoyakup">カフェ</p>
        <div class="spbtm"><a href="#">予約</a></div>
        <p><a href="tel:0123456789" class="sp_tel_ablack">0123456789</a></p>
        <p>定休日:なし</p>
    </div>
</div>
#panel {display: none; width: 100%; margin: 0; padding: 20px 0 35px 50px; background: #fff; z-index: 100; position: fixed; top: 60px; font-size: 13px; overflow-y: scroll; width: 100%; height: 100%;}
    #panel a{color:#000;}
    #panel2 {display: none; width: 100%; margin: 0; padding: 0; background: #fff; z-index: 100; position: fixed; top: 60px; right: 0px; font-size: 13px; overflow-y: scroll; width: 100%; height: 100%;}

    #panel_in2{padding:20px; text-align: center;}
    #panel_in2 .yoyakup{padding-bottom: 0.5em; font-weight: bold;}

    .panel_in_li{font-size: 13px;}

    .menu-trigger,
    .menu-trigger span {
      display: inline-block;
      transition: all .4s;
      box-sizing: border-box;
      z-index: 100;
    }
    .menu-trigger {
      position: relative;
      position: fixed;
      width: 32px;
      height: 16px;
    }
    .menu-trigger span {
      position: absolute;
      left: 0;
      width: 100%;
      height: 2px;
      background-color: #000;
    }
    .menu-trigger span:nth-of-type(1) {
      top: 0;
    }
    .menu-trigger span:nth-of-type(2) {
      top: 7px;
    }
    .menu-trigger span:nth-of-type(3) {
      bottom: 0;
    }
    .menu-trigger.active span:nth-of-type(1) {
      -webkit-transform: translateY(7px) rotate(-45deg);
      transform: translateY(7px) rotate(-45deg);
    }
    .menu-trigger.active span:nth-of-type(2) {
      opacity: 0;
    }
    .menu-trigger.active span:nth-of-type(3) {
      -webkit-transform: translateY(-7px) rotate(45deg);
      transform: translateY(-7px) rotate(45deg);
    }

    #spbatsu{
        width:32px;
        position: absolute;
        top: 20px;
        left: 15px;
    }

    .menu-trigger2,
    .menu-trigger2 span {
      display: inline-block;
      transition: all .4s;
      box-sizing: border-box;
      z-index: 100;
    }
    .menu-trigger2 {
      position: relative;
      position: fixed;
      width: 32px;
      height: 16px;
    }
    .menu-trigger2 span {
      position: absolute;
      left: 0;
      width: 100%;
      height: 2px;
      background-color: #000;
    }
    .menu-trigger2 span:nth-of-type(1) {
      top: 0;
    }
    .menu-trigger2 span:nth-of-type(2) {
      top: 7px;
    }
    .menu-trigger2 span:nth-of-type(3) {
      bottom: 0;
    }
    .menu-trigger2.active span:nth-of-type(1) {
      -webkit-transform: translateY(7px) rotate(-45deg);
      transform: translateY(7px) rotate(-45deg);
    }
    .menu-trigger2.active span:nth-of-type(2) {
      opacity: 0;
    }
    .menu-trigger2.active span:nth-of-type(3) {
      -webkit-transform: translateY(-7px) rotate(45deg);
      transform: translateY(-7px) rotate(45deg);
    }

    #spbatsu2{
        width:32px;
        position: absolute;
        top: 20px;
        left: 15px;
    }


勉強中で知識不足ですので
「ココにこの意味のコードを入力すればこうなる」といった事も教えていただけたら幸いです。
どなたかお分かりになられる方見えましたらお助け下さい。宜しくお願い致します。

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 1

checkベストアンサー

+1

こんにちは。

ベストな修正方法とは思っていませんが、
以下の修正で、意図した動きに近づいているか
と思いますが、いかがでしょうか?

修正前: https://jsfiddle.net/jun68ykt/x2kbtua5/10/

上記においては、ご質問の JS のコードを、ひとつの $(function(){ ・・・ }); の中にまとめています。

修正案: https://jsfiddle.net/jun68ykt/x2kbtua5/25/

修正の要点としては、修正前 の以下のコード

 $(".menu-trigger2").click(function() {
    $("#panel2").slideToggle(350);
    $(".menu-trigger2").toggleClass('active');
    $(".menu-trigger").removeClass('active');
    $("#panel").slideUp(350);
    return false;
  });

でクリックハンドラに設定している関数を、変数 menuTrigger2Handler に入れて
これを、予約や電話番号のリンクをクリックしたときのハンドラとしても使っていることです。

そのために、新たに menu-toggle というクラスを追加していますが、
これは、既存のクラス名 menu-trigger や menu-trigger2 と
意味的にかぶるので、クラス名を改善する余地があります。

また、上記の修正案で

    return $(this).attr('href').match(/^tel:/) || $(this).text() == '予約' ? true : false;


とややこみ入ったコードになってしまっているように、var menuTrigger2Handler に
入れる関数が返す値の true だったり false だったりを適切に作る必要があるかと
思います。

以上参考になれば幸いです。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/02/15 15:32

    こんにちは、初めまして。
    そして丁寧な回答ありがとうございます!
    サンプルのページも修正前、修正後に分かれており大変わかりやすく感激いたしました。
    無事自分が想像していた動きになり大変満足しております。
    本当にありがとうございました!

    キャンセル

  • 2018/02/15 15:56

    解決されたようで、よかったです!

    ひとこと付け加えますと、修正案のコードで、
    var menuTrigger2Handler
    という変数に関数を代入していますが、こうしないで
    修正前のように、

    $(".menu-toggle").click( function() { ・・・} );

    と書いてももちろんよいのですが、関数を変数に入れて、
    それをどこにでも使い回せるものとして扱えることを
    意識していると、JavsScript の関数が
    https://ja.wikipedia.org/wiki/第一級オブジェクト
    であることのメリットをできるだけ享受するようなコードに
    つながります。

    キャンセル

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

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

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

  • JavaScript

    22144questions

    JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

  • HTML

    12619questions

    HTMLとは、ウェブ上の文書を記述・作成するためのマークアップ言語のことです。文章の中に記述することで、文書の論理構造などを設定することができます。ハイパーリンクを設定できるハイパーテキストであり、画像・リスト・表などのデータファイルをリンクする情報に結びつけて情報を整理します。現在あるネットワーク上のほとんどのウェブページはHTMLで作成されています。

  • jQuery

    8777questions

    jQueryは、JavaScriptライブラリのひとつです。 簡単な記述で、JavaScriptコードを実行できるように設計されています。 2006年1月に、ジョン・レシグが発表しました。 jQueryは独特の記述法を用いており、機能のほとんどは「$関数」や「jQueryオブジェクト」のメソッドとして定義されています。