【JavaScript】querySelectorAllで取得したn個のli要素に関して、2番目以降のli要素にも正常に挙動させたいです。

Question

######【内容】
顧客管理システムを開発中で、顧客一覧画面にて、顧客の名前にマウスオーバーすると電話番号とメアドが表示されるようにしたいです。

1番目の顧客についてはマウスオーバー中は電話番号とメアドが表示されますが、
2番目の顧客においてはマウスオーバーをしているのに、1番目の顧客の部分で1番目の顧客の情報が表示されており、2番目の顧客においては何も表示されません。

######【仮説】
querySelectorAllで取得できているli要素が1番目のみ？
→console.logで確認してみたところ、取得自体は1番目も2番目も問題なくできているよう。

一度、配列に変換してからforEachだと上手くいくか？
→下記コードの通りですが、結果としては上手くいかず、同じように1番目だけが表示されています。

昨日より色々と調べておりますが、どうにも解決の糸口が掴めません。皆様のお力をお借りしたいです。
何卒宜しくお願い致します。

function clientIndex() {
  const clients = [...document.querySelectorAll(".lists")];
  const phoneEmail = document.getElementById("phone-email");

  clients.forEach(function(target) {
    target.addEventListener('mouseover', function(){
      phoneEmail.setAttribute("style", "display:block;")
    })
    target.addEventListener('mouseout', function(){
      phoneEmail.removeAttribute("style", "display:block;");
    })
  })
}

window.addEventListener('load', clientIndex)

 <div class="clients-index">
    <ul class="clients">
      <div class="child-a">
        <h3 class="text-title">登録済み顧客一覧</h3>
      </div>
      <% @allclients.each do |c| %>
      <% if c.user_id == current_user.id %>

        <li class="lists">
          <%=link_to client_path(c.id), method: :get, class:"client-name" do %>
            <%= c.last_name %><%= " " %><%= c.first_name %>
          <% end %>
          <div class="phone-email hidden", id= "phone-email">
            <%= c.phone_num %><br><%= c.email%>
          </div>
        </li>

      <% end %>
      <% end %>
    </ul>
  </div>

Answer 1

.listsは複数なので、各々にイベントがあたっていますが、
#phone-emailは、IDのため、一個しか存在してはいけなく、
例えHTML上で複数個あっても、
一番最初の要素しか取得されません、
document.getElementByIdでは。

なので、各々の電番・メールアドレスを表示する、という仕組みに作り替えねばなりません。

Comments

[tomas0528]

ありがとうございます！
phone-emailの方もそれぞれ表示させるための記述が必要と気が付きませんでした。
ご回答ありがとうございました！

Answer 2

id="phone-email" を持つ要素が単一文書中に複数存在します。

  const phoneEmail = document.getElementById("phone-email");

このコードで取得されるのは、id="phone-email" を持つ要素の中で一番最初のものです。

イベントリスナの中で対応する phone-email 要素を上手いこと見つける必要があります。たとえば event.target.querySelector('.phone-email') とか。

Comments

[tomas0528]

ありがとうございます！
mouseoverとmouseoutそれぞれに、phoneEmail = target.querySelector('.phone-email')
という記述を入れると上手く行きました！
表示させる情報の方に気が回っていませんでした。
本当にありがとうございました。