search

OPUS4 / application

OPUS 4 application.
Other
15 stars 21 forks source link

Fehler im neuen Layout bei ID-Icons auf der Frontdoor #1198

Closed alw-bsz closed 3 months ago

alw-bsz commented 6 months ago

Das letzte ID-Icon bei der letzten Person ist im neuen Layout nach unten verschoben:

grafik

Ursächlich dafür ist die Klasse

.frontdoordata td>*:last-child {
  margin-bottom: 0;
}

in frontdoor.css.

Soweit ich herausgefunden habe, dient die Klasse nur dazu, den Abstand zum unteren Rand bei Bemerkungen zu verringern.

"Bemerkung" mit der Klasse: grafik "Bemerkung" ohne die Klasse: grafik

Der Effekt lässt sich auch erreichen, wenn man die Klasse .preserve-spaces (ebenfalls in frontdoor.css) um margin-bottom ergänzt: 

.preserve-spaces {
  white-space: pre-wrap;
  margin-bottom: 0;
}

.preserve-spaces wird ansonsten nur noch beim Abstract verwendet. Soweit ich feststellen konnte, hat die margin-bottom-Angabe keine Auswirkungen auf dessen Darstellung.

Ich würde also die Klasse .frontdoordata td>*:last-child löschen und stattdessen .preserve-spaces um margin-bottom ergänzen.

Da ich nicht in die Entwicklung des neuen Layouts involviert war, möchte ich das erst hier zur Diskussion stellen, bevor ich es umsetze.

j3nsch commented 5 months ago

Danke, für den Hinweis. Die Klasse ''preserve-spaces" zu erweitern ist nicht in Ordnung, auch wenn das im Augenblick funktionieren würde. Die Klasse hat einen konkreten Zweck und einen entsprechenden Namen. Da zusätzliche Formatierungen mit reinzubringen ist eine "Falle" für zukünftige Änderungen und Entwickler.

j3nsch commented 3 months ago

Das verursachende CSS wirkt sich auf so viele Element aus, dass dort ein Kommentar zur Erläuterung der Gründe sinnvoll wäre. Ich habe den Wirkungsbereich jetzt eingeschränkt, so dass Bemerkungen immer noch wie vorher angezeigt werden, aber die ORCID- und GND-Links nicht mehr beeinträchtigt werden. Falls das an anderen Stellen unerwünschte Auswirkungen hat, bitte melden.