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

Ansicht "Vorhersage vom DWD" #86

Closed
Daveiano opened this issue Oct 20, 2022 Discussed in #85 · 11 comments
Closed

Ansicht "Vorhersage vom DWD" #86

Daveiano opened this issue Oct 20, 2022 Discussed in #85 · 11 comments
Labels
enhancement New feature or request
Milestone

Comments

@Daveiano
Copy link
Owner

Discussed in #85

Originally posted by hoetzgit October 20, 2022
Hallo David,

ich möchte Dich nicht überstrapazieren, aber was hältst Du davon, die Vorhersage etwas zu strukturieren? (Muster: DWD Seite)

Beispiel:

image

@Daveiano Daveiano added the enhancement New feature or request label Oct 20, 2022
@hoetzgit
Copy link
Contributor

hoetzgit commented Oct 20, 2022

@Daveiano

Der DWD hat ja in seinen Vorhersagetexten nochmals gewisse Merkmale (GEWITTER:/NEBEL:/WIND/STURM: ...) unterteilt. In Sachsen aktuell "WIND:") Ich habe jetzt mal nach dem wget der Texte auf die Schnelle ein sed gemacht und diese Merkmale mit <b> und <br> versehen. Evtl. könntest Du dies aber auch in Deiner Skin schöner und mit den entsprechenden Klassen formatieren. Das sieht dann bspw. so aus:

image

@Daveiano
Copy link
Owner Author

Daveiano commented Oct 20, 2022

So etwas (Manipulation der Daten vom DWD) würde ich eher bei weewx-DWD direkt in wget-dwd sehen als im Skin. Der Skin zeigt nur an was er bekommt. Denke das sollte man dort anbringen und vielleicht als optional implementieren.

@Daveiano Daveiano added this to the 2.3.1 milestone Oct 20, 2022
Daveiano added a commit that referenced this issue Oct 21, 2022
@Daveiano
Copy link
Owner Author

Screenshot from 2022-10-21 02-01-27

Ich hätte es jetzt so eingebaut, was hälst du davon?

@hoetzgit
Copy link
Contributor

Sehr gut! Viel besser lesbar finde ich.

@hoetzgit
Copy link
Contributor

Ich habe Deine Änderungen für die DWD Seite eingebaut, dabei ist mir was komisches aufgefallen: Die Schriften sind irgendwie so "verkrummelt", egal welchen Browser ich benutze. Habe auch mal auf Deine Seite geschaut, da ist es genauso.

image

@Daveiano
Copy link
Owner Author

Daveiano commented Oct 21, 2022

Du meinst, das das so ein bisschen "ausgefranst" aussieht oder? Sieht bei mir nicht so aus würde ich meinen:

Screenshot from 2022-10-21 18-10-07

Welche Betriebssytem/Browser Kombination nutzt du? Kannst du das auch bei anderen Texten auf der Seite feststellen?

@hoetzgit
Copy link
Contributor

Ja, genau "ausgefranst" ist das bessere Wort. Komisch, auch auf Mobile/Tab ist die Darstellung völlig ok. Der Screenshot ist auf einem Win11 PC entstanden, geprüft mit Firefox und Chrome basierten Browsern (Opera, Chrome, Edge). Ich müsste das mal auf einem Linux System ansehen. Ich kenne mich da nicht aus, aber kann es sein, dass die Schriften irgendwie gerendert werden müssen oder sowas in der Richtung? Aber da das Problem wahrscheinlich irgendwo bei mir liegt kannst Du für Dich einen Haken dran machen.

Viele Grüße und Danke
Henry

@hoetzgit
Copy link
Contributor

Da ich Deine Lösung mit Schleife und den Wochentagen sehr schön finde, habe ich das in meiner Anpassung für die Belchertown Skin nachgezogen. Dadurch ist mir noch eine Anpassung bei Dir aufgefallen. Du könntest gettext noch in die $headings einbauen.

@Daveiano
Copy link
Owner Author

Daveiano commented Oct 23, 2022

Ja, genau "ausgefranst" ist das bessere Wort. Komisch, auch auf Mobile/Tab ist die Darstellung völlig ok. Der Screenshot ist auf einem Win11 PC entstanden, geprüft mit Firefox und Chrome basierten Browsern (Opera, Chrome, Edge). Ich müsste das mal auf einem Linux System ansehen. Ich kenne mich da nicht aus, aber kann es sein, dass die Schriften irgendwie gerendert werden müssen oder sowas in der Richtung? Aber da das Problem wahrscheinlich irgendwo bei mir liegt kannst Du für Dich einen Haken dran machen.

Viele Grüße und Danke Henry

Mein Screenshot ist von einem Ubuntu, da sieht alles schick aus. Für Windows habe ich das gefunden: https://www.it-zeugs.de/grauslige-unscharfe-schrift-im-browser-chrome.html

Da ich Deine Lösung mit Schleife und den Wochentagen sehr schön finde, habe ich das in meiner Anpassung für die Belchertown Skin nachgezogen. Dadurch ist mir noch eine Anpassung bei Dir aufgefallen. Du könntest gettext noch in die $headings einbauen.

Du meinst damit man "Aktuelle Wetterlage", "Heute" und "Morgen" auch ändern kann (Die 2 Wochentagsnamen werden ja generiert)? Ja das macht Sinn!

Edit: Eigentlich wäre es auch sinnvoll alle Texte auf der DWD Seite noch in $gettext() zu packen? Ich dachte am Anfang das wäre nicht nötig, da es wohl nur in Deutschland genutzt wird (un eine Übersetzung nicht nötig wäre). Aber es wäre ja auch sinnvoll damit die Texte vom Nutzer einfacher geändert werden können in der de.conf!?

@hoetzgit
Copy link
Contributor

Mein Screenshot ist von einem Ubuntu, da sieht alles schick aus. Für Windows habe ich das gefunden: https://www.it-zeugs.de/grauslige-unscharfe-schrift-im-browser-chrome.html

Das habe ich jetzt überprüft, danke für den Link. War allerdings aktiviert und hat auch keine Verbesserung gebracht. Da das Problem offensichtlich clientabhänging ist, habe ich ja eh keine Chance und ignoriere das jetzt einfach mal.

Du meinst damit man "Aktuelle Wetterlage", "Heute" und "Morgen" auch ändern kann (Die 2 Wochentagsnamen werden ja generiert)? Ja das macht Sinn!

Ja genau, die drei fest codierten Label.

Edit: Eigentlich wäre es auch sinnvoll alle Texte auf der DWD Seite noch in $gettext() zu packen? Ich dachte am Anfang das wäre nicht nötig, da es wohl nur in Deutschland genutzt wird (un eine Übersetzung nicht nötig wäre). Aber es wäre ja auch sinnvoll damit die Texte vom Nutzer einfacher geändert werden können in der de.conf!?

Diese Überlegungen hatte ich auch, als ich Dir das mit den drei Labels und gettext geschrieben habe. Da ich aber gesehen habe wie sorgfältig Du Deinen Code schreibst ist es nur konsequent, wenn Du diese Möglichkeiten nutzt, auch wenn es auf den ersten Blick nicht an jeder Stelle sinnvoll erscheint.

@Daveiano
Copy link
Owner Author

Ok, habe das bei allen Texten/Überschriften sofern möglich noch hinzugefügt. Wird im nächsten 2.3.1 release drin sein, danke für den Input!

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

2 participants