@pierre-gilles : Je viens de corriger tout les retours
1 « J'aime »
Merci pour les retours, c’est beaucoup mieux!
J’ai quelques feedbacks UX et un tech, rien de très grave 
On tient le bon bout, je pense que ça va pouvoir être mergé bientôt !
1 « J'aime »
Je viens de corriger tes derniers retours
Merci pour les retours
Je quelques review encore des petits détails : Add tag on scene by callemand · Pull Request #1900 · GladysAssistant/Gladys · GitHub
Merci pour la review
J’ai fait les retours sauf celui du bouton « désélectionner tous ». Je ne suis pas sûr que cela soit utile. A voir avec les utilisations de chacun. Qu’est ce que tu en penses ?
Tout depends le nombre de tags ajoutés.
Ok je veux bien voir en utilisation réelle, mais à mon avis la demande va venir vite ! Quand j’ai testé je trouvais ça pas très pratique de devoir faire un refresh pour clear la liste de tags, surtout qu’en PWA les utilisateurs ont pas forcément de bouton refresh.
Je te fais un retour demain sur la partie code, je suis en freelance aujourd’hui
1 « J'aime »
C’est tout bon pour moi @Lokkye, il y a juste Eslint qui crie sur ton dernier push 
Et j’ai juste une question dans les tests Cypress:
Après ces 2 petites trucs, c’est bon pour moi pour merge! Hâte de publier ça
Edit: Ah attend, petite régression sur la vue « Nouvelle scène »:
Avant:
Après:
C’était plus propre avant non ?
Oui c’est mieux avec le bouton en bas
Visuellement ça ne me choque pas, mais c’est beaucoup plus logique le bouton en bas.
Le flow c’est bien Nom > Icone > Valider.
@pierre-gilles @Hizo @lmilcent : Oui je suis d’accord avec vous. Je me demande encore comment j’ai fait pour inversé ces 2 composants alors que je ne devais pas toucher a ce code
@pierre-gilles : Je viens de faire les fix sur la PR et j’ai rebase depuis master pour que tu puisses facilement la merger
C’est bon pour moi @Lokkye, merci pour les correctifs 
C’est mergé sur master et ça partira dans la prochaine version de Gladys
J’ai juste quelques petits retours qui n’étaient pas bloquant pour la review que j’ai remarqué :
- Avec Preact, pas besoin de renommer
class=enclassName=. Classname est une spécificité de React qui ne s’applique pas à Preact, c’est même recommandé de rester surclass= (mais className marche quand même) - Pour masquer du contenu en écran de petite taille, tu peux utiliser des class Bootstrap toute faite, ça évite de re-écrire du CSS ( Display property · Bootstrap )
2 « J'aime »
Cette fonctionnalité est disponible dans Gladys Assistant 4.31 ! 
Je ferme ce sujet pour libérer les votes, n’hésitez pas à créer un autre sujet en cas de bugs sur cette fonctionnalité
2 « J'aime »