Skip to content
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

fix: custom theme #469

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from
Open

Conversation

Naomi-Fischer
Copy link

@Naomi-Fischer Naomi-Fischer commented May 24, 2023

Objectif : permettre une personnalisation simple et rapide du theme de l'application

Les couleurs "en dur" ont été remplacées par des variables afin de n'avoir que les valeurs de ces dernières à modifier dans le fichier atlas.css

image

@camillemonchicourt
Copy link
Member

OK intéressant. Merci.
Les utilisateurs ne sont pas censés modifier le fichier atlas.css qui est écrasé à chaque mise à jour, mais uniquement surcoucher le CSS dans le fichier custom.css (https://github.com/PnX-SI/GeoNature-atlas/blob/master/atlas/static/custom/custom.css.sample).
Cela fonctionne avec ces variables ?

@Naomi-Fischer
Copy link
Author

Ah d'accord, my bad !
Effectivement les variables peuvent aussi etre placées dans le custom.css sans probleme.

@joelclems
Copy link
Contributor

Est ce que cette PR est toujours d'actualité ?

@camillemonchicourt
Copy link
Member

Oui à tester et vérifier qu'il n'y a pas de régression pour les instances et customisations existantes.
Aussi à documenter, au moins dans un changelog et dans la doc de customisation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants