Development #43807
déplacer la var $primary-color dans general.scss
0%
Description
La variable sass $primary-color est déclarée dans user-infos.scss. Elle ne peut donc pas être utilisée dans les styles précédent sa déclaration.
Je propose de la déclarer directement dans general.scss
Fichiers
Historique
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a presque 4 ans
- Fichier 0001-css-move-primary-color-to-general.scss-43807.patch 0001-css-move-primary-color-to-general.scss-43807.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Frédéric Péters il y a presque 4 ans
Ok mais ça en vient à officialiser cette variable et son sens (très bien), ça veut dire aussi l'ajouter/documenter dans help/fr/misc-scss.page.
À côté, on pourrait aussi avoir un ticket pour prendre un peu de temps pour réfléchir en plus large la gestion des couleurs ? On a dans certianes intégrations également un $secondary-color, on généraliserait ? Ou avoir un nom plus sémantique, genre $accent-color ? Et il y a aussi eu des idées à un moment de gestion des palettes de couleur, on pourrait avoir ça aussi.
Mais je parle bien pour tout ça d'un ticket différent, qu'on garde celui-ci pour le petit bout primary-color.
Mis à jour par Frédéric Péters il y a presque 4 ans
Aussi, je ne trouve pas #910956 terrible comme couleur par défaut si on officialise ça, et ça me fait penser aussi qu'à instituer un système de couleurs, il y a aussi $button-background qui aujourd'hui à une couleur tout à fait arbitraire.
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a presque 4 ans
Ok mais ça en vient à officialiser cette variable et son sens (très bien), ça veut dire aussi l'ajouter/documenter dans help/fr/misc-scss.page.
Pour le moment, les modules/composants utilisant $primary-color :
- _dashboard (je n'ai pas encore exploré ce module)
- _user-info pour la couleur de la bordure ??
- wcs/_bulk pour la couleur centrale des disques/pastilles du composant "evolutions"
- wcs/_steps pour la couleur de l'étape active
Voilà pourquoi il me paraissait incohérent d'avoir cette variable cantonnée au fichier user-infos.
Mais je ne sais pas si le design system actuel est prêt pour officialiser cette variable dans la doc. Annoncer la variable et lister les éléments si dessus me parait sans grand intérêt. Il faudrait peut-être généraliser un peu plus son utilisation. Du moins valider son intérêt.
côté, on pourrait aussi avoir un ticket pour prendre un peu de temps pour réfléchir en plus large la gestion des couleurs ?
Oui, ouvrir un chantier sur la gestion des couleurs d'un thème serait intéressant, j'y ai déjà pensé, et rangé l'idée dans un item de todo liste :).
Et en effet, il faut avoir une réflexion globale sur la définition d'un nuancier pour un thème.
La possibilité d'appliquer rapidement les couleurs de ce nuancier à un thème (et là $primary-color ou d'autres variables peuvent être utiles).
La possibilité d'ajuster facilement chaque composant ou bloc individuellement.
Permettre de créer facilement des variantes comme $primary-color-light, $primary-color-dark ?
Mis à jour par Frédéric Péters il y a presque 4 ans
Bon, plutôt que passer ça dans #44400; ce patch minimal + deux phrases dans la documentation scss ?
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a presque 4 ans
Bon, plutôt que passer ça dans #44400;
c'était une erreur, corrigée.
Je pensais ce patch + patch pour passer quelques couleurs (boutons, liens) en $primary-color + documentation.
Mis à jour par Frédéric Péters il y a presque 4 ans
Ok, je proposais de limiter ce ticket dans l'idée que l'autre était bloqué par celui-ci; si ce n'est pas le cas pas de problème pour prendre le temps nécessaire ici pour intégrer différentes couleurs.
Mis à jour par Frédéric Péters il y a plus de 3 ans
- Statut changé de Solution proposée à En cours
- Patch proposed changé de Oui à Non