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

Eingefärbte Datenpunkte in den Objekten bei Admin5 #1078

Closed
crunchip77 opened this issue Aug 9, 2021 · 16 comments
Closed

Eingefärbte Datenpunkte in den Objekten bei Admin5 #1078

crunchip77 opened this issue Aug 9, 2021 · 16 comments
Labels

Comments

@crunchip77
Copy link
Contributor

Mit neuer Admin Oberfläche(dunkles Design) wird der Pfad trashschedule.0.type so angezeigt, getestet mit Chrome und FF

image

@Feuer-sturm
Copy link
Collaborator

In trashschedule instance it is possible to set a color for every type. If a color is selected here the text in the objects will be changed as well

grafik

@crunchip77
Copy link
Contributor Author

"This color code can be used to colorize the individual appointments in VIS".
What does this have to do with the object view in the admin then?

@klein0r
Copy link
Contributor

klein0r commented Aug 11, 2021

@crunchip77 Thanks for the hint, I will update the description. But that's not the point in this issue 😄

The question is: Why is there a white or black background? Is that "automatic background color" a feature to improve contrast? Or is that an issue with common.color?

@GermanBluefox
Copy link
Contributor

It is automatic background for better contrast

@GermanBluefox
Copy link
Contributor

Ich sehe kein Fehler in diesem Issue. Works as designed.

@Feuer-sturm
Copy link
Collaborator

@GermanBluefox Diese Objekte sind auch farbig aber haben keinen weißen Hintergrund. Warum ist das Verhalten hier anders?
grafik

@GermanBluefox
Copy link
Contributor

GermanBluefox commented Aug 11, 2021

Weil das die spezielle Objekte sind: system, adapter werden extra eingefärbt.

@Feuer-sturm
Copy link
Collaborator

@GermanBluefox @Apollon77 Da das Verhalten "Works as designed." ist, kann das Ticket auch geschlossen werden.

@Diginix
Copy link

Diginix commented Feb 9, 2022

Das im Darkmode Objekte grünen Schriftfarbe auf weißem Hintergrund haben ist "as designed"?

Auch Einstellungen div. Instanzen sind bei mir vollständig unlesbar.
image

Der weiße Hintergrund kommt auf jeden Fall aus der admin.css
image

Imho dürfte kein background #FFF im dark mode erhalten.

@klein0r
Copy link
Contributor

klein0r commented Feb 9, 2022

@Diginix Wie hier schon diskutiert wurde, wird der Hintergrund weiß eingefärbt, um bei dunklen Themes einen besseren Kontrast zur Farbe zu bieten.

Verbesserungen werden per Pull-Request sicher gern angenommen

@Diginix
Copy link

Diginix commented Feb 9, 2022

Hm, weißer Adler auf weißem Grund = null Kontrast ;-)
Aber vllt liegt es auch an meinem Monitor. Imho ist eine größere weiße Fläche bei gewähltem dark theme einfach echt fieees.
Der Fehler bei Jarvis ist also die weiße Schriftfarbe in der adapter.css. Da jarvis2 keiner pflegt, muss ich es wohl lokal patchen.

@simatec
Copy link
Contributor

simatec commented Feb 9, 2022

Für Jarvis hatte ich bereits einen PR gemacht. Aber soweit ich weiß, ist der nur in Version 3 eingeflossen

@Diginix
Copy link

Diginix commented Feb 10, 2022

@klein0r Also der weiße Hintergrund bei der jarvis2 Instanz kommt auch aus der admin.css des jarvis Adapters.
Nachdem ich dort nun Zeile 14 auskommentiert habe, sieht es sauber aus. Das beißt sich mit deiner Aussage dass der weiße Hintergrund beim dark theme gewollt ist. Die weiße Schriftfarbe kommt nämlich vom Admin. Wie kann jemand weiße Schrift- und Hintergrundfarbe wollen?
Anyway: fixed for me
Musste schon einiges in jarvis2 fixen. Bringt aber nichts dafür noch ein Fork zu machen.

image

@klein0r
Copy link
Contributor

klein0r commented Feb 10, 2022

@Diginix Hier (und im verlinkten Issue) ging es um die Darstellung in der Objekt-Ansicht. Was die Adapter in ihren iFrames machen steht auf einem anderen Blatt

@Diginix
Copy link

Diginix commented Feb 10, 2022

Ok, dann war dein Kommentar auf mein Bild missverständlich. Soweit alles geklärt.

@DcFamas
Copy link

DcFamas commented Feb 19, 2022

Wäre es ggf. möglich dennoch das so anzupassen, dass der Text nur Grün erscheint und nicht der Hintergrund weiß eingefärbt wird? Das macht meiner Meinung nach den Dark Mode hinfällig,. Generell bricht es schon sehr das schöne Design.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

8 participants