web-dev-qa-db-fra.com

JavaScript onClick addClass

Je ne sais pas trop pourquoi mon code ne fonctionne pas correctement, j'espère que vous me direz ce que j'ai fait de mal… .. Je souhaite mettre en surbrillance l'onglet de navigation lorsque vous cliquez.

HTML:

<header class="mainheader">
  <!-- Obrazek tutaj-->
  <nav>
    <ul>
      <li><a id="a-home" onclick="dodajAktywne(this)" href="index.html">Home</a></li>
      <li><a id="a-omnie" onclick="dodajAktywne(this)" href="omnie.html">O mnie</a></li>
      <li><a id="a-kurs" onclick="dodajAktywne(this)" href="kurs.html">Kurs</a></li>
      <li><a id="a-kontakt" onclick="dodajAktywne(this)" href="kontakt.html">Kontakt</a></li>
    </ul>
  </nav>
</header>

JavaScript:

   function dodajAktywne(elem) {
    var a = document.getElementsByTagName('a')
    for (i = 0; i < a.length; i++) {
        a[i].classList.remove('active');
    }
    elem.classList.add('active');
}

CSS:

.active {
    color: blue;
    background-color: #cf5c3f;
}
javascripthtmlcssonclick
9
Sheil

Vous pouvez simplifier votre JavaScript pour:

Fiddle

function dodajAktywne(elem) {
    // get all 'a' elements
    var a = document.getElementsByTagName('a');
    // loop through all 'a' elements
    for (i = 0; i < a.length; i++) {
        // Remove the class 'active' if it exists
        a[i].classList.remove('active')
    }
    // add 'active' classs to the element that was clicked
    elem.classList.add('active');
}

Si vous passez le paramètre this dans votre code HTML à:

<header class="mainheader">
    <!-- Obrazek tutaj-->
    <nav>
        <ul>
            <li><a id="a-home" onclick="dodajAktywne(this)" href="#">Home</a>
            </li>
            <li><a id="a-omnie" onclick="dodajAktywne(this)" href="#">O mnie</a>
            </li>
            <li><a id="a-kurs" onclick="dodajAktywne(this)" href="#">Kurs</a>
            </li>
            <li><a id="a-kontakt" onclick="dodajAktywne(this)" href="#">Kontakt</a>
            </li>
        </ul>
    </nav>
</header>

Note: J'ai changé l'attribut href en #, vous devrez le remettre dans vos pages .html

Sinon, vous pouvez le faire sans JavaScript, en utilisant le :focus de CSS:

Fiddle

a:focus {
    color: blue;
    background-color: #cf5c3f;
}
12
Weafs.py

Ce code mettra en surbrillance l'onglet de navigation sans qu'il soit nécessaire d'inclure onclick dans le code HTML, même s'il ne supprime pas la couleur de fond si un autre onglet est cliqué.

    document.onclick = function(event) {
         var target = event.target || event.srcElement;
         target.style.background = '#cf5c3f';
        };

https://codepen.io/mdmcginn/pen/BwrNaj/

1
Michael McGinnis

var list = document.querySelector('ul');
list.addEventListener('click', function(ev) {
  if (ev.target.tagName === 'LI') {
    ev.target.classList.toggle('checked');
  }
}, false);

0
Đức Lê