Project

General

Profile

Development #43807

Autre #43311: Plan de refactorisation du block `user-info`

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

Added by Thomas Jund 8 months ago. Updated 4 months ago.

Status:
En cours
Priority:
Normal
Assignee:
Target version:
-
Start date:
09 Jun 2020
Due date:
% Done:

0%

Estimated time:
Patch proposed:
No
Planning:
No

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


Files

History

#1

Updated by Thomas Jund 8 months ago

#2

Updated by Frédéric Péters 8 months ago

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

Updated by Frédéric Péters 8 months ago

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

Updated by Thomas Jund 8 months ago

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

Updated by Frédéric Péters 7 months ago

Bon, plutôt que passer ça dans #44400; ce patch minimal + deux phrases dans la documentation scss ?

#6

Updated by Thomas Jund 7 months ago

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

Updated by Frédéric Péters 7 months ago

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

Updated by Frédéric Péters 4 months ago

  • Status changed from Solution proposée to En cours
  • Patch proposed changed from Yes to No

Also available in: Atom PDF