search

OPUS4 / application

OPUS 4 application.
Other
15 stars 21 forks source link

Dokumentation der CSS Variablen #1104

Open j3nsch opened 1 year ago

j3nsch commented 1 year ago

Könnte man im Default-Thema-CSS (theme-vars--default.css) Kommentare hinter die Einträge schreiben, die sagen wofür diese Variable verwendet wird, vielleicht ein paar Beispiele auflisten, z.B. Frontdoor-Document-Titles?

j3nsch commented 1 year ago

Was kann über diese Dateien nicht gesteuert werden, der Hintergrund? Wir hatten ja schon das Problem, das weißer Text auf weißem Hintergrund angezeigt wurde. Wenn die Farben aber in einer Datei festgelegt werden, weiß ich nicht wie solche "Konflikte" zustande kommen können.

vgerlach commented 1 year ago

Das Problem an der Stelle ist/war, dass wir für die Startseite und auch die anderen Unterseiten (Suche, Browsing etc.) die Hintergrundfarbe des Headers und dementsprechend die Schriftfarbe der Überschrift (h2) geändert haben. Es gibt aber Seiten, wo die Header-Farbe gar nicht angezeigt, aber weiterhin die Überschrift h2 genutzt wird - wir hatten also z.B. einen dunkelblauen Header und haben für die Lesbarkeit die Schriftfarbe für h2 auf weiß gesetzt, was dann auf manchen Unterseiten ohne Header zu dem beschriebenen Phänomen geführt hat.

j3nsch commented 1 year ago

Das sollte also im Design der Seiten vereinheitlicht oder es sollte zwei Variablen für die Schriftfarbe geben, für Seiten mit und ohne Header. Lässt sich das direkt über CSS lösen, oder must das HTML geändert werden? Sollte ein anderes Tag (h2) verwendet werden oder sollte das Tag mit einer CSS-Klasse markiert werden. Der "Header" sollte ja eigentlich im HTML erkennbar sein, so dass die Werte für H2 in abhängig davon angepasst werden können.

vgerlach commented 1 year ago

Also ich bin ja kein ausgewiesener CSS-Experte. Was wir gemacht haben, ist für folgendes eine andere Farbe zu nehmen: div#content.opus.home h2, div#content.opus.solrsearch h2, div#content.opus.solrsearch_browse h2, div#content.opus.publish h2, div#content.opus.home.home_help h2 { color: var(--etwas_dunkles); }

Das könnte man sicher im CSS unterbringen. Das der Header nicht immer angezeigt wird bzw. der Hintergrund von h2 ist ja ok, siehe Einzeltrefferansicht.