Produits Entr'ouvert » Intégrations graphiques Publik

En fait ce n'est pas si simple qu'il n'y parait donc "Laissez mouaaaaaa regarder"

"Toot toot, hey, beep beep, toot toot, hey, beep beep"

Je serais pour ne pas faire ça et plutôt démarrer "donna" comme étant une copie du naoned existant + la correction de divers points qui ne peuvent pas l'être pour raison de compatibilité. Cette liste est à établir, pour moi un truc important est la situation de naoned concernant les boutons, parce que le bouton le plus commun (submit) inverse la logique (color/background) avec :

--background: #{$button-color};
--color: #{$button-background};

Frédéric Péters a écrit :

Je serais pour ne pas faire ça et plutôt démarrer "donna" comme étant une copie du naoned existant

Et en fait j'avais aussi raté qu'il y a "naoned" partout dans les noms de variable.
Donc pour moi il n'y a pas vraiment d'autres choix que de faire une vrai copie. Sinon on va se retrouver avec des --naoned dans le theme donna ou vice versa

Le jour ou l'on passe des instances d'un thème à l'autre il ne faudra pas oublier le settings "APPEARANCE_OPTIONS_PREFIX": "naoned",
Je n'ai pas regardé si cette variable est stockée ailleurs une fois la personnalisation commencée

+ la correction de divers points qui ne peuvent pas l'être pour raison de compatibilité. Cette liste est à établir,

Des tickets sont ouverts pour des demandes d'améliorations mais je ne vois pas de points qui posent des problèmes de compatibilité.
Ou peut-être rendre uniforme l'utilistion des tiret et underscore dans les noms de variables

pour moi un truc important est la situation de naoned concernant les boutons

J'ai posé un commentaire pour avancé sur ce sujet sur le ticket dédié : https://dev.entrouvert.org/issues/108555#note-4

Je n'ai pas regardé si cette variable est stockée ailleurs une fois la personnalisation commencée

Tu veux dire que ça risque de compliquer le passage de Naoned vers Donna en conservant les personnalisations existantes ?

Thomas Jund a écrit :

Tu veux dire que ça risque de compliquer le passage de Naoned vers Donna en conservant les personnalisations existantes ?

Oui mais en fait non. Ca ne semble pas stocké ailleurs. Ca arrive ici depuis les settings: https://git.entrouvert.org/entrouvert/publik-base-theme/src/commit/12f228123cd6098d98c89cb3562fdcce95ae4386/templates/theme.html#L50

Pourtant j'aime bien l'idée d'avoir plusieurs boutons avec des impacts graphiques différents en fonction de leurs roles sans avoir à définir 4 - 8 - 12 paramètres (parce qu'in fine on va demander la couleur de fond et de texte des survols, va pour les focus, etc.). Je reste sur l'idée qu'un thème personnalisable doit rester concis dans ces options et n'a pas vocation à repoduire un design system complet
Et cela n'empeche en rien de proposer de définir une 2e couleur et continuer à générer à partir de ces 2 couleurs un bouton standard et un bouton inversé, j'ai l'impression que c'est plus un problème de compéhension donc d'explication.

Je comprends qu'il y ai confusion en l'état, et en même temps ce serait dommage de nous retrouver avec un seul design de bouton. J'ai pas envie qu'on tranche dans la précipitation.

Ce serait en effet bien et ça amènerait également un problème de compatibilité d'avoir des options enregistrées sous des noms qui ne sont plus d'actualité.

Ok donc on ouvre un ticket dédié à ce problème pour être attentif

Sinon il y le ticket sur footer qui pourrait être traité, il y a plusieurs retours sur le fait que la couleur choisie via l'UI du footer n'est pas celle choisie dans l'UI, mais elle est assombrie (à la base pour forcée un contraste suffisant). et c'était bien une mauvaise idée.

J'ai pas envie qu'on tranche dans la précipitation.

Mon commentaire dans #108555#note-2 date d'il y a deux mois.

Ok donc on ouvre un ticket dédié à ce problème pour être attentif

Il s'agit pour moi clairement de quelque chose à traiter dans la PR associée à ce ticket.

De manière générale la question est de voir si la correction du problème aura des conséquences sur des sites qui seraient déjà configurés. Pour le cas ici, changement sur les noms de variable, les problèmes me semblent très clairs à voir c'est "la couleur primaire est réinitialisée à sa valeur par défaut" (cas "bascule vers l'utilisation de tirets") ou "la couleur du fond est réinitialisée à sa valeur par défaut" (cas "bascule vers l'utilisation d'underscores").

🤖 Modifications demandées sur la pull request :

• URL : https://git.entrouvert.org/entrouvert/publik-base-theme/pulls/1043

Ticket apparemment toujours bloqué. Je fais un point sur la situation mercredi.