Zefau / ioBroker.jarvis

jarvis - just another remarkable vis
https://forum.iobroker.net/topic/49776
Other
147 stars 38 forks source link

Dark Mode based on system-settings #2686

Closed Zefau closed 1 week ago

Zefau commented 1 week ago

Dark Mode automatically based on the system settings.

Zefau commented 1 week ago

grafik

Zefau commented 1 week ago

Dieser Feature Request wurde mit v3.2.0-beta.33 implementiert. Bitte bestätigen und auf Fehler prüfen. Wenn alles in Ordnung ist, gerne das Issue schließen.


This feature request has been implemented with v3.2.0-beta.33. Please verify and test the feature for any bugs. If everything works as expected, you may close the issue.

Bernd9000 commented 1 week ago

In der Firefox Konsole kommt bei beta.33: App-hGejAmgnwn window.darkMode is undefined TypeError: window.darkMode is undefined

Zefau commented 1 week ago

Jo, fix kommt

Bernd9000 commented 1 week ago

Kein Fehler mehr seit beta.37 Ich mußte einmal den Darkmode in jarvis aus und wieder einschalten damit es funktioniert.

mcuiobroker commented 1 week ago

[beta.37] DarkMode geht gar nicht mehr. Umschalten in ioBroker hilft auch nicht. Browser Edge Cache gelöscht -> image

Verbinden -> normaler Frontend (LightMode-Anzeige, obwohl "Dunkel" in Windows gewählt und themeDarkMode: true) image

Es kommen auch zig Meldungen Die hohe Anzahl kommt vom eigenen reconnect(), bei Fehler image

Es kommt zu ersten reconnect image Nach jedem Reconnect verdoppeln sich die eigenen "subscribeStates" auf scripts Wie fange ich das ab?

Bernd9000 commented 1 week ago

Stimmt, die Einstellung steht jetzt immer auf "off". Versucht man das zu ändert bleibt es bei "off" oder der Browser hängt.

Zefau commented 1 week ago

Verbinden -> normaler Frontend (LightMode-Anzeige, obwohl "Dunkel" in Windows gewählt und themeDarkMode: true)

themeDarkMode muss auf ´autosein, dann greift die Windows-Einstellung.truegibt es nicht mehr, ist nun entwederonoderoff`.

mcuiobroker commented 1 week ago

Dann wird es beim Update nicht automatisch angepasst? image Auch nach Löschen und Neustart der Instanz immer noch boolean Upload hat auch nicht geholfen.

Bernd9000 commented 1 week ago

beta.38 Hab nur eine Instanz, hier ist es zum Type "string" geworden.

Zefau commented 1 week ago

Also passt alles, außer, dass der type nicht aktualisiert ist? auto schaltet bei euch auch korrekt hin und her?

Bernd9000 commented 1 week ago

auto schaltet hier korrekt unter windows 11.

Nur in den jarvis Einstellungen wird es beim ersten Aufruf oder nach F5 immer als off angezeigt obwohl im Datenpunkt ein anderer Wert steht. Also ein Anzeigefehler bei beta.38

Nachtrag: Auf dem Android Handy funktioniert der auto Modus bei mir nicht Nachtrag2: Unter Firefox funktioniert es jetzt auch auf dem Handy

mcuiobroker commented 1 week ago

Fully Kiosk Browser - funktioniert auch ( Other Settings -> DarkMode -> Auto -> Fully Browser neu starten)

Edge (Handy) - funktioniert Chrome (Handy) - funktioniert Firefox (Handy) - funktioniert Android APP - nur Hintergrund wird angezeigt, keine Widgets, keine Menüeinträge, keine Tabs

Zefau commented 1 week ago

Dieser Feature Request wurde mit v3.2.0-beta.39 implementiert. Bitte bestätigen und auf Fehler prüfen. Wenn alles in Ordnung ist, gerne das Issue schließen.


This feature request has been implemented with v3.2.0-beta.39. Please verify and test the feature for any bugs. If everything works as expected, you may close the issue.

mcuiobroker commented 1 week ago

Soll das so sein? image

Oder sollte es ein Dropdown mit states (auto, off, on) sein? Wäre so für die User besser?

{
  "type": "state",
  "common": {
    "name": "Modify setting themeDarkMode",
    "role": "config",
    "type": "string",
    "states": {
      "auto": "Auto",
      "off": "Aus",
      "on": "An"
    },
    "write": true,
    "read": true
  },
  "native": {},
  "_id": "jarvis.0.settings.themeDarkMode",
  "acl": {
    "object": 1636,
    "state": 1636,
    "owner": "system.user.admin",
    "ownerGroup": "system.group.administrator"
  },
  "from": "system.adapter.admin.0",
  "user": "system.user.admin",
  "ts": 1719620078216
}
Zefau commented 1 week ago

Ne, passt so. Das Backend kennt die Werte gar nicht und schreibt nur stumpf weg.