Add Sticky-Filters

[psi-4ward]

Bitte noch mal testen ;)

[psi-4ward]

Ref #243

[jens-maus]

Bitte create-patches.sh aufrufen und das patch file auch einchecken.

[psi-4ward]

ah jo ;)

Ich hab die latest webui.js von eq-3/occu genommen - ist das richtig?

PS: Man sollte den Patch noch mal ausgiebig testen ;)

[jens-maus]

Nein, du musst die latest webui.js von meinen occu fork nehmen da der auf der 3.45.7 basiert. Auch bitte beachten das die dateien alle iso-8859-1 kodiert sind und nicht fälschlicherweise in utf8 umkodiert werden dürfen.

[psi-4ward]

auf ein neues ;)

[jens-maus]

So, ich kam endlich dazu deinen Patch mal zu testen. Folgende Anmerkungen hätte ich diesbzgl. noch:

1. Der "Filter zurücksetzen" Button unten setzt deinen Sticky Filter nicht zurück -> sollte er aber
2. Es kommt zu einem kleinen "Flackern" wenn man auf eine Seite zurückkehrt wo ein Filter aktiv ist. Das ist sicher dadurch begründet weil er natürlich alles einliest und danach dann die Ansicht wechselt. Gibt es irgendeine Möglichkeit das noch so zu ändern das das flackern nicht sichtbar wird? Ggf. das irgendwie im Hintergrund laden oder ähnliches?
3. Für folgende Seiten geht der Sticky Filter nicht: "Status und Bedienung -> Geräte", "Einstellungen -> Geräte-Firmware - Übersicht"

Ansonsten bin ich von deinem Patch wirklich sehr angetan und finde Ihn super! Wenn du also die paar Dinge noch anpassen könntest würde ich den zeitnah dann mergen und mit der nächsten RaspberryMatic Version dann ausrollen wollen!

[jens-maus]

@psi-4ward Da bald ein neuer RaspberryMatic release ansteht: Kommst du zeitnah ggf. dazu deinen PR entsprechend meiner Kommentare noch anzupassen? Das wäre super.

[psi-4ward]

Wird wahrscheinlich noch etwas dauern. Sry.

[psi-4ward]

Es kommt zu einem kleinen "Flackern" wenn man auf eine Seite zurückkehrt wo ein Filter aktiv ist. Das ist sicher dadurch begründet weil er natürlich alles einliest und danach dann die Ansicht wechselt. Gibt es irgendeine Möglichkeit das noch so zu ändern das das flackern nicht sichtbar wird? Ggf. das irgendwie im Hintergrund laden oder ähnliches?

Das liegt daran, dass ich keine Vernünftige Codestelle gefunden habe an der ich die Filter halbwegs generisch injecten kann bevor der "updateView" kommt. Ich fürchte, damit müssen wir erst mal leben.

Sticky Filter geht nicht: Startseite > Status und Bedienung > Geräte

Hmm das tut bei mir!

Sticky Filter geht nicht: Einstellungen -> Geräte-Firmware - Übersicht

Hei das ist ja wieder anders gemacht. Hab mal die ic_deviceFirmwareOverview.cgi eingefügt.

[jens-maus]

Wunderbar! Nun geht erst einmal alles soweit das ich denke das man das auf die Allgemeinheit loslassen kann. Ich danke dir recht herzlich @psi-4ward und hoffe natürlich auch das du vielleicht hier/da jetzt angefixt bist noch weitere solche WebUI patches zu generieren. Einfach mal in https://github.com/jens-maus/RaspberryMatic/projects/3 reinschauen und die eine oder andere Sache umsetzen, das wäre cool! ;-)

[mifricke]

Ich will mich auch mal an dieser Stelle für diesen Patch bedanken. Das Issue #243 dazu kam vor Uhrzeiten von mir, weil mich die Filter der WebUI immer schon aufgeregt habe. Also fettes Danke an @psi-4ward!

[psi-4ward]

So gerade mal selbst wieder einiges an HM "verstellt" und war direkt mal Stolz auf den Sticky-Filter :)
Ich hoffe es gibt keine unerwarteten Probleme, ich hab bislang nur im Chrome getestet.