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

Biowetter benötigt zusätzliche Styles #67

Open
olwaldi opened this issue Jul 28, 2024 · 8 comments
Open

Biowetter benötigt zusätzliche Styles #67

olwaldi opened this issue Jul 28, 2024 · 8 comments
Labels
enhancement New feature or request

Comments

@olwaldi
Copy link

olwaldi commented Jul 28, 2024

Aktuell nutzt die von biowetter erzeugte Tabelle Style-Elemente von Belcherton. Daher sollte man den Style-File dwd.css wie folgt ergänzen:

.records-header {
    text-align:center;
    background:#D7D7D7;
    color:#000;
    font-weight:400;
}
.table-striped {
    tr:nth-child(even) {background-color: #f2f2f2;}
}

Ggf. die Doku hier entsprechend erweitern.

@roe-dl
Copy link
Owner

roe-dl commented Jul 28, 2024

Ich werde hier noch allgemeine Klassen ergänzen, mit denen man die Gestaltung entsprechend beeinflussen kann.

@roe-dl
Copy link
Owner

roe-dl commented Jul 29, 2024

Folgende Ergänzungen habe ich in Commit 360a5ff vorgenommen:

Die Tabelle selbst hat zusätzlich die Klasse dwdbiowettertable bzw. dwdpollentable, die im CSS definiert werden können. Die Klasse table-striped stammt aus dem Bootstrap-Framework, das nicht nur bei Belchertown sondern auch noch bei anderen Skins verwendet wird.

Zusätzlich sind die Klassen jetzt konfigurierbar. Dabei ist schon vorbereitet, daß es einmal ein Tabellenformat für hochformatige Bildschirme wie bei Mobiltelefonen geben soll. Wie bei dwd-mosmix gibt es jetzt die Option show_placemark, mit der die Angabe des Namens oberhalb der Tabelle ausgeblendet werden kann. Dann gibt es die Option orientation, die momentan noch keine Wirkung hat. Mit horizontal_table_classes können die vorgegebenen Klassen für die Tabelle überschrieben werden, mit horizontal_main_effect_td_classes die der Zwischenüberschriften. Die restlichen Optionen sind noch nicht wirksam.

@roe-dl roe-dl added the enhancement New feature or request label Jul 29, 2024
@olwaldi
Copy link
Author

olwaldi commented Jul 29, 2024 via email

@olwaldi
Copy link
Author

olwaldi commented Jul 30, 2024

Option tut fast perfekt - nur die Legende bleibt noch zu groß. In Zeile 621 sollte es wohl

                            effect = symbol(effect,self.plusminus_icon_size)

heißen.

Vielen Dank!

@teffi71
Copy link

teffi71 commented Jul 30, 2024

Ich verwende Safari als Browser und mir ist aufgefallen, dass das Datum der Tage der Wetterfühligkeit beim Scrollen stehen bleibt, aber leider über dem Inhalt der Tabelle (siehe 2. Bild) ...
Gleiches mit Chrome ...

Screenshot 2024-07-30 at 15 15 56 Screenshot 2024-07-30 at 15 16 19

@teffi71
Copy link

teffi71 commented Jul 30, 2024

Daher sollte man den Style-File dwd.css wie folgt ergänzen:

wo findet man die Datei dwd.css? ich finde nur eine Datei styles.css ...

@roe-dl
Copy link
Owner

roe-dl commented Jul 30, 2024

Da die Tabelle sehr lang ist, ist es hilfreich, wenn die Kopfzeile beim Scrollen oben stehen bleibt. Dazu muss sie im CSS eine Hintergrundfarbe erhalten. Für dunklen Hintergrund etwa:

thead {
    background-color: #222;
}

Wenn es sowohl dunklen als auch hellen Hintergrund gibt und man umschalten kann, etwa:

.light thead {
    background-color: #000;
}
.dark thead {
    background-color: #222;
}

Im Normalfall haben Tabellen einen transparenten Hintergrund, weil die globale Hintergrundfarbe durchscheinen soll. Das geht bloß bei der Scrollfunktion nicht.

Wenn die Kopfzeile beim Scrollen nicht am oberen Bildschirmrand klebt, ist irgendetwas mit dem CSS im Allgemeinen faul.

Eine dwd.css gibt es nicht, aber es gibt in der Beschreibung einen Satz CSS-Deklarationen, den @olwaldi möglicherweise in einer Datei dwd.css gespeichert hat. Im Falle von Belchertown ist custom.css der Ort, wo die CSS-Deklarationen hineingehören. Wenn die Datei existiert, wird sie automatisch eingebunden.

@teffi71
Copy link

teffi71 commented Jul 30, 2024

Obwohl ich überhaupt keine Ahnung von CSS habe, bin ich immer wieder erstaunt, dass ich mit Deiner Hilfe Deine Anregungen umsetzen kann. Danke! :)

Screenshot 2024-07-30 at 18 22 48

roe-dl added a commit that referenced this issue Jul 31, 2024
roe-dl added a commit that referenced this issue Aug 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants