ハンバーガーメニューを「閉じる」ボタンで閉じたい

Question

バーガーメニューを閉じたいのですが、方法がわかりません。
単純にメニューの中に「閉じる」項目を作り、それを押したら閉じる仕組みです。
リンク先を'#'にしたのですが、メニューは閉じませんでした。
ご教示お願い致します。

<div class="sp sp_navi">
  <img id="sp_navi_btn" class="u-max-full-width" src="img/sp_navi.png" alt="sp_navi" srcset="img/[e-mail] 2x" />

  <ul class="sp_navi_li">
    <li><a href="index.html">TOPページ</a></li>
    <li><a href="sp_lesson.html">内容</a></li>
    <li><a href="faq.html">内容</a></li>
<li><a href="sp_voice.html">内容</a></li>
    <li><a href="company.html">内容</a></li>
    <li><a href="contact.html">内容</a></li>
    <li><a href="#">閉じる</a></li>
  </ul>
</div>

<script src="http://ajax.googleapis.com/ajax/libs/jquery/3.0.0/jquery.min.js"></script>
<script>
  $('.container').click(function(){
$('.sp_navi').removeClass('sp_navi_li');
});

  $(function(){
    $(document).ready(function(){

      $("#sp_navi_btn").click(function () {
        $(this).next().slideToggle();
      });

    });
  });
</script>

Comments

[m.ts10806]

どこにも「閉じるを押したら」の処理が書かれていないことは認識済みですか？そもそも元々どうなったら閉じるときに作られていますか？そこが分かれば同じ処理を呼び出すだけかと思います。

[yasu1110]

すみません、なにぶん知識が少なくて。。「リンクを押したらメニューが消える」というアクションが必要ということでしょうか？考えていた閉じ方は、バーガーメニュー内の「閉じる」リンクを押したらメニューが消えるイメージです。

[m.ts10806]

提示htmlにcontainerがないので全て把握できるわけではないですけど、「リンクを押したら」ではなく「閉じるを押したら」です（最初にそう書いたつもりですが）まずは「閉じるを押した」というclickイベントを取得するように書いてみてください。もう1つ、コメントの後半に書いてますが、「メニューが閉じる」は今の状態でどこで対応していますか？それともそもそも閉じる自体ない状態ですか？

Answer 1

いろいろコメントで確認はしていますが、失礼ながら、おそらく提示のコードをあまり理解せずに使われているように思われますので、ざっと気になった点とあくまで1案として回答します。

ちょっと細かいですが。
画像がどのようなものかわからないですが、動きを見た感じ「ハンバーガーメニュー」ではなく普通の「ドロップダウンメニュー」ではないでしょうか。
画像次第ですけど、「ハンバーガーメニュー」は確かメニューのボタンが横の三本線になっているもの、そのボタン自体のことを言ったような。
機能的には「ドロップダウンメニュー」かなと。

• 「閉じるを押す」イベント自体が書かれていない

提示のJavaScriptコードの「click」で指定されている処理が「クリック時の処理」ですが、
見たところclass="container"をクリックしたときとid="sp_navi_btn"をクリックしたときの処理しかありません。
containerの方はhtmlの提示がないためどうなっているかわかりませんが、コードを読んだ限りだと、sp_navi_btnクリック時にその次の要素をslideToggle()するように書かれていますね。
slideToggle()はslideDown/slideUpを切り替えるものｄす。
ここでメニューの開閉が行われています。

• 冗長な処理がある

下記は結局同じことが繰り返し書いてあるだけのように思います。
1つだけで良いです。

  $(function(){
    $(document).ready(function(){

jQueryの基本 - $(document).ready

上記３つの書き方も同じ動作をします。HTMLの読み込みが終わった時点で //なにかしらの処理 が実行されます。

• click直接よりonから呼び出したほうが良い

jQuery 便利なonを使おう（on click)

あと既に開閉の仕組みが組まれているので同じ物を使いまわしたいですよね。
jQueryでは複数の要素に対して同じイベントを拾うように書けます。
[JS]jQuery で複数の要素に同じ処理をする方法

上記を踏まえて、また「閉じる」をきちんと認識するためにhtmlにも若干変更を加えるとこんな感じになるかなと。
（あくまで1つのやり方です。これだけが唯一ではありません）

<li><a href="#" class="close">閉じる</a></li>

  $(function(){
      $("#sp_navi_btn,.close").click(function () {
        $("#sp_navi_btn").next().slideToggle();
      });
      $('.container').click(function(){
          $('.sp_navi').removeClass('sp_navi_li');
      });
  });

＊一応動作確認済みですが、想定通りのものと違う場合、参考に色々やってみてください。

Comments

[yasu1110]

お礼が遅くなり申し訳ありません。
また大変丁寧にご教示くださり、ありがとうございました。
改めて基礎から勉強してみようと思います。