How to change css property when you click on button?-CodePudding

I want when i click in each one of them to pop the span element with class color but only for that button.

[![enter image description here][1]][1]

I only achieved when i click on one of them to change the visibility to visible on all elements with class colors.

my Code : https://jsbin.com/govowakasa/edit?js

document.getElementById("btn-apple").addEventListener("click", function(e) {
  showColor()
});

function showColor() {
  const colors = document.getElementsByClassName("color");
  for (let i=0; i<colors.length; i  ) {
    colors[i].style.visibility = 'visible';
}};

CodePudding user response：

You can simply add this line

 fruits[i].style.backgroundColor = 'red'

CodePudding user response：

You can not use the same ID for more than 1 element.

I replaced the IDs of buttons with a new class name fruit-btn.
Then I made an array in JS of both buttons and color spans.
Then I looped through each button to add an EventListener.

In the event I added another loop that hides every color span and then makes the i span visible. There is no need for a function.

const fruit_btn = document.getElementsByClassName("fruit-btn");
const color_spans = document.getElementsByClassName("color");

for (let i = 0; i < fruit_btn.length; i  ) {
  fruit_btn[i].addEventListener("click", function(e) {
    for (let i = 0; i < fruit_btn.length; i  ) {
      color_spans[i].style.visibility = "hidden";
    }
    color_spans[i].style.visibility = "visible";
  });
}

body {
  font-size: 2rem;
}

li {
  margin: 20px;
  list-style: none;
}

button i {
  cursor: pointer;
  font-size: 3rem;
}

.color {
  margin: 10px;
  border: 1px solid black;
  background-color: blanchedalmond;
  display: inline-block;
  padding: 10px;
  visibility: hidden;
}

<!DOCTYPE html>
<html lang="en">

<head>
  <meta charset="UTF-8">
  <meta http-equiv="X-UA-Compatible" content="IE=edge">
  <meta name="viewport" content="width=device-width, initial-scale=1.0">
  <title>Fruits</title>
  <link rel="stylesheet" href="list.css">
  <link rel="stylesheet" href="https://cdnjs.cloudflare.com/ajax/libs/font-awesome/5.11.2/css/all.min.css" integrity="sha512-0S nbAYis87iX26mmj/ fWt1MmaKCv80H Mbo Ne7ES4I6rxswpfnC6PxmLiw33Ywj2ghbtTw0FkLbMWqh4F7Q==" crossorigin="anonymous" referrerpolicy="no-referrer"
  />
</head>

<body>
  <p>
    FRUITS
  </p>
  <ul>
    <li >
      <button ><i ></i></button>
      <span >red and yellow</span>
    </li>
    <li >
      <button ><i ></i></button>
      <span >orange</span>
    </li>
    <li >
      <button ><i ></i></button>
      <span >yellow</span>
    </li>
    <li >
      <button ><i ></i></button>
      <span >red and green</span>
    </li>
  </ul>
  <script src="/week-4/index.js"></script>
</body>

</html>

I would suggest to add/remove a Class instead of adding/removing directly CSS styling.