@pierre-gilles: Ich habe gerade alle Rückmeldungen korrigiert
Danke für das Feedback, es ist viel besser!
Ich habe ein paar UX-Feedbacks und ein technisches, nichts Schlimmes 
Wir sind auf dem richtigen Weg, ich denke, es kann bald gemerged werden!
Ich habe gerade deine letzten Rückmeldungen korrigiert
Danke für das Feedback
Ich habe noch ein paar kleine Details zu überprüfen : Add tag on scene by callemand · Pull Request #1900 · GladysAssistant/Gladys · GitHub
Danke für das Feedback
Ich habe alle Rückmeldungen gemacht, außer die zum „Alle abwählen“-Button. Ich bin mir nicht sicher, ob das wirklich nützlich ist. Das kommt wohl auf die Nutzung jedes Einzelnen an. Was denkst du?
Das hängt von der Anzahl der hinzugefügten Tags ab.
Okay, ich würde das gerne in der echten Nutzung sehen, aber meiner Meinung nach wird die Anforderung schnell kommen! Als ich es getestet habe, fand ich es nicht sehr praktisch, ein Refresh durchführen zu müssen, um die Tag-Liste zu leeren, besonders weil Nutzer in PWAs nicht unbedingt einen Refresh-Button haben.
Ich gebe dir morgen Feedback zum Code-Teil, heute bin ich Freelancer
Alles klar für mich @Lokkye, nur ESLint schreit bei deinem letzten Push 
Und ich habe nur eine Frage zu den Cypress-Tests:
Nach diesen beiden Kleinigkeiten ist es für mich okay zum Mergen! Ich freue mich darauf, das zu veröffentlichen
Edit: Ah warte, kleine Regression bei der Ansicht « Neue Szene »:
Vorher:
Nachher:
War es nicht vorher sauberer?
Ja, das ist besser mit dem Button unten
Optisch stört mich das nicht, aber logischer ist der Button unten.
Der Ablauf ist gut: Name > Icon > Bestätigen.
@pierre-gilles @Hizo @lmilcent: Ja, ich bin mit euch einverstanden. Ich frage mich immer noch, wie ich es geschafft habe, diese beiden Komponenten zu vertauschen, obwohl ich den Code nicht anrühren sollte
@pierre-gilles: Ich habe gerade die Fixes im Pull Request gemacht und von master rebase, damit du ihn leicht mergen kannst.
Das ist in Ordnung für mich @Lokkye, danke für die Korrekturen 
Es ist in master gemerged und wird in der nächsten Version von Gladys veröffentlicht
Ich habe nur ein paar kleine Rückmeldungen, die nicht blockierend für die Überprüfung waren, die mir aufgefallen sind:
- Mit Preact muss
class=nicht inclassName=umbenannt werden. ClassName ist eine Spezifität von React, die nicht auf Preact angewendet wird, es wird sogar empfohlen, beiclass=zu bleiben (aber className funktioniert trotzdem) - Um Inhalte auf kleinen Bildschirmen auszublenden, kannst du fertige Bootstrap-Klassen verwenden, das spart dir das erneute Schreiben von CSS ( Display property · Bootstrap )
Diese Funktion ist in Gladys Assistant 4.31 verfügbar! 
Ich schließe dieses Thema, um die Abstimmungen freizugeben. Zögern Sie nicht, ein anderes Thema zu erstellen, falls Sie auf Fehler stoßen