C’est top comme ça, niveau couleurs et UI !
Merci @Lokkye pour tout le travail.
cce66
Septembre 12, 2023, 7:34
42
salut @Lokkye
Petite question cela marchera aussi pour la température ? (mais la le rouge s’imposera ! )
Lokkye
Septembre 14, 2023, 6:01
43
@pierre-gilles : My bad, j’avais oublié l’initialisation des variables. C’est fixé maintenant.
@cce66 : Une fois que l’on sera complétement d’accord pour l’humitidé, je m’occuperais de faire pour le temperature.
@Lokkye Très beau boulot, c’est très propre ça marche nickel
J’ai un mini retour mais c’est vraiment un petit détail :
Pour quelqu’un qui irait jusqu’à 100% (ce qui a peu de sens, mais bon), le % dépasse de la barre
Mais sinon, je trouve ça très intuitif et très clair
1 « J'aime »
Moi en fait…
Et je n’ai pas de sauna
Pour le terrarium tropical, entre 85% et 100% sont les valeurs idéales…
1 « J'aime »
C’est clair que c’est plus joli avec cette barre que la première proposition en indiquant juste des nombres et des couleurs. La c’est très clair.
1 « J'aime »
Lokkye
Septembre 15, 2023, 1:26
47
@pierre-gilles : Je viens de faire la correction pour le « 100% »
2 « J'aime »
@Lokkye Merci du retour, ça marche mieux !
Petit problème de z-index par contre:
Lokkye
Septembre 19, 2023, 6:42
49
@pierre-gilles : C’est corrigé
Excellent @Lokkye ! C’est bon pour moi, testé et validé
Je viens de merger la PR, ça partira dans la prochaine version de Gladys.
2 « J'aime »
Lokkye
Septembre 20, 2023, 8:09
52
Je vais donc m’attaquer à la partie température si cela convient à tout le monde
5 « J'aime »
Lokkye
Septembre 21, 2023, 12:20
53
Voila le résultat pour la temperature :
Avec la petite PR :
GladysAssistant:master
← callemand:min_max_temp
opened 12:19PM - 21 Sep 23 UTC
### Pull Request check-list
To ensure your Pull Request can be accepted as fa… st as possible, make sure to review and check all of these items:
- [x] If your changes affects code, did your write the tests?
- [x] Are tests passing? (`npm test` on both front/server)
- [x] Is the linter passing? (`npm run eslint` on both front/server)
- [x] Did you run prettier? (`npm run prettier` on both front/server)
- [x] If you are adding a new features/services, did you run integration comparator? (`npm run compare-translations` on front)
- [x] Did you test this pull request in real life? With real devices? If this development is a big feature or a new service, we recommend that you provide a Docker image to the community ([french forum](https://community.gladysassistant.com/)/[english forum](https://en-community.gladysassistant.com/)) for testing before merging.
NOTE: these things are not required to open a PR and can be done afterwards / while the PR is open.
### Description of change
Add a threshold for the room temperature widget
<img width="538" alt="Screenshot 2023-09-21 at 14 15 22" src="https://github.com/GladysAssistant/Gladys/assets/11317212/eb6d616a-72ec-48b1-93d1-4dc2bda6014c">
<img width="543" alt="Screenshot 2023-09-21 at 14 15 11" src="https://github.com/GladysAssistant/Gladys/assets/11317212/cf3a38ad-e184-4a06-b338-ae7504d1afb4">
<img width="546" alt="Screenshot 2023-09-21 at 14 14 49" src="https://github.com/GladysAssistant/Gladys/assets/11317212/d140adf1-1cf6-4742-a0c3-b56ead8d5736">
<img width="539" alt="Screenshot 2023-09-21 at 14 14 26" src="https://github.com/GladysAssistant/Gladys/assets/11317212/bd6d124f-3ce6-4055-8696-6d2b3905cda0">
<img width="675" alt="Screenshot 2023-09-21 at 14 14 18" src="https://github.com/GladysAssistant/Gladys/assets/11317212/6cd8489c-aa63-4f9b-bfb7-8f6d34287a8b">
7 « J'aime »
Super
Est-ce que tu peux mettre l’icône en jaune quand il n’y a pas de valeur récente ? Classe bg-warning
pour être cohérent avec la dernière évolution sur le tableau de bord
1 « J'aime »
Merci et merci pour tout le travail sur cette partie
guim31
Septembre 21, 2023, 2:26
58
Tu as des Bitcoins dans ton salon ???
1 « J'aime »
Il faut bien les stocker quelque part (dans Gladys)
1 « J'aime »
Super propre @Lokkye
L’échelle est peut-être un peu haute dans le positif, et pas assez basse dans le négatif (0 à 100) :
Du coup, les valeurs par défaut sont collées, alors qu’en soit dans notre salon la température ne va pas non plus jusqu’à 90°C
Je penserais qu’on pourrait prendre plutôt une échelle -20°C → 50°C (je sais que les températures peuvent aller avant ou après, mais personne ne mettra une température de confort à -40, ou à 60)
Je pense aussi qu’il faudrait gérer les 2 unités: celsius ou fahrenheit, et adapter les niveaux par défaut en conséquence ? Tu peux garder une seule constante de format par défaut, mais faire une conversion si l’utilisateur est en fahrenheit?