Question

Je me demande si ma fonction jquery est correcte

<script>
window.blinker = setInterval(function(){
  if(window.alerta){
    $('a.cadastrotopo').css('color','#346698');
    $('a.cadastrotopo').css('text-decoration','underline');
      window.alerta=false;
    }
    else{
      $('a.cadastrotopo').css('color','#000');
      $('a.cadastrotopo').css('text-decoration','none');
      window.alerta = true;
    }
},500);
</script>

travaille ok, mais je me demande si je fais de la bonne façon.

Je vous remercie.

Était-ce utile?

La solution

Personnellement, j'utiliser CSS plus, en particulier les classes:

a.cadostropo {
  color: #000;
  text-decoration: none;
}
a.alert { 
  color: #346698;
  text-decoration: underline;
}

et la solution devient trivial:

setInterval(toggleAlert, 500);

function toggleAlert() {
  $("a.cadostropo").toggleClass("alert");
}

Une note de côté:. Au lieu de plusieurs appels css() vous pouvez utiliser des objets anonymes pour spécifier plusieurs propriétés

$("a.cadostropo").css({color: "#346698", textDecoration: "underline"});

Cela étant dit, je préfère ne pas utiliser la manipulation hardcoded CSS comme celui-ci. cours de désapprouvaient. Ils sont beaucoup plus souples et vous n'avez pas à vous soucier des changements destructeurs et les faire reculer.

Autres conseils

Oui, vous êtes, bien que vous pouvez combiner les déclarations de CSS pour ressembler à ceci.

.css({'color' : "#346698", 'text-decoration' : "underline"});
Licencié sous: CC-BY-SA avec attribution
Non affilié à StackOverflow
scroll top