#127 Des filtres |ajouter_class |supprimer_class et |basculer_class

Merged
cerdic merged 3 commits from issue_3460 into master 6 days ago
cerdic commented 1 week ago
Owner

Fix #3460 donc

Fix #3460 donc
cerdic added 2 commits 1 week ago
cerdic commented 1 week ago
Poster
Owner

cf aussi spip/tests#1 pour les tests unitaires

cf aussi https://git.spip.net/spip/tests/pulls/1 pour les tests unitaires
erational commented 1 week ago
Poster
Collaborator

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 ?
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* ?
maieul commented 1 week ago
Poster

dans saisies on utilise le francais :).

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

dans saisies on utilise le francais :). Et dans SPIP on dit "inserer_attribut" et pas "inserer_attribute".
Poster
Owner

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

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
maieul commented 1 week ago
Poster

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

je ne faisais que constater l'existant. Pour ma part, pas d'avis sur la question.
b_b commented 1 week ago
Poster
Owner

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 ?

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 ?
JLuc commented 1 week ago
Poster

Ya aussi "inverser"

Ya aussi "inverser"
nicod_ commented 1 week ago
Poster
Owner

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 ?

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` ?
JLuc commented 1 week ago
Poster

Ou "changer" tout simplement

Ou "changer" tout simplement
Poster
Owner

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

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
maieul commented 1 week ago
Poster

alterner risque de confusion avec le filtre |alterner

alterner risque de confusion avec le filtre |alterner
erational commented 1 week ago
Poster
Collaborator

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'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.
cerdic added 1 commit 1 week ago
cerdic commented 1 week ago
Poster
Owner

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 :(

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 :(
b_b commented 1 week ago
Poster
Owner

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

> 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
Eric commented 1 week ago
Poster
Owner

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

Toggle se traduit le plus souvent par Basculer. On voit aussi le verbe Commuter.
maieul commented 1 week ago
Poster

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

je pense que basculer si c'est bien documenté, c'est le plus simple.
maieul commented 1 week ago
Poster

ou bien soyons explicite : ajouter_ou_supprimer

ou bien soyons explicite : ajouter_ou_supprimer
b_b commented 1 week ago
Poster
Owner

@maieul huhu, soyons précis ajouter_ou_supprimer_ou_ne_rien_faire ^^

/me => []

@maieul huhu, soyons précis ajouter_ou_supprimer_ou_ne_rien_faire ^^ /me => []
Poster
Owner

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

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
maieul commented 1 week ago
Poster

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.

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.
Eric commented 1 week ago
Poster
Owner

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.

> 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.
b_b commented 1 week ago
Poster
Owner

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

> 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
JLuc commented 1 week ago
Poster

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.

`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`.
erational commented 1 week ago
Poster
Collaborator

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

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
cerdic added 3 commits 6 days ago
cerdic merged commit be10ed9dd5 into master 6 days ago
cerdic commented 6 days ago
Poster
Owner

Renommé et intégré

Renommé et intégré
cerdic deleted branch issue_3460 6 days ago
erational commented 6 days ago
Poster
Collaborator

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' />

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' />`
The pull request has been merged as be10ed9dd5.
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No Assignees
8 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.