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

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

ただいまの
回答率

87.95%

JavaScriptにてドロップダウンメニューでサブリストを歯抜けで作成した際に表示されるサブリストをずれないように表示させたい

解決済

回答 2

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 126

score 5

前提・実現したいこと

JavaScriptにてドロップダウンメニューを作成中です。
希望としては下記の様に「メインリスト」にマウスがホバーした時
「サブリスト」が表示されるようにしたいです。
イメージ説明

発生している問題・エラーメッセージ

「メインリスト」と「サブリスト」の数が同じであればドロップダウンメニューが出来るのですが歯抜けの様な状態にすると別な「メインリスト02」にマウスがホバーした時「サブリスト」が表示されてしまいます。
イメージ説明

該当のソースコード

<body>
  <div class="wrap">
    <div class="ly_box">
      <ul class="ly_horizontal">
        <li><a href="#">メインリスト01</a>
          <ul>
            <li><a href="#">サブリスト01</a></li>
            <li><a href="#">サブリスト02</a></li>
            <li><a href="#">サブリスト03</a></li>
          </ul>
          <!-- /.js_sub -->
        </li>
        <li><a href="#">メインリスト02</a></li>
        <li><a href="#">メインリスト03</a>
          <ul>
            <li><a href="#">サブリスト01</a></li>
            <li><a href="#">サブリスト02</a></li>
            <li><a href="#">サブリスト03</a></li>
          </ul>
          <!-- /.js_sub -->
        </li>
        <li><a href="#">メインリスト04</a></li>
        <li><a href="#">メインリスト05</a></li>
      </ul>
      <!-- /.ly_horizontal -->
    </div>
    <!-- /.ly_box -->
  </div>
  <!-- /.wrap -->
  <script src="main.js"></script>
</body>
</html>
body, ul {
  margin: 0;
  padding: 0;
}

body {
  border-top: 5px solid rgb(0, 0, 0);
  background: rgb(245, 245, 245);
}

ul {
  list-style: none;
}

.wrap {
  width:750px;
  margin: 20px auto;
}

.ly_box {
  height: 250px;
  margin-bottom: 20px;
}

.ly_horizontal {
  display: flex;
}

.ly_horizontal li {
  position: relative;
}

.hidden {
  height: 0px;
  visibility: hidden;
}

.ly_horizontal a {
  display: block;
  width: 150px;
  height: 50px;
  line-height: 50px;
  text-align: center;
  text-decoration: none;
  color: rgb(0, 0, 0);
  background: rgb(220, 220, 220);
}

.ly_horizontal a:hover {
  background: rgb(200, 200, 200);
}
const mainList = document.querySelectorAll('.ly_horizontal > li');
const subList = document.querySelectorAll('.ly_horizontal ul');

subList.forEach(list =>{
  list.classList.add('hidden');
});

mainList.forEach((mlist, index) =>{
  mlist.addEventListener('mouseover', ()=>{
    subList.forEach((slist, index2)=>{
      if(index === index2){
        slist.classList.remove('hidden');
      };
    });
  });

  mlist.addEventListener('mouseout', ()=>{
    subList.forEach((slist, index2)=>{
      slist.classList.add('hidden');
    });
  });
});

試したこと

if(index === index2){
        slist.classList.remove('hidden');
      };


「index」「index2」の値で比べているのでこの状態になっている事は分かるのですが、ここからどうすればずれなく表示させる事が出来るか分かりませんでした。

つたない説明で申し訳ありませんがご教授お願い致します。

補足情報(FW/ツールのバージョンなど)

端末:Windows10 Home
テキストエディタ:VS Code
ブラウザ:Chrome

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 2

checkベストアンサー

+1

「index」「index2」の値で比べているのでこの状態になっている事は分かる

とのことなので、原因はしっかり把握されていることと思います。
なので、配列の添え字で判断するのではなく、mouseoverした際のElementから、自身の子要素のULを抽出する方針に変えましょう。

mainList.forEach((mlist, index) =>{
  // mouseoverが発生した際に主体をイベントリスナを付与した要素にしたいので、
  // コールバック関数をラムダ式からfunctionに変える。
  // そうするとマウスオーバーしたli要素がthisになる。
  mlist.addEventListener('mouseover', function(e){
    this.querySelectorAll('ul') // mouseoverしたli要素の子要素であるulを取得
      .forEach( e2 => e2.classList.remove('hidden'));  // querySelectorAllで取得する理由は、サブメニューが存在しない要素にマウスオーバーした際のnullチェックを省略するため
  });
  // mouseout側も同様に実装
  mlist.addEventListener('mouseout', function(e){
      this.querySelectorAll('ul').forEach( e2 => e2.classList.add('hidden'));
  });
});

codepenでのデモ

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2021/06/11 18:20

    ご回答いただき誠にありがとうございます。
    「コールバック関数をラムダ式からfunctionに変える」という部分はまだ未熟で完全な理解には至ってはいませんが、これを元に再度勉強し直したいと思います。
    理想通りの動きになりました。
    本当にありがとうございました。

    キャンセル

  • 2021/06/11 18:35

    問題が解決できて何よりです。
    functionとラムダ式(アロー関数)とthisの関係については、手始めに以下のような記事を読んでみて、thisに入るものが違うんだぞということを理解してから、深い仕様を追いかけるのが良いかと思います。

    https://qiita.com/haduki1208/items/f9cd375d5af157b00af9
    https://qiita.com/takkyun/items/c6e2f2cf25327299cf03

    キャンセル

  • 2021/06/11 18:38

    「functionとラムダ式(アロー関数)とthisの関係について」
    ご丁寧にリンク先まで教えて頂きありがとうございます。
    実は「サブリスト」がある「ul」だけを特定する方法が分からず困っていました。
    本当にありがとうございました。

    キャンセル

+1

.ly_box ul{
list-style:none;
padding:0;
}
.ly_box> ul> li >a,.ly_box> ul> li >ul> li{
background-Color:lightgray;
}
.ly_box> ul>li>a:hover,.ly_box> ul> li >ul> li:hover{
background-Color:gray;
}
.ly_box> ul{
display:flex;
}
.ly_box> ul>li:not(:hover)>ul{
display:none;
}


※色変えも追加しときました

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2021/06/11 16:01

    ご回答いただきありがとうございます。
    私の文章がつたない説明で意図が伝わっておらず申し訳ありませんでした。
    念の為確認ですが、JavaScriptとの組み合わせて教えて頂いたコードを追記すれば再現できるということでしょうか?
    実は教えて頂いたコードの追記、修正では希望の動作にならなかったのですが。

    キャンセル

  • 2021/06/11 17:26

    あれ・・イメージ違いましたか、失礼しました

    https://codepen.io/yambejp/pen/oNZagVr

    キャンセル

  • 2021/06/11 18:22

    再度回答頂きありがとうございます。
    CSSでの再現という事でご教授頂きありがとうございます。
    私は必要以上にクラス名をつけてしまう時があるので今回教えて頂いたコードも今後の参考にさせて頂きたいと思います。
    本当にありがとうございました。

    キャンセル

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

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

関連した質問

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

  • トップ
  • JavaScriptに関する質問
  • JavaScriptにてドロップダウンメニューでサブリストを歯抜けで作成した際に表示されるサブリストをずれないように表示させたい