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

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

ただいまの
回答率

88.77%

グローバルメニューを正しく動作させたい

解決済

回答 1

投稿 編集

  • 評価
  • クリップ 1
  • VIEW 246

lingwood

score 40

前提・実現したいこと

グローバルメニューを作成しています。
【PC】768px~
→ 横並びメニュー
→ メニューをホバーすると子メニューがスライドダウンする
→ スマホ時のクリックイベントを解除したい

【スマホ】768px以下
→ ハンバーガーメニュー
→ ハンバーガーメニューをクリックしてメインメニューを表示する
→ メインメニューの中にある子メニューをクリックしてスライドダウンで表示させたい
※この部分をPCサイズ表示ではホバーにしたいです。

いろいろ参考にしながら3回作ってみたのですが、
それぞれどこかしらに問題があるようで、挙動がおかしいです。
どこが悪いのか、ご教示いただけないでしょうか。

一応、1についてはローカルで作成したところうまく動いていたのですが、
Codepenやお客様のデモ環境にアップロードすると後述の通り、
「スマホサイズにした時に、オンマウスすると子メニューが開いてしまう」
といった現象が発生してしまいます。

以下、デモになります。
基本的にはクロームのデベロッパーツールを使ってブラウザ動かしながら作業しています。

1 https://codepen.io/lingwood/pen/gOpyLwZ

→ PCサイズの時にはJQueryでホバーアクションつけています。

問題点1
PCサイズで表示した際に、オンマウスでメニュー表示はされますが、トリガーとなっているメニューをクリックしても子メニューがスライドアップ、スライドダウンが発生してしまいます。

問題点2
スマホサイズにした時に、オンマウスすると子メニューが開いてしまいます。

2 https://codepen.io/lingwood/pen/JjdVbwZ

→1をもっとスマートにしたい、JQueryを簡単に書きたい

問題点1
PCサイズで表示した際に、メニュー2の子メニューが開きっぱなしになってしまっています。

問題点2
スマホサイズで表示した際に、オンマウスでスライドダウンしてしまいます。
また、クリックアクションが動きません。

3 https://codepen.io/lingwood/pen/ZEGZBNJ

→PCサイズの時にはCSSでホバーアクションつける記述になっています。

問題点1
スマホからPCサイズにリサイズするとホバーアクションが解除されてしまっています。(サンプルデモメニューの1)

問題点2
スマホサイズにした時に子メニューが開きっぱなしになってしまいます。
またこれを解決しようとして.hide()やCSSでdisplay:noneを付けると今度はスライドしなかったり
いろいろと問題が発生します。

1週間くらい考えこんでおりまして・・・どうか助けていただけないでしょうか
よろしくお願いします。

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 1

checkベストアンサー

+1

だいぶ混乱しているコードなので、もういっそPC用のメニューとスマホ用のメニューを別に作って、メディアクエリで不要な方を表示しないようにしてはいかがですか?

コメントを受けて追記

要素を二つに分ける、というご提案はお気に召さなかったようなので、別のご提案をいたします。3番目の方針をベースにします。

  1. まず、前提として「メニューの開閉はすべてCSSで表現する」ことに統一してください。
    JavaScript と CSS が重複していることが混乱のもとです。jQueryのアニメーションはすべて削除しましょう。

  2. 次に、全ての状態を整理してください。そしてこれを全てCSSで反映できるようにクラスをトグルするスクリプトを書きます。
    ・PCサイズかスマホサイズか
    ・オープンかクローズか
    ・ホバーか非ホバーか

  3. 最後に、CSSで全ての状態のスタイルを表現します。

以上、ご不明の点がありましたら、コメント欄にてご連絡ください。

コメントを受けて追記2

これは例ですが、

$(function() {
  $(".gnavi14MenuItem._trigger, .gnavi14ChildMenuDeco").on("click", function() {
    $(".gnavi14ChildMenu", this).toggleClass('open');
  })
});


とすれば、
たとえば、SPでオープンになっていてホバーしている状態は、

@media screen and (max-width: 768px) {
  .gnavi14MenuItem._trigger:hover .gnavi14ChildMenu.open{
  }
}


でスタイルをつけられますよね?

状態が2*2*2=8通りなので、全部列挙したとしても8パターンで済みます。
実際には、SPの時にはホバーについて書かなくてもいいですし、PCの時にはオープンの状態について書かなくてもいいので、もっとパターンが減るはずです。

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2020/04/07 11:51

    Lhankor_Mhy 様
    ありがとうございます!できました!!
    今までネット上にあるプログラムをちょっといじる程度でやってきたので、thisについて英語の直訳でしか理解していませんでした。ちゃんと調べたところ、複数の要素からクリック要素だけを取得することもできるとのことで、考えてみればそういう捉え方も確かにできることと大変勉強になりました。
    最後に2つ教えていただけませんでしょうか?
    まず、アコーディオンを開く際にスムーズに開くようにしたかったため、スマホ時はtoggleClassからslideTggleにしました。
    ひとつめの質問ですが、
    toggleClassなどでクラスを付与した場合に、CSSでうまくslideToggleのようにスムーズにメニューを開くことができませんでした。CSSでもこれを解決する方法(コード)はあるのでしょうか?
    もうひとつの質問ですが、
    slideToggleを指定のWindowサイズで完全に切り分けて実行させたり解除する方法はあるのでしょうか?
    例えば今回のコードでは、スマホ表示で子メニューを閉じた際にslideToggleが入っているためPC表示に戻した際にdisplay:none;が継承されてしまい、それをメディアクエリでmin-width:768px;の時、display:block !importantとすることで解決しています。!importantを使わないでも解決できる方法はありますでしょうか?
    何度もすみませんが、ご教示いただけましたら本当にうれしく思います。
    よろしくお願い致します。

    キャンセル

  • 2020/04/07 12:27

    >CSSでもこれを解決する方法(コード)はあるのでしょうか?

    transition でいいのではないですか?

    >slideToggleを指定のWindowサイズで完全に切り分けて実行させたり解除する方法はあるのでしょうか?

    すでに、CSSで表現するように統一するか、サイズごとに要素を用意することをお勧めいたしました。

    どうしても、CSSとJavaScriptを混在させたいのであれば、丁寧に一つずつロジックで切り分けるしかないと思いますよ。たとえば、datasetに状態を持たせておいて、resizeイベントの際にすべてをチェックして不整合のある状態であれば変更する、などです。

    キャンセル

  • 2020/04/07 13:34

    ありがとうございます!
    以前にも「slideToggle 解除」などで調べた際にめぼしいものがなく今日まであやふやに考えてきましたがとてもすっきりしました。
    何とか納品に間に合いそうです。いろいろとご教示いただきありがとうございました。

    キャンセル

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

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

関連した質問

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