Discussion :

[SpaceFrog]

Bonjour,

J'ai un tableau de li colorisé en even / odd, dans lequel j'ajoute des lignes avec un append de clone, et je supprime de lignes par remove.
après chacune de ses deux actions je recolorise avec cette focntion:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
function coloriseTable(){
	var line=0;
	$('#surface li:gt(1)').each(function(){
		++line;
		$(this).attr('class', (line%2)?'surfeven':'surfodd').find('span .line').html(line +'. ')
		})
}

Elle fonctionne très bien, mais j'aimerais le faire avec toggleClass, je ne pige pas le coup du switch

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
$('#surface li:gt(1)').filter(':even').toggleClass('surfeven',line%2==0).end()
	.fliter(':odd').toggleClass('surfodd',line%2==1)

[edit] oublié un filter ...
mais pas mieux
on dirait que le li conserve son odd et even de création ??

[vermine]

Salut,

Je ne comprends pas trop ton switch. Là il aura la même valeur pour toutes les lignes paires et la même valeur pour toutes les lignes impaires. Or il ne devrait pas avoir la même valeur. Les nouvelles lignes doivent prendre une coloration. Tandis que les anciennes lignes doivent soit garder la leur, soit en changer si elles ont été décalées dans le tableau.

Ensuite, tu fait un double toggle sur deux classes différentes. Je ne sais pas trop comment il digère l'affaire. C'est comme si ta ligne était bleue, puis tu lui dit "non, devient verte aussi". Quelle couleur prend-t-il ? Le vert ? Les deux ? Ou bien que ta ligne est verte et que tu lui dis "retire le bleu". Est-ce qu'il garde le vert ?

Dans ton premier code, tu changes une classe par une autre. Dans le second, tu ajoutes ou bien tu retires une classe, et puis tu ajoutes ou bien tu retires une autre classe. Tu peux alors te retrouver avec les deux classes en même temps.

[SpaceFrog]

le but est d'avoir une coloration alternée et un renumérotage (sans trou) même après suppression de ligne.

[vermine]

Oui, oui, j'ai bien compris.

Imaginons que tes lignes paires soient vertes et les impaires bleues.

Ligne 0
Ligne 1
Ligne 2
Ligne 3
Ligne 4
Ligne 5

Tu supprimes la 1, tu ajoutes la 6 :

Ligne 0
Ligne 2
Ligne 3
Ligne 4
Ligne 5
Ligne 6

Je n'ai pas trop vérifier (cerveau embrumé) mais en gros je crois que tu as ce comportement :

La ligne 0 doit rester verte mais avec ton switch tu lui dis de retirer le vert. Ko
Les lignes 2 et 4 doivent devenir bleues, tu leur ajoutes une classe qu'elles n'ont pas. Ok
Les lignes 3 et 5 doivent devenir vertes, tu leur ajoutes une classe qu'elle n'ont pas. Ok
La ligne 6 doit devenir bleue. Tu lui ajoutes une classe qu'elle n'a pas. Ok

Ligne 0
Ligne 2
Ligne 3
Ligne 4
Ligne 5
Ligne 6

Pour les lignes 2, 3, 4 et 5, tu as ajouté une nouvelle classe mais tu n'as pas retiré l'ancienne. Du coup je ne sais pas quelle couleur va primer.

[Bovino]

Oui, mais tu ne supprimes pas, si besoin est, la mauvaise classe

Il me semble que toggleClass() n'est pas le mieux adapté dans ce cas :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
$('#surface li:gt(1)').filter(':even').removeClass('surfodd').addClass('surfeven').end()
	.fliter(':odd').removeClass('surfeven').addClass('surfodd')

[SpaceFrog]

du coup je suis mieux de rester avec ma boucle each ...