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

[Enhancement]: Thème sombre/clair #490

Closed
fraecar opened this issue Dec 16, 2024 · 8 comments · Fixed by #542
Closed

[Enhancement]: Thème sombre/clair #490

fraecar opened this issue Dec 16, 2024 · 8 comments · Fixed by #542
Assignees
Labels
✨ enhancement New feature or request

Comments

@fraecar
Copy link

fraecar commented Dec 16, 2024

Description du problème

Salut, je propose de pouvoir ajouter dans les paramètre de l'application, un option qui nous permet de choisir entre thème sombre/clair

Description de l'amélioration

cette option dans les paramètre permet de choisir quel affichage on veut, en effet, j'utilise principalement le mode sombre dans la plupart des applications, hors sur certaine application je préfère le mode claire pour améliorer la lisibilité des textes
Est ce que sa serait possible de rajouter une option qui permet de choisir le thème (clair/sombre) par un select menu avec les options :
-mode sombre
-mode clair
-thème par défaut du système

merci par avance

Contexte supplémentaire

No response

@yannouuuu
Copy link
Member

Bonne idée!

@yannouuuu yannouuuu added the ✨ enhancement New feature or request label Dec 23, 2024
@raphckrman
Copy link
Contributor

Bonne idée!

C'est déjà le cas depuis peu, quelqu'un a déjà fait une PRimage

@Gabriel29306
Copy link
Contributor

Ah oui, ça date d'il y a moins d'une semaine, je ne l'ai pas dans mon build

@yannouuuu
Copy link
Member

Elle a été merge ???

@Gabriel29306
Copy link
Contributor

Oui, mais petit problème, on a un message Redémarrer l'application qui est indiqué lors du changement de mode.
Mais avec la façon dont est dit Redémarrer l'application, j'ai bêtement essayé de cliquer dessus alors que ce n'est pas cliquable

@Gabriel29306
Copy link
Contributor

Gabriel29306 commented Dec 23, 2024

De plus le code de la page aurait pû, et aurait dû, être plus optimisé avec une liste pour les check box.
Créer un component juste pour une page n'est pas utile

@Gabriel29306
Copy link
Contributor

@godetremy j'ai testé ta fonctionnalité, j'ai quelques soucis sur ta page.
En mode clair, le texte de l'infobulle (on dit ça comme ça ?) reste en blanc, donc illisible
Et je ne sais si c'est facilement faisable, mais ce serait de faire comme avec le toggle system où l'app change instantanément de thème de couleurs (peut être avec une propriété sur la personnalisation du compte ?)

@Kgeek33
Copy link
Contributor

Kgeek33 commented Dec 25, 2024

déjà merged dans une pr
@Gabriel29306 j'te recommande de faire une nouvelle issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants