#127 Des filtres |ajouter_class |supprimer_class et |basculer_class

cf aussi spip/tests#1 pour les tests unitaires

Top ! Je teste en détail lundi

J'ai juste un doute sur le nommage selon la nomenklatura SPIP

• |ajouter_class |supprimer_class et |basculer_class ?
• ou |ajouter_classe |supprimer_classe et |basculer_classe ?

dans saisies on utilise le francais :).

Et dans SPIP on dit "inserer_attribut" et pas "inserer_attribute".

pas de mélange fr-en par défaut mais pour le coup là c'est légitime : "attribute" ce n'est pas le nom d'un attribut, alors que là "class" c'est pas du tout pour dire le mot "classe" en anglais, c'est spécifiquement le nom d'un élément HTML précis que l'on modifie avec ces fonctions, tout comme il y a "balise_img"

donc il me semble que ça doit bien rester comme ça

je ne faisais que constater l'existant. Pour ma part, pas d'avis sur la question.

Super, par contre j'ai du lire le code pour comprendre ce que fait basculer_class ^^

Ça semble être un toggle, donc pourquoi pas nommer le truc intervertir plutôt que basculer, ha non, mieux : commuter ! Vos avis ?

Ya aussi "inverser"

Oui, class c'est très bien comme ça, c'est le nom de l'attribut.
Par contre, basculer n'est pas évident, c'est vrai.

alterner ?

Ou "changer" tout simplement

inverser/alterner c'est pas mal, c'est plutôt ça l'idée de toggle en informatique : inverser une valeur binaire
changer c'est trop vague, toggle c'est vraiment inverser/alterner deux valeurs

et si on va par là, en anglais c'est remove class, et non pas delete class, donc c'est plutôt retirer que supprimer

alterner risque de confusion avec le filtre |alterner

J'aime bien bascule 🤹

• court et simple à écrire même pour un dyslexique
• simple à retenir (siège à bascule)
• on a bien l'idée d'alternance entre deux états.

J'ai corrigé les 2 remontées de test de @erational : espace surnuméraire en debut et attribut class vide.

Pour le nommage je vous laisse trancher, pas d'avis : basculer m'est venu parce qu'il m'a semblé que ce serait simple à se souvenir une fois découvert, mais c'est vrai qu'au départ c'est pas intuitif - et j'ai pas l'impression qu'il y ait un équivalent français usuel pour le toogle :(

Pour le nommage je vous laisse trancher, pas d'avis : basculer m'est venu parce qu'il m'a semblé que ce serait simple à se souvenir une fois découvert, mais c'est vrai qu'au départ c'est pas intuitif - et j'ai pas l'impression qu'il y ait un équivalent français usuel pour le toogle :(

Idem, c'est pour ça que je proposais commuter (comme un commutateur), mais bon, si basculer reste on s'y fera très bien avec le temps :p

Toggle se traduit le plus souvent par Basculer. On voit aussi le verbe Commuter.

je pense que basculer si c'est bien documenté, c'est le plus simple.

ou bien soyons explicite : ajouter_ou_supprimer

@maieul huhu, soyons précis ajouter_ou_supprimer_ou_ne_rien_faire ^^

/me => []

une fois documenté, c'est sûr qu'on s'y fera, comme on s'est fait à toutes les bizareries de SPIP depuis des années :p

dans un texte classique c'est peut-être basculer, mais spécifiquement dans le cadre d'une fonctionnalité informatique, j'ai jamais vu le mot "basculer" utilisé de toute ma vie dans une doc française :)

wiktionary anglais dit "to alternate between two positions"

wiktionary français dit "alterner entre deux positions" en premier aussi, en deuxième seulement "basculer entre plusieurs états" (or nous c'est bien entre deux positions seulement)

donc dans les deux cas, plutôt "alterner_class" comme disait nico

Je maintiens qu'il y a un risque de confusion avec |alterner, alors que sémantiquement les 2 fonctions ne font pas du tout la même chose.

donc dans les deux cas, plutôt "alterner_class" comme disait nico

Alterner à mon avis n'est pas optimal. On alterne entre un truc et un autre comme par exemple la couleur des lignes paires et impaires. Donc bof.

Là si j'ai bien compris on ajoute ou supprime une classe. Donc ça ressemble plus au fait d'allumer ou d'éteindre, une fonction de commutation. Peut-être que "commuter" serait plus adapté comme le proposait b_b.

Là si j'ai bien compris on ajoute ou supprime une classe. Donc ça ressemble plus au fait d'allumer ou d'éteindre, une fonction de commutation. Peut-être que "commuter" serait plus adapté comme le proposait b_b.

Oui, c'est bien pour ça que je proposais ce terme, cf https://www.linguee.com/english-french/search?source=auto&query=toggle

alterner confuse trop avec le filtre.
inverser confuse un peu avec les mathématiques aussi.
basculer est plus flou.
commuter est peut-être bien notre meilleur toggle.

Comme l'opération est un peu spéciale, il me semble que "commuter" soit le plus approprié.
Quand le pull sera validé, je m'occupe d'écrire la documentation des 3 filtres sur spip.net

Renommé et intégré

Génial 🥇

L'ébauche de documentation
https://www.spip.net/ecrire/?exec=article&id_article=6623
https://www.spip.net/ecrire/?exec=article&id_article=6624
https://www.spip.net/ecrire/?exec=article&id_article=6625

A noter:

[(#CHEMIN{img/logo.svg}|balise_img|ajouter_class{logo-site})]

produit toujours un blanc en trop
<img src='squelettes/img/logo.svg?1579179279' alt='' width='100' height='100' class=' logo-site' />