ulaval / modul

Library of VueJS components.
Apache License 2.0
10 stars 4 forks source link

[scss] - Ajout des vartiable css pour les styles globaux (.m-u--app-body) #406

Closed Atiomi closed 3 years ago

Atiomi commented 3 years ago

Description

J'ai ajouté des custom properties (variables css) pour les quelques styles de base qui leakaient dans les css non scoper de brio

Le problème est causé par la classe .m-u--app-body qui augmente la spécificité de ces styles et qui nous obligeait à surenchérir inutilement de notre côté.

Il ne devrait pas y avoir aucun changement, les valeurs de base étant les mêmes. Les var css nous permettent simplement de redéfinir plus facilement les valeurs de notre côté.

P.s. Perso je pense que ces styles ne devraient même pas être dans modUL, mais bon pour respecter les besoins des projets moins récents je propose un compromis. ;)

Types de changements

Comment cela peut-il être testé?

Inclure cette section dans les release notes

Liens internes

https://jira.dti.ulaval.ca/browse/ENA2-12436

matur367 commented 3 years ago

10/10 :)