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

fix: Hilfetext ist falsch formatiert (fehlende Leerzeichen) wegen display: flex #421

Closed
wants to merge 1 commit into from

Conversation

yankee42
Copy link

Der Hilfetext in Formularen ist falsch formatiert:
missing-space

Hier fehlt ein Leerzeichen, was daran liegt, dass die Box in dem der Text steht display: flex hat. Das wiederrum liegt daran, dass die Hilfebox die CSS-Class form-row hat, was eigentlich keinen Text enthalten soll, sondern Flex-Columns. Richtig wäre die CSS-Class form-text und sowohl den Hilfetext als auch das Kontrollelement in eine gemeinsame form-row zu packen. Dieser PR tut das. Ergebnis:

space-ok

Hier auf GitHub sieht der PR größer aus als er ist, weil sich bei einem großen Block die Einrückung ändert. Lässt man Whitespaces beiseite haben sich nur 3 Zeilen geändert:

changes-ignoring-whitespaces

  1. Das <div class="form-row"> habe ich entfernt. Es ist doppelt, weil schon in in Zeile 2.
  2. form-text statt form-row

Ich habe stichprobenartig ein paar Menüs durchgeklickt ob irgendwo durch die Änderung etwas kaputt gegangen ist und mir ist nichts aufgefallen. Ein kritisches Blick wäre dennoch sinnvoll, denn es ist das erste Mal, dass ich etwas mit Vue mache und das erste Mal etwas mit oWB2.

…lo <span>world</span>` to be displayed as `helloworld` (missing space character)
@benderl
Copy link
Contributor

benderl commented Feb 27, 2024

Einen negativen Effekt hat die Änderung dann leider schon:
form-row hat dafür gesorgt, dass die horizontalen Abstände kleiner werden.
image
Gerade bei schmalen Displays sollte das vermieden werden.
image
Damit es zumindest einheitlich aussieht, müssten auch die anderen Komponenten angepasst werden (oder ein Refactoring gemacht werden, um diesen Teil in eine eigene Komponente auszulagern). Bei der Gelegenheit kann dann auch der Abstand wieder korrigiert werden. form-row setzt margin-right und margin-left auf -5px.
URL für diese Testseite: /openWB/web/settings/#/TestingStore

@benderl benderl self-assigned this Feb 27, 2024
@yankee42
Copy link
Author

OK, es sollte es wert sein das zu tun. Den Rest dieser Woche komme ich wahrscheinlich nicht mehr dazu, sehe mir später aber gerne an.

@benderl
Copy link
Contributor

benderl commented Feb 28, 2024

Ich habe mal angefangen. Sieht soweit ganz gut aus. PR kommt bis zum WE mit ein paar umstrukturierten Elementen.
image

@benderl
Copy link
Contributor

benderl commented Mar 1, 2024

Hier der erste Entwurf: #422
Kann sein, dass sich die neue Basis Komponente noch etwas ändert, wenn weitere Komponenten adaptiert werden.

@benderl benderl closed this Apr 25, 2024
This was referenced Jul 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants