Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add 0138-WebUI-DeviceAndChannelSettingsButtonInControl #1444

Conversation

jp112sdl
Copy link
Contributor

Description

Adds a button to take a shortcut to the device / channel settings in "Status and Control"
"Status and Control"->"Devices":
Bildschirmfoto 2021-10-10 um 18 47 11

"Status and Control"->"Rooms"/"Functions":
Bildschirmfoto 2021-10-10 um 18 48 35

Related Issue

https://homematic-forum.de/forum/viewtopic.php?f=65&t=41974&start=470#p681819

Types of changes

  • Docs change / refactoring / dependency upgrade
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Alternate Designs

Possible Drawbacks

Verification Process

Release Notes

Contributing checklist

  • My code follows the code style of this project.
  • I have read the CONTRIBUTING and LICENSE document.
  • I fully agree to distribute my changes under Apache 2.0 license.

@jens-maus
Copy link
Owner

@jp112sdl So, ich hab deinen Patch nun einmal etwas umgearbeitet. Hoffe es ist ok für dich das ich diese zusätzliche Spalte + Buttons wieder rückgängig gemacht habe und stattdessen einfach jetzt die Zellenfelder mit den Gerätenamen entsprechend verlinkt. Das würde ich jetzt so erst einmal mergen wollen um erstes Feedback einzusammeln. Können wir aber gerne weiterentwickeln wenn du noch andere Ideen hast. Aber ich würde ungern da wirklich eine extra / neue spalte reinbasteln wollen wie du dir das vorgestellt hattest weil sonst ist der Unterschied zwischen den beiden Bereichen ("Status und Bedienung" und "Einstellungen") einfach zu gering. Aber vllt. finden wir ja einen Kompromiss dazwischen.

@jens-maus jens-maus merged commit 7afb34b into jens-maus:master Oct 12, 2021
@jp112sdl
Copy link
Contributor Author

Hoffe es ist ok für dich das ich diese zusätzliche Spalte + Buttons wieder rückgängig gemacht habe und stattdessen einfach jetzt die Zellenfelder mit den Gerätenamen entsprechend verlinkt.

Hab es mir nun angeschaut... Leider ist überhaupt nicht ersichtlich, dass das Feld mit dem Kanalnamen einen Funktion hat, geschweige denn, welche.
Gut, der Mauszeiger verändert sich, aber mehr auch nicht. Kein Hint o.ä.
Wenn da nicht jemand intensiv das Changelog zu genau der nächsten Version studiert, wird diese Funktion wohl keine Beachtung finden.

Wo muss man in "Status und Bedienung"->"Geräte" hinklicken, um die gesamte Geräte-Einstellungsseite aufzurufen?
Ich hatte da ursprünglich den Button im Tabellenkopf eingebaut.

@jens-maus
Copy link
Owner

Hab es mir nun angeschaut... Leider ist überhaupt nicht ersichtlich, dass das Feld mit dem Kanalnamen einen Funktion hat, geschweige denn, welche.

Gut, der Mauszeiger verändert sich, aber mehr auch nicht. Kein Hint o.ä.

Wenn da nicht jemand intensiv das Changelog zu genau der nächsten Version studiert, wird diese Funktion wohl keine Beachtung finden.

Nun, das ist im Grunde dann genau das gleiche wie mit der neuen Systemvariableneditierfunktion die du ja hinzugefügt hat. Da ist auch nicht ersichtlich das man die editieren kann und nur die Mauszeigerveränderung verrät das. Empfinde ich ehrlich gesagt auch nicht als schlimm, sondern belässt die status ind bedienungoberflächen eben in der funktion hauptsächlich genau da wo sie jetzt sind. Sonst werden die IMHO den Einstellungsseiten zu gleich wenn da buttons zu sehen sind.

Man könnte natürlich noch darüber nachdenken die text oder hintergrundfarbe zu ändern damit das ein tick besser ersichtlich wird. Aber ne zusätzliche spalte halte ich für etwas over-the-top.

Wo muss man in "Status und Bedienung"->"Geräte" hinklicken, um die gesamte Geräte-Einstellungsseite aufzurufen?

Ich hatte da ursprünglich den Button im Tabellenkopf eingebaut.

Das wäre noch zu diskutieren wo man das geschickt aber vesteckt irgendwie unterbringt. Ein button im tabellenkopf war IMHO auch nicht gerade intuitiv. Die erste Frage wäre ob es so nicht genug ist wie es ist und wenn nicht dann könnte man ja drüber nachdenken irgendeinen weg in den kanaleinstellungen einzubauen womit man zu den Geräteeinstellungen wechseln kann.

@jp112sdl
Copy link
Contributor Author

Ich springe schon sehr häufig hin und her, zwischen Geräteeinstellung und Status und Bedienung.
Das ist der AskSin++ Geräteentwicklung geschuldet.

Mit den Systemvariablen hast du recht. Da könnte man auch noch ein "Klicken, um Wert zu ändern"-Hint on hover einbauen.

Ganz diplomatisch würde ich sagen - Ja ok, lass es so wie es jetzt implementiert ist.
Für einige persönliche Sonderlocken habe ich ja noch meine private Patch-Sammlung, wo ich mir dann halt den Einstellungsbutton im Tabellenheader mit einbauen kann.

@jens-maus
Copy link
Owner

Mit den Systemvariablen hast du recht. Da könnte man auch noch ein "Klicken, um Wert zu ändern"-Hint on hover einbauen.

Ich sehe das eher als "Powerfeatures" die ohnehin nur bei einem Admin Konto funktionieren und nen hover hint würde ich jetzt nicht unbedingt an der stelle wollen. Aber was man wie gesagt überlegen könnte wäre, ggf. den Text oder Hintergrund andersfarbig darzustellen (gerne auch ggf. erst wenn der mauszeiger über das zu klickende Objekt geht).

Ganz diplomatisch würde ich sagen - Ja ok, lass es so wie es jetzt implementiert ist. Für einige persönliche Sonderlocken habe ich ja noch meine private Patch-Sammlung, wo ich mir dann halt den Einstellungsbutton im Tabellenheader mit einbauen kann.

Ich will da wirklich nichts durchdrücken oder erzwingen, denke aber in der Tat das der Bereich "Status und Bedienung" weiterhin hauptsählich getrennt bleiben sollte. Ich verstehe aber den Wunsch (und unterstütze den voll) von dort aus in die Einstellungen schnell rüberwechseln zu können, deshalb find ich das prinzipiell super. Allerdings möchte ich eben versuchen, dass so minimalistisch und unauffällig wie möglich umzusetzen. Wir können aber gerne brainstorming dazu machen ob man den Hintergrund der Tabellenzelle irgendwie andersfarbig darstellt oder der Text kursiv wird oder ähnliches um das noch deutlicher zu machen das man auf die Tabellenzelle klicken kann. Und vllt. finden wir auch ne praktikable möglichkeit auch direkt in die Geräteeinstellungen vs. Kanaleinstellungen zu wechseln. Meine Vorstellung wäre ggf. mal zu schauen ob man von innerhalb der Kanaleinstellungen es schafft irgendwo ein wechsel zu den Gesamtgeräteeinstellungen hinzubekommen. Oder eben irgendwie doch noch ne Möglichkeit unterzubringen direkt von Status und Bedienung in die Geräteeinstellungen zu springen.

@jp112sdl
Copy link
Contributor Author

Wie wäre es, die "Geräteeinstellung"-Seite durch Klicken auf das Geräte-Piktogramm zu erreichen?
Das hat keine onClick-Funktion und zeigt lediglich bei OnHover ein großes Bild an.

Das würde die Selektion eines Geräts nicht stören und bräuchte keine Extras rechts in der Kanal-Liste.

@jens-maus
Copy link
Owner

Wie wäre es, die "Geräteeinstellung"-Seite durch Klicken auf das Geräte-Piktogramm zu erreichen? Das hat keine onClick-Funktion und zeigt lediglich bei OnHover ein großes Bild an.

Das hatte ich auch bereits überlegt, ja. Ist nur die Frage ob man da auch noch eine mouseover/mouseout funktionalität irgendwie umgesetzt bekommt (grauen hintergrund setzen) damit man es auch optisch besser wahrnehmen kann das man da draufklicken kann. Aber ich schau mir das nochmal an.

jens-maus added a commit that referenced this pull request Oct 18, 2021
click on the device image to directly go to the device settings. (cf.
#1444 (comment))
@jens-maus
Copy link
Owner

Ok, hab es nun entsprechend in deec419 umgesetzt. Nun sollte man auf das Icon klicken können und schwubs landet man in den Geräteeinstellungen. Danke für den Tipp!

@jp112sdl jp112sdl deleted the feature/0138-WebUI-DeviceAndChannelSettingsButtonInControl branch November 20, 2021 12:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants