FHEM Forum

FHEM => Frontends => readingsGroup / readingsHistory => Thema gestartet von: dennisk am 27 Oktober 2024, 11:10:59

Titel: [patch] Mindesthöhe bei Header einer ReadingsGroup
Beitrag von: dennisk am 27 Oktober 2024, 11:10:59
Hallo,

unter https://forum.fhem.de/index.php?msg=1323007 (https://forum.fhem.de/index.php?msg=1323007) hatte ich von einem Problem berichtet, dass die Schrift im Header einer ReadingsGroup in die erste Tabellenzeile hineinragen kann. Dafür habe ich nun einen möglichen Fix gefunden, siehe angehängter Patch. Nebenwirkungen sind mir bisher keine aufgefallen. Ich würde mich freuen, wenn sich das einer der Entwickler ansehen und dann ggf. übernehmen könnte. Sollten doch Nebenwirkungen auftreten, freue ich mich über Rückmeldung und Hinweise/Vorschläge hinsichtlich einer alternativen Lösungsmöglichkeit.

Viele Grüße
dennisk
Titel: Aw: [patch] Mindesthöhe bei Header einer ReadingsGroup
Beitrag von: dennisk am 07 November 2024, 22:08:50
Hallo zusammen,

da ich noch nichts gehört habe, nochmal die Nachfrage, ob dieser kleine Patch integriert werden kann? Bisher ist mir jedenfalls keine Nebenwirkung aufgefallen.

Danke und viele Grüße
Titel: Aw: [patch] Mindesthöhe bei Header einer ReadingsGroup
Beitrag von: TomLee am 07 November 2024, 23:33:21
Zitat... da ich noch nichts gehört habe, ...

Hallo,

Du hast die falsche "Adresse" (siehe help readingsGroup) angegeben, nicht auszuschliessen das darum beim FHEM-Modul-Entwickler bisher nix ankam.

Gruß Thomas
Titel: Aw: [patch] Mindesthöhe bei Header einer ReadingsGroup
Beitrag von: dennisk am 08 November 2024, 07:10:37
Zitat von: TomLee am 07 November 2024, 23:33:21Hallo,

Du hast die falsche "Adresse" (siehe help readingsGroup) angegeben, nicht auszuschliessen das darum beim FHEM-Modul-Entwickler bisher nix ankam.

Gruß Thomas

Vielen Dank für den Hinweis! Hab das Thema verschoben.
Titel: Aw: [patch] Mindesthöhe bei Header einer ReadingsGroup
Beitrag von: dennisk am 17 November 2024, 13:28:52
@justme1968 könntest Du Dir das mal anschauen und ggf. übernehmen? Bis jetzt ist mir noch nichts negatives aufgefallen. Danke.
Titel: Aw: [patch] Mindesthöhe bei Header einer ReadingsGroup
Beitrag von: dennisk am 12 Januar 2025, 09:42:46
Weiß jemand, was mit justme1968 ist? Hier hat er noch nicht reagiert, und per PM auch nicht. Er ist doch der richtige Ansprechpartner für diese kleine Anpassung, oder?
Titel: Aw: [patch] Mindesthöhe bei Header einer ReadingsGroup
Beitrag von: dennisk am 18 Februar 2025, 16:41:32
Leider konnte ich justme1968 bis heute nicht erreichen, sowohl hier nicht als auch nicht per PN.
@rudolfkoenig Könntest Du Dir den Patch vielleicht ansehen und ggf. einchecken?
Vielen Dank und viele Grüße
Titel: Aw: [patch] Mindesthöhe bei Header einer ReadingsGroup
Beitrag von: rudolfkoenig am 19 Februar 2025, 10:24:09
Ich habe das jetzt angeschaut: f18style.css ist meine Baustelle.

Mit dem Patch habe ich links im Menu Artifakte, siehe Anhang (neu1/neu2), die in der alten Version so nicht habe.
Kannst Du mir bitte eine Testkonfiguration bereitstellen, damit ich das Problem selbst sehe?
Titel: Aw: [patch] Mindesthöhe bei Header einer ReadingsGroup
Beitrag von: TomLee am 19 Februar 2025, 11:58:14
Sowas in der Art reicht eigentlich schon um das nachzuvollziehen:

defmod rg_ip readingsGroup .*:ip$
attr rg_ip mapping %ALIAS
attr rg_ip notime 1
Titel: Aw: [patch] Mindesthöhe bei Header einer ReadingsGroup
Beitrag von: TomLee am 19 Februar 2025, 12:39:32
Ah, reicht es offensichtlich nicht.
Bei mir kann ich es auch nur am Tablet (Chrome/Brave) nachvollziehen.
Titel: Aw: [patch] Mindesthöhe bei Header einer ReadingsGroup
Beitrag von: TomLee am 19 Februar 2025, 17:01:22
min-height:auto; verhindert bei mir auf dem Tablet nicht, das der Linktext in die erste Tabellenzeile hineinragt.
Mit der zusätzlichen Angabe von line-height: 1em; rutscht der Link bei mir mit 2px Abstand nach oben und unten ist bspw. das g dann bündig mit dem Rahmen:
div.pinHeader { height:1em; line-height: 1em; padding:2px; }
Titel: Aw: [patch] Mindesthöhe bei Header einer ReadingsGroup
Beitrag von: TomLee am 20 Februar 2025, 13:19:57
ZitatMit dem Patch habe ich links im Menu Artifakte, siehe Anhang (neu1/neu2), die in der alten Version so nicht habe.

Das kommt von der fehlenden height-Angabe im Patch und das div an der Stelle keinen Inhalt hat. Weil das div in dem div rechts floated, hat dessen Höhe keinen Einfluss auf sein Elternelement und es kommt zu dem Effekt.
Titel: Aw: [patch] Mindesthöhe bei Header einer ReadingsGroup
Beitrag von: dennisk am 22 Februar 2025, 10:04:34
@rudulfkoenig @TomLee vielen Dank für eure Rückmeldungen!
Wenn ich es jetzt richtig verstehe, dann hat mein Patch Seiteneffekte - ist mir bis dato noch gar nicht aufgefallen, tritt bei mir aber genauso auf.

Ich habe nochmal ein wenig ausprobiert und komme auf folgende Variante:
div.pinHeader { height:1.1em; line-height:1em; padding:2px; }Damit habe ich in Brave und Firefox unter Android sowie Vivaldi, Chromium und Firefox unter Linux keine Artefakte auf der linken Seite und das g ragt nicht in den Rahmen. Wie sieht das bei euch aus? Wäre das so überhaupt eine adäquate Lösung?
Titel: Aw: [patch] Mindesthöhe bei Header einer ReadingsGroup
Beitrag von: dennisk am 05 März 2025, 18:36:57
Hallo @rudulfkoenig @TomLee
Was haltet ihr von obiger Lösung?
Titel: Aw: [patch] Mindesthöhe bei Header einer ReadingsGroup
Beitrag von: dennisk am 15 März 2025, 13:36:39
Hallo nochmal @rudolfkoenig und @TomLee,

könntet ihr euch meinen alternativen Lösungsansatz einmal anschauen und ggf. übernehmen? Danke schon mal.
Titel: Aw: [patch] Mindesthöhe bei Header einer ReadingsGroup
Beitrag von: rudolfkoenig am 16 März 2025, 16:37:48
Ich habe die Alternative angeschaut: die Ueberschrift-Zeile finde ich damit "grundlos" zu hoch.

Da ich das Problem nicht nachstellen kann (bei mir haengt kein g ueber die Linie): kannst Du mir bitte eine Konfiguration bauen, mit dem ich selber nach einer Loesung suchen kann?
Titel: Aw: [patch] Mindesthöhe bei Header einer ReadingsGroup
Beitrag von: dennisk am 16 März 2025, 17:20:14
Danke fürs Reinschauen. Folgend mal eine Minimalconfig, die in einem frisch aufgesetzten fhem Docker Container zum gleichen Darstellungsfehler führt:
define AbfallkalenderShow Calendar ical file /tmp/abfall.ics
setuuid AbfallkalenderShow 67d6f822-f33f-629e-04b9-4cebe2963aa85936
attr AbfallkalenderShow hideOlderThan 1
define AbfallkalenderShowView CALVIEW AbfallkalenderShow 2
setuuid AbfallkalenderShowView 67d6f86a-f33f-629e-5cbd-54a5a6fea556bd89
attr AbfallkalenderShowView maxreadings 10
attr AbfallkalenderShowView modes next
define AbfallkalenderShowRg readingsGroup <Datum>,<Abfallart> AbfallkalenderShowView:t_001_bdate,t_001_summary AbfallkalenderShowView:t_002_bdate,t_002_summary AbfallkalenderShowView:t_003_bdate,t_003_summary AbfallkalenderShowView:t_004_bdate,t_004_summary AbfallkalenderShowView:t_005_bdate,t_005_summary AbfallkalenderShowView:t_006_bdate,t_006_summary AbfallkalenderShowView:t_007_bdate,t_007_summary AbfallkalenderShowView:t_008_bdate,t_008_summary AbfallkalenderShowView:t_009_bdate,t_009_summary AbfallkalenderShowView:t_010_bdate,t_010_summary
setuuid AbfallkalenderShowRg 67d6f899-f33f-629e-428f-fa7ef3a7aa35fb8e
attr AbfallkalenderShowRg alias Entsorgungstermine
attr AbfallkalenderShowRg group Müll
attr AbfallkalenderShowRg nonames 1
attr AbfallkalenderShowRg room Müll

Wenn ich noch mehr Infos liefern kann, bitte Bescheid sagen.
Titel: Aw: [patch] Mindesthöhe bei Header einer ReadingsGroup
Beitrag von: rudolfkoenig am 16 März 2025, 17:46:08
abfall.ics fehlt noch.
Wird aber vermutlich bei mir nichts aendern, siehe Anhang.
Ich vermute, dass das Problem anderswo (Browser, OS, etc) liegt.
Titel: Aw: [patch] Mindesthöhe bei Header einer ReadingsGroup
Beitrag von: rudolfkoenig am 16 März 2025, 17:57:12
In der Mobilansicht sehe das Problem.
Da hilft es, wenn man line-height aendert:
body.touch a { font-size: 20px; line-height:0px; }
Kannst Du das bitte probieren (z.Bsp. ueber das im Css FHEMWEB Attribut)?
Titel: Aw: [patch] Mindesthöhe bei Header einer ReadingsGroup
Beitrag von: dennisk am 16 März 2025, 18:53:31
Stimmt, ist unter Firefox auf Android. Anbei ein Screenshot mit Deinem CSS, besser aber immer noch reinragend. Unter Brave auf Android übrigens das gleiche Problem, ist ne andere Rendering Engine als Firefox. Auf nem Desktop kann ich nachher erst testen.
Titel: Aw: [patch] Mindesthöhe bei Header einer ReadingsGroup
Beitrag von: dennisk am 16 März 2025, 20:51:06
Auf Firefox und Vivaldi auf dem Desktop macht das CSS keinen Unterschied, kann das sein? Das g ist zwar sehr eng an der Linie, ragt aber nicht rein.
Titel: Aw: [patch] Mindesthöhe bei Header einer ReadingsGroup
Beitrag von: rudolfkoenig am 28 März 2025, 21:17:02
Ich habe jetzt in der pinHeader Anweisung height gegen min-height getauscht, und es schaut fuer mich akzeptabel aus.

Die Unterschiede kommen von den unterschiedlichen Fonts: f18.js spezifiziert nur "Arial, sans-serif", das wird aber je nach Plattform mit unterschiedlichen Fonts realisiert.
Titel: Aw: [patch] Mindesthöhe bei Header einer ReadingsGroup
Beitrag von: dennisk am 28 März 2025, 22:31:34
Sieht viel besser aus, vielen Dank fürs Dranbleiben!