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

Correction des couleurs vert illisibles sur Jeedom v4.x #28

Merged
merged 1 commit into from Jan 11, 2021
Merged

Correction des couleurs vert illisibles sur Jeedom v4.x #28

merged 1 commit into from Jan 11, 2021

Conversation

FabriceGo
Copy link
Contributor

Pour rendre compatible les couleurs ok/nok avec tous les thèmes de Jeedom v4+

Remplacement du vert : #00FF00
par : var(--al-success-color) !important
Replacement du rouge : #FF0000
par : var(--al-danger-color) !important

Pour rendre compatible les couleurs ok/nok avec tous les thèmes de Jeedom v4+

Remplacement du vert : #00FF00
par : var(--al-success-color) !important
Replacement du rouge : #FF0000
par : var(--al-danger-color) !important
@phifi92
Copy link
Owner

phifi92 commented Jan 10, 2021

Pour rendre compatible les couleurs ok/nok avec tous les thèmes de Jeedom v4+

Remplacement du vert : #00FF00
par : var(--al-success-color) !important
Replacement du rouge : #FF0000
par : var(--al-danger-color) !important

Bonjour,

Merci, j'ai poussé en bêta.

@phifi92 phifi92 closed this Jan 10, 2021
@phifi92
Copy link
Owner

phifi92 commented Jan 10, 2021

doublon

@phifi92 phifi92 reopened this Jan 10, 2021
@phifi92 phifi92 closed this Jan 10, 2021
@FabriceGo
Copy link
Contributor Author

Bonsoir.

Il n'y pas de doublon. l'un des pr est pour le dashboard et l'autre pour l'affichage mobile.
Il faut prendre en compte les deux.

Merci

Copy link
Contributor Author

@FabriceGo FabriceGo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bonjour,
Il y avait deux modifications à faire, l'une pour la partie mobile et une pour la partie Dashboard.
Il faut donc bien prendre aussi celle-ci en compte. Merci.

@phifi92
Copy link
Owner

phifi92 commented Jan 11, 2021

Bonjour,
oups, vu que j'ai regardé rapidement, j'y ai vu une ressemblance, et j'avais vraiment cru à un doublon.
Je vous fais encore confiance, et je valide. En ce moment, je manque de temps pour Jeedom

@phifi92 phifi92 reopened this Jan 11, 2021
@phifi92 phifi92 merged commit 2636256 into phifi92:master Jan 11, 2021
@FabriceGo
Copy link
Contributor Author

FabriceGo commented Jan 11, 2021

Je sais bien pour votre temps (Lunarok nous l'a dit), mais là, vous faites une modification que beaucoup attende, donc c'est super sympa pour nous.
=> On ne peut que vous remercier ;-)

Merci beaucoup.

Plus qu'a attendre le passage en version finale (cela a déjà été testé chez moi manuellement et au moins un autre utilisateur à l'instant sur le forum (il doit avoir la version bêta).

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

2 participants