Verbesserungen in der zigbee2mqtt UI-Integration

Entwicklung
1

Hallo zusammen,

ich hätte gerne gewusst, ob es sinnvoll und machbar wäre, die Anzeige der Geräte in der zigbee2mqtt-Integration zu verbessern.

Mehrere Punkte:

  • GERÄTE-Teil: Das Suchfeld ist umständlich zu bedienen, ich habe kaum Zeit, etwas einzugeben, da es bereits mit dem Filtern der Geräte beginnt und dazu neigt, einige der von mir eingegebenen Buchstaben zu überspringen… vielleicht haben andere das auch schon bemerkt. Es scheint, als bräuchte es etwas mehr „Latenz“ zwischen den eingegebenen Buchstaben und dem Filtern.

  • ENTDECKUNG-Teil: Wäre es möglich, die bereits zu Gladys hinzugefügten Geräte standardmäßig auszublenden? Im schlimmsten Fall mit einem Button, der anbietet, sie anzuzeigen (ich glaube, das gibt es irgendwo… in der LAN-Integration, oder?) Denn bei mir, besonders auf dem Handy, ist die Liste so lang, dass es eine Mühe ist, so viel zu scrollen.

  • Beim Thema Scrollen, wenn man viele Geräte hat, wäre es möglich, einen kleinen Pfeil hinzuzufügen, um mit einem Klick nach oben auf die Seite zu gelangen?

Das sind meine Vorschläge, jetzt liegt es an euch, liebe Entwickler, mir zu sagen, ob das für euch umsetzbar ist! :wink:

2

Danke für das Feedback! Zum ersten Punkt, das ist definitiv etwas, das eine GitHub-Issue sein könnte, das ist nicht normal.

Zum zweiten Punkt, ich würde gerne noch mehr Feedback haben, um zu sehen, ob diese Meinung von allen geteilt wird :slight_smile:

Zum dritten Punkt, warum nicht!

Im Moment bin ich allein mit Zigbee, seit Alex Papa ist, also werde ich das wahrscheinlich übernehmen.

3

Alles klar, super! Dann warten wir mal auf andere Meinungen! :+1:

4

Vollkommen einverstanden mit den von @guim31 angesprochenen Punkten :slight_smile:

5

Ok, ich denke, diese Punkte sollten irgendwo gemeldet werden, sonst verschwinden sie in der Versenkung :slight_smile:

Meine Meinung:

  • Zum Punkt 1, eine GitHub-Issue erstellen (auf Englisch, falls möglich mit Video zur Reproduktion des Bugs)
  • Zum Punkt 2, ebenfalls eine GitHub-Issue. Ein kurzer Text reicht aus
  • Zum Punkt 3, ebenfalls eine GitHub-Issue.

Füge in jede Issue einen Link zu diesem Thema für das Archiv hinzu.

Ich garantieren nicht für eine kurzfristige Bearbeitung, ich bin momentan allein bei der Integration von Zigbee (auf der Suche nach einem Maintainer, falls jemand vorbeikommt…) und es gibt viele Anfragen!

6

Alles klar, ich kümmere mich darum, sobald ich etwas Zeit habe.

7

Das ist erledigt :+1: Ich hoffe, ich habe es richtig gemacht !!

8

Danke! Ich würde mich freuen, wenn du in jedem Issue einen Link zu diesem Thema einfügst.

Bei Videos versuche bitte, sie im mp4-Format hochzuladen, damit man sie direkt auf Github ansehen kann und sie nicht herunterladen muss… :slight_smile:

9

Alles erledigt! :wink:

10

Hallo, ich habe zwischen zwei Nickerchen eine Entwicklung gemacht, um bereits bestehende z2m-Geräte zu filtern (cc @VonOx).

Das Test-Image (wird in 1 Stunde fertig sein) ist atrovato:z2m-filter-devices.

Danke für euer Feedback (ich gebe zu, dass ich die Entwicklung blind gemacht habe, inspiriert von der lan-manager-Funktion).

11

Ich habe einen Fehler beim Laden der Seite „Zigbee entdeckt“

actions.js:42 Uncaught TypeError: Cannot read properties of undefined (reading 'filter')
    at Object.setDiscoveredDevices (actions.js:42:49)
    at Object.setDiscoveredDevices (index.js:60:22)
    at c.componentWillMount (index.js:22:16)

Ich glaube, dass incomingDevices zu diesem Zeitpunkt leer ist

12

Danke @cicoub13 für dein Feedback!
Ich erstelle gerade ein neues Image mit dem Fix für incomingDevices = undefined, das dauert etwa 6 Minuten.

13

Die Seite wird angezeigt.
Auf dem Handy ist es etwas eng.

Screenshot_20231003-220149
Screenshot_20231003-2201491440×3120 301 KB

Der Button funktioniert nicht. Ich teste morgen am Computer und gebe dir mehr Details.

14 
props.js:153 Uncaught TypeError: this.l[(e.type + false)] is not a function
    at HTMLInputElement.x (props.js:153:20)

Ich glaube, du hast vergessen, setDiscoveredDevices in den Props in Zeile 50 in index.js hinzuzufügen.

15

Tatsächlich hatte ich auf dem onClick die Toggle-Funktion nicht gesetzt, das ist jetzt korrigiert, ebenso die Anzeige.

Das korrigierte Bild ist auf Docker verfügbar.

Nochmals vielen Dank @cicoub13 <3

16

Das läuft perfekt :+1:

17

Super, ich habe den PR zur Überprüfung eingereicht.

18

@pierre-gilles der PR für die Hinzufügung des Filters für bereits hinzugefügte Geräte:

Die folgenden 2 Issues sind behoben:

19

Danke für das PR!

Das sieht gut aus, ich habe nur eine kleine Rückmeldung:

20

Danke für die Fixes @AlexTrovato! Vergiss nicht, mir beim nächsten Mal eine Nachricht im Forum zu schreiben, wenn es behoben ist. Auf Github habe ich tausende Benachrichtigungen, das ist unbrauchbar :joy:

Ich habe gemerged, das wird in der nächsten Version von Gladys veröffentlicht!