Javascript - getAttribute () ne fonctionne que pour le premier élément [duplicate]

Dec 11 2020

c'est mon HTML:

<div class="tool-top">
    <ul>
        <li><a role="button" id="button" value="bold"><i class="fas fa-bold"></i></a></li>
        <li><a role="button" id="button" value="italic"><i class="fas fa-italic"></i></a></li>
    </ul>
</div>

c'est mon JS:

window.onload = function(){
    document.getElementById('button').onclick = function(){
    
        var a = this.getAttribute('value');
        console.log(a);
    }
}

J'obtiens 'gras' en cliquant sur le premier bouton mais rien dans le second

J'ai regardé sur cette page, mais elle a écrit avec Jquery.

Réponses

blex Dec 11 2020 at 00:01

Un identifiant doit toujours être unique. Vous avez deux éléments avec le même ID ici, utilisez plutôt une classe:

window.onload = function() {
  var btns = document.getElementsByClassName('button');

  for (var i = 0; i < btns.length; i++) {
    btns[i].onclick = function() {
      var a = this.getAttribute('value');
      console.log(a);
    };
  }
};
<link rel="stylesheet" href="https://cdnjs.cloudflare.com/ajax/libs/font-awesome/5.15.1/css/all.min.css" integrity="sha512-+4zCK9k+qNFUR5X+cKL9EIR+ZOhtIloNl9GIKS57V1MyNsYpYcUrUeQc9vNfzsWfV28IaLL3i96P9sdNyeRssA==" crossorigin="anonymous" />
<div class="tool-top">
  <ul>
    <li><a role="button" class="button" value="bold"><i class="fas fa-bold"></i></a></li>
    <li><a role="button" class="button" value="italic"><i class="fas fa-italic"></i></a></li>
  </ul>
</div>

Si vous avez jQuery (vous avez ajouté la balise à votre question) , vous pouvez le faire:

window.onload = function() {
  $('.button').click(function() {
    var a = this.getAttribute('value');
    console.log(a);
  });
};
<script src="https://cdnjs.cloudflare.com/ajax/libs/jquery/3.3.1/jquery.min.js"></script>
<link rel="stylesheet" href="https://cdnjs.cloudflare.com/ajax/libs/font-awesome/5.15.1/css/all.min.css" integrity="sha512-+4zCK9k+qNFUR5X+cKL9EIR+ZOhtIloNl9GIKS57V1MyNsYpYcUrUeQc9vNfzsWfV28IaLL3i96P9sdNyeRssA==" crossorigin="anonymous" />
<div class="tool-top">
  <ul>
    <li><a role="button" class="button" value="bold"><i class="fas fa-bold"></i></a></li>
    <li><a role="button" class="button" value="italic"><i class="fas fa-italic"></i></a></li>
  </ul>
</div>