Syntaxe javascript

Résolu/Fermé
inspiring Messages postés 1813 Date d'inscription vendredi 28 janvier 2011 Statut Membre Dernière intervention 31 décembre 2018 - 25 oct. 2014 à 17:40
inspiring Messages postés 1813 Date d'inscription vendredi 28 janvier 2011 Statut Membre Dernière intervention 31 décembre 2018 - 25 oct. 2014 à 21:19
Bonjour,

je ne suis pas un as du JS donc je me tourne vers vous pour une question que beaucoup trouveront simple:

J'ai écrit ce bout de code qui me sert à enlever ou ajouter une class au click

$(".op-open-btn").click(function(){
		$("#overlay-menu").removeClass("op-menumasked");
        $("#overlay-menu").addClass("op-menuvisible");
});
$(".op-close").click(function(){
		$("#overlay-menu").removeClass("op-menuvisible");
        $("#overlay-menu").addClass("op-menumasked");
});
$(".op-menulink").click(function(){
		$("#overlay-menu").removeClass("op-menuvisible");
        $("#overlay-menu").addClass("op-menumasked");
});


Je suis sur qu'il y a une façon plus "propre" d'écrire ça. Merci à vous de m'éclairer

2 réponses

prosthetiks Messages postés 1189 Date d'inscription dimanche 7 octobre 2007 Statut Membre Dernière intervention 12 juin 2020 431
25 oct. 2014 à 18:42
Hello,

La seule chose que tu peux faire pour réduire ton code est de mettre à la suite addClass et removeClass.

Exemple:


$("#overlay-menu").removeClass("op-menuvisible").addClass("op-menuvisible");


A appliquer aux 3 cas, le reste est tout bon et respecte les bonnes pratiques
0
inspiring Messages postés 1813 Date d'inscription vendredi 28 janvier 2011 Statut Membre Dernière intervention 31 décembre 2018 265
25 oct. 2014 à 21:19
Je te remercie, je pensais qu'il y avais possibilités d'écrire ceci autrement ... Un doute éclairci!

Bonne soirée, je passe donc en résolu
0