liを押しても、ハンバーガーメニューが回らないバグのみが残ってしまいました。

その後、自分で変更し、ある程度はできたのですが、
liを押しても、ハンバーガーメニューが回らないバグのみが残ってしまいました。
理由はわかりますでしょうか？

また、ハンバーガーボタンをクリックしてドロワーメニューを閉じる時に、
背景の黒が二重に作られ、より濃い黒になっているように見えるのですが、
そのようになっているでしょうか？

行動規範の内容に同意します

回答1件

ベストアンサー

とりあえず、下記のように対象を減らすと改善しました。
対象を増やした結果、一度のクリックで二重に実行されてないか確認した方がいいです。

javascript
1// 下記をクリックすると実行される
2$('#drawermenu, .overlay-haikeikurokari, .modal-button-wrap').on('click', function() {
3

プログラムは、思った通りには動きません。書いたとおりに動きます。

投稿2016/02/27 03:50

総合スコア2092

退会済みユーザー

2016/02/27 04:41

ありがとうございます。確かになぜかできました。下記のクラス名の付けはずしが二重になっており、回ってから戻ってしまっているのですかね。 $('.modal-button-wrap__button').toggleClass('active');

退会済みユーザー

2016/02/27 05:19

#drawermenuの子要素に#drawermenuと同じ動きをする子要素がありましたそれがかぶっていて二度動いていたのかもしれないです。親要素と子要素に同じメソッドが適応された場合動かなくなるのでしょうか？それとも二度動くのでしょうか？

2016/02/27 05:57

両方で同じ処理が実行されます。addClassの場合は2回以上実行されてもクラスが付けられるだけですが、toggleClassの場合は実行した回数分付け外しになります。実行する処理によるので、不用意なバインドは避けるべきです。

退会済みユーザー

2016/02/27 07:16

2回つけはずしがされて、結果的にアクションがおきないように人の目に見えていたのですね。

退会済みユーザー

2016/02/27 07:22

bindとは下記のことでしょうか？ http://uhyohyo.net/javascript/11_2.html

行動規範の内容に同意します

あなたの回答

tips

プレビュー

行動規範の内容に同意します

質問の解決につながる回答をしましょう。サンプルコードなど、より具体的な説明があると質問者の理解の助けになります。また、読む側のことを考えた、分かりやすい文章を心がけましょう。

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

ただいまの回答率
85.48%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

関連した質問