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