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

[bug]: Editor Theme should not influence Navigation (as it is like a Widget - runtime related) #352

Closed
1 task done
eisbaer721 opened this issue Feb 3, 2024 · 7 comments · Fixed by #358 or #360
Closed
1 task done
Labels
bug 🐞 Something isn't working

Comments

@eisbaer721
Copy link

No existing issues.

  • There is no existing issue for my problem.

Describe the bug

Beim betätigen des Farbthemenauswahl ändert sich die Farbe im Editorbereich.
Die Runtimes-Ausgaben sind auch unterschiedlich bei geänderten Farben im Navigationsbereich.

To Reproduce

Screenshot 2024-02-03 143014

dunker Hintergrund:

Screenshot 2024-02-03 143435

heller Hintergrund:

Screenshot 2024-02-03 143607

Stellt man auch die Runtime im Editor auf aktiv sieht man die Icons die nicht aktiv sind (Seite1) nicht mehr.
Siehe:

Screenshot 2024-02-03 144234

Wieder da:

Screenshot 2024-02-03 144402

Expected behavior

Farben im Editor und in der Runtime sollten auf allen Geräten (PC, Tablet, gleich sein egal welche Farbthemenauswahl man im Menü eingestellt hat. Dieses sollte nur das Vis-2 Editormenü betreffen.

Am besten sollten sogar die Symbole und die Schrift im Navigationsmenü einstellbar sein.
(Größe, Farbe (aktiv abgewählt oder inaktiv),Schrift (aktiv angewählt oder inaktiv).

Screenshots & Logfiles

Testprojekt:

{ "___settings": { "folders": [], "openedViews": [ "default", "Seite 1" ], "ts": "1706965524735.uwt305" }, "default": { "name": "Default", "settings": { "style": { "background-color": "black" }, "theme": "", "navigation": true, "navigationBackground": "black" }, "widgets": {}, "activeWidgets": {} }, "Seite 1": { "name": "Seite 1", "parentId": null, "settings": { "style": { "background-color": "black" }, "navigation": true, "navigationBackground": "black" }, "widgets": {}, "activeWidgets": {} } }

Adapter version

2.9.26

js-controller version

5.0.19

Node version

18.19.0

Operating system

debian

Additional context

No response

@eisbaer721 eisbaer721 added the bug 🐞 Something isn't working label Feb 3, 2024
@foxriver76
Copy link
Collaborator

und die richtige Runtime (separates Fenster) hat auch das Problem dass z. B. der Menüeintrag nicht mehr sichtbar ist?

@eisbaer721
Copy link
Author

Ja. Das stimmt.

@foxriver76
Copy link
Collaborator

Hm also wäre Ziel hier generell - kein Einfluss von theme auf runtime wie du oben auch schon geschrieben hast

@eisbaer721
Copy link
Author

Das finde ich auch.

@foxriver76 foxriver76 changed the title Farbthemenauswahl ändert die Farben im Editormodus und unterschiedliche Runtime-Ausgaben [bug]: [bug]: Editor Theme should not influence Navigation (as it is like a Widget - runtime related) Feb 5, 2024
foxriver76 added a commit that referenced this issue Feb 6, 2024
@foxriver76
Copy link
Collaborator

die Navigation wird in der nächsten Version voll konfigurierbar sein was Farben betrifft, als Fallback habe ich das aktuelle Verhalten gelassen, falls jemand speziell damit arbeitet, da da wohl auch einiges an Arbeit rein ging von BF damals.

foxriver76 added a commit that referenced this issue Feb 6, 2024
@eisbaer721
Copy link
Author

Ich habe es gerade kurz getestet. Die Auswahl die dazu gekommen ist klappt wunderbar.
Nur leider fehlt noch der Kopfzeilentext. "Kopfzeilentext des Menüs"
Auf mein Tablet ist leider bei den Einstellungen diese noch schwarz.
Vielen Dank

Screenshot 2024-02-06 162120
Screenshot 2024-02-06 162211

@foxriver76
Copy link
Collaborator

Ah Mist da hatte ich wohl keinen ausgewählt

@foxriver76 foxriver76 reopened this Feb 6, 2024
foxriver76 added a commit that referenced this issue Feb 6, 2024
foxriver76 added a commit that referenced this issue Feb 6, 2024
…ader text color) (#360)

* apply default overflow correctly

- closes #31
- closes #22

* navigation style fixes (fix icon background and allow to customize header text color)

- closes #352

---------

Signed-off-by: Max Hauser <moritz.heusinger@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐞 Something isn't working
Projects
None yet
2 participants