Projet

Général

Profil

Development #43807

déplacer la var $primary-color dans general.scss

Ajouté par Thomas Jund (congés, retour le 29/04) il y a presque 4 ans. Mis à jour il y a 11 mois.

Statut:
En cours
Priorité:
Normal
Version cible:
-
Début:
09 juin 2020
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Non
Planning:
Non

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

0001-css-move-primary-color-to-general.scss-43807.patch (1002 octets) 0001-css-move-primary-color-to-general.scss-43807.patch Thomas Jund (congés, retour le 29/04), 09 juin 2020 11:25

Historique

#1

Mis à jour par Thomas Jund (congés, retour le 29/04) il y a presque 4 ans

#2

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.

#3

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.

#4

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 ?

#5

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 ?

#6

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.

#7

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.

#8

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
#9

Mis à jour par Frédéric Péters il y a 11 mois

  • Tâche parente #43311 supprimé

Formats disponibles : Atom PDF