回答編集履歴
2
(コメントを受けて)追記
answer
CHANGED
|
@@ -9,4 +9,31 @@
|
|
|
9
9
|
|
|
10
10
|
1度でもビューポート幅が768pxを超えるとhoverハンドラが仕込まれてしまうので、その後、たとえ768px以下になったとしてもhoverイベントに反応してしまうんだと思います。768px以下になったときにhoverハンドラを解除してみたらどうでしょうか? 解除方法は `$('.menu__item').off('mouseenter mouseleave');` です(詳しくは[マニュアル(https://api.jquery.com/hover/)](https://api.jquery.com/hover/)参照)。
|
|
11
11
|
|
|
12
|
-
同様に、768px以上になったら、クリックイベントを解除する必要がありますね。
|
|
12
|
+
同様に、768px以上になったら、クリックイベントを解除する必要がありますね。
|
|
13
|
+
|
|
14
|
+
---
|
|
15
|
+
**[コメントを受けて、改修例を追加]**
|
|
16
|
+
|
|
17
|
+
改修後の`handle()`関数は↓こんな感じになると思います。
|
|
18
|
+
|
|
19
|
+
```javascript
|
|
20
|
+
function handle(mm) {
|
|
21
|
+
if (mm.matches) {
|
|
22
|
+
$(function() {
|
|
23
|
+
$(".menu__item").on("click", function() {
|
|
24
|
+
$(".submenu", this).slideToggle();
|
|
25
|
+
});
|
|
26
|
+
$(".menu__item").off("mouseenter mouseleave"); // ←これと
|
|
27
|
+
});
|
|
28
|
+
} else {
|
|
29
|
+
$(function() {
|
|
30
|
+
$('.menu__item').hover(function() {
|
|
31
|
+
$(".submenu:not(:animated)", this).slideDown();
|
|
32
|
+
}, function() {
|
|
33
|
+
$(".submenu", this).slideUp();
|
|
34
|
+
});
|
|
35
|
+
$(".menu__item").off("click"); // ←これ
|
|
36
|
+
});
|
|
37
|
+
}
|
|
38
|
+
}
|
|
39
|
+
```
|
1
コメントを受けて補足
answer
CHANGED
|
@@ -1,3 +1,12 @@
|
|
|
1
1
|
メニュー(`a`)の`href`属性値が`""`なので、クリックするたびにページがリロードされてるんだと思います。
|
|
2
2
|
|
|
3
|
-
たとえば、`<a class="menu__item__a" href="#">メニュー4</a>`などとすれば、ちゃんとサブメニューが開閉しましたよ。
|
|
3
|
+
たとえば、`<a class="menu__item__a" href="#">メニュー4</a>`などとすれば、ちゃんとサブメニューが開閉しましたよ。
|
|
4
|
+
|
|
5
|
+
---
|
|
6
|
+
**[コメントを受けて、hoverの件などについて補足]**
|
|
7
|
+
|
|
8
|
+
失礼しました、hoverのことはスルーしてました(私はタッチデバイスで試してたので)。
|
|
9
|
+
|
|
10
|
+
1度でもビューポート幅が768pxを超えるとhoverハンドラが仕込まれてしまうので、その後、たとえ768px以下になったとしてもhoverイベントに反応してしまうんだと思います。768px以下になったときにhoverハンドラを解除してみたらどうでしょうか? 解除方法は `$('.menu__item').off('mouseenter mouseleave');` です(詳しくは[マニュアル(https://api.jquery.com/hover/)](https://api.jquery.com/hover/)参照)。
|
|
11
|
+
|
|
12
|
+
同様に、768px以上になったら、クリックイベントを解除する必要がありますね。
|