-
Notifications
You must be signed in to change notification settings - Fork 56
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
text-primary avec white background... #274
Comments
Bonjour, C’est volontaire et livré en 4.4.0, comme indiqué dans le changelog, dans la section Orange Brand Digital Guidelines → Color palette. C’est un choix assumé dans Boosted (qui se démarque de Bootstrap, sur ce point parmi d’autres) pour garantir un niveau de contraste suffisant, inciter à utiliser ces classes avec parcimonie, mais aussi respecter la charte graphique. Le tout est documenté dans la page dédiée aux utilitaires de couleurs. Les couleurs supports sont un couple couleur et arrière-plan nécessairement — à l’exception notable du orange qui n’est pas le même selon que son arrière-plan est blanc ou noir… Par ailleurs dans la version en cours (4.4.1), un certain nombre d’utilitaires sont supprimés, n’étant pas conformes à la charte. La documentation de la version en cours concernant les couleurs est ici. La palette disponible sera d’ailleurs documentée dans la page dédiée à la marque. Cependant ce retour est très pertinent (merci) car je m’aperçois que deux utilitaires m’ont échappé : Je boucle à nouveau avec @PigeardSylvain notamment pour ajuster tout ça, tout nouveau commentaire est évidemment bienvenu ! |
D'accord, ça parait cohérent. La raison pour laquelle ça m'a choqué, c'est que du coup quand il se trouve dans un élément qui à un fond différent en :hover, bah le fond du text primary reste blanc. Par exemple: Est-ce qu'il manquerait pas aussi la prise en compte de ce genre de cas? |
Le cas que tu décris est précisément celui qui sort de la parcimonie : les onglets sont chartés, leur aspect ne devrait pas être surchargés de la sorte — par exemple dans ton cas, parce que dans la version light des onglets, la couleur orange est utilisée au survol. Ainsi, dans cette version du composant on ne distinguerait plus l’élément « important » de l’élément survolé. On touche un point qui n’a plus grand chose à voir avec le CSS ni avec Boosted : si cet onglet est plus important que les autres, peut-être que :
Dans tous les cas, la charte graphique proscrit certains usages de certaines couleurs, mais avant tout propose directement des « UI Kits » que Boosted se doit de respecter (tout comme les projets l’utilisant) :
J’espère que ces ressources te seront utiles, il y en a d’autres à trouver sur design.orange.com ou brand.orange.com en cas de besoin :) |
Bonjour,
La classe text-primary ajoute un fond de couleur blanche alors que text-muted ne le fait pas par exemple.
Me semble que l'idée de la classe est de changer la couleur de font, pas celle de background
The text was updated successfully, but these errors were encountered: