[patch] Mindesthöhe bei Header einer ReadingsGroup

[dennisk]

Hallo,

unter 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

[dennisk]

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

[TomLee]

... 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

[dennisk]

Hallo,

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.

[dennisk]

@justme1968 könntest Du Dir das mal anschauen und ggf. übernehmen? Bis jetzt ist mir noch nichts negatives aufgefallen. Danke.

[dennisk]

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?

[dennisk]

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

[rudolfkoenig]

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?

[TomLee]

Sowas in der Art reicht eigentlich schon um das nachzuvollziehen:

Code Auswählen Erweitern

defmod rg_ip readingsGroup .*:ip$
attr rg_ip mapping %ALIAS
attr rg_ip notime 1

[TomLee]

Ah, reicht es offensichtlich nicht.
Bei mir kann ich es auch nur am Tablet (Chrome/Brave) nachvollziehen.

[TomLee]

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:

Code Auswählen Erweitern

div.pinHeader { height:1em; line-height: 1em; padding:2px; }

[TomLee]

Mit 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.

[dennisk]

@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:

Code Auswählen Erweitern

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?

[dennisk]

Hallo @rudulfkoenig @TomLee
Was haltet ihr von obiger Lösung?

[dennisk]

Hallo nochmal @rudolfkoenig und @TomLee,

könntet ihr euch meinen alternativen Lösungsansatz einmal anschauen und ggf. übernehmen? Danke schon mal.

[rudolfkoenig]

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?

[dennisk]

Danke fürs Reinschauen. Folgend mal eine Minimalconfig, die in einem frisch aufgesetzten fhem Docker Container zum gleichen Darstellungsfehler führt:

Code Auswählen Erweitern

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.

[rudolfkoenig]

abfall.ics fehlt noch.
Wird aber vermutlich bei mir nichts aendern, siehe Anhang.
Ich vermute, dass das Problem anderswo (Browser, OS, etc) liegt.

[rudolfkoenig]

In der Mobilansicht sehe das Problem.
Da hilft es, wenn man line-height aendert:

Code Auswählen Erweitern

body.touch a { font-size: 20px; line-height:0px; }
Kannst Du das bitte probieren (z.Bsp. ueber das im Css FHEMWEB Attribut)?

[dennisk]

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.

[dennisk]

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.

[rudolfkoenig]

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.

[dennisk]

Sieht viel besser aus, vielen Dank fürs Dranbleiben!