remove justify content 'center' #239

srsh · 2024-01-13T18:50:45+01:00

srsh commented

2024-01-13 18:50:45 +01:00

We got this hint from a ux/ui designer, it will improve the reading flow significantly. He will be visiting our telebier on 17.01, If you like to discuss it with him

srsh added 1 commit 2024-01-13 18:50:45 +01:00

remove justify content 'center'

continuous-integration/drone/pr Build is passing

Details

cf004302b1

muli commented

2024-01-13 19:43:08 +01:00

Soweit ich sehe hat die Zeile, die du gelöscht hast keinerlei Auswirkung und kann deshalb ohnehin weg. Die Ausrichtung des Texts wird für die Überschrift und den Fließtext in den Karten jeweils separat überschrieben.

Das zentrierter Text von der Lesbarkeit her schlecht ist, ist bekannt, deshalb kommt das auf der Seite auch kaum vor. Hier geht es ja auch nicht um Mengen an Fließtext, sondern nur um mehrere kurze Abschnitte. Trotzdem bin ich für linksbündigen Fließtext in den Karten durchaus offen.
Bei der Überschrift sieht das aber mMn in sämtlichen sinnvollen Kombination aus font-size und font-weight bei mindestens einer Karte sehr ungünstig aus.

Würde das für den Fließtext also so umsetzen. Falls der UX/UI-Mensch Zeit hat für die Karten insgesamt etwas neues zu bauen, was linksbündig schick ist, bin ich für PRs offen.

Soweit ich sehe hat die Zeile, die du gelöscht hast keinerlei Auswirkung und kann deshalb ohnehin weg. Die Ausrichtung des Texts wird für die Überschrift und den Fließtext in den Karten jeweils separat überschrieben. Das zentrierter Text von der Lesbarkeit her schlecht ist, ist bekannt, deshalb kommt das auf der Seite auch kaum vor. Hier geht es ja auch nicht um Mengen an Fließtext, sondern nur um mehrere kurze Abschnitte. Trotzdem bin ich für linksbündigen Fließtext in den Karten durchaus offen. Bei der Überschrift sieht das aber mMn in sämtlichen sinnvollen Kombination aus `font-size` und `font-weight` bei mindestens einer Karte sehr ungünstig aus. Würde das für den Fließtext also so umsetzen. Falls der UX/UI-Mensch Zeit hat für die Karten insgesamt etwas neues zu bauen, was linksbündig schick ist, bin ich für PRs offen.

muli commented

2024-01-13 19:48:02 +01:00

Soweit ich sehe hat die Zeile, die du gelöscht hast keinerlei Auswirkung und kann deshalb ohnehin weg. Die Ausrichtung des Texts wird für die Überschrift und den Fließtext in den Karten jeweils separat überschrieben.

Das zentrierter Text von der Lesbarkeit her schlecht ist, ist bekannt, deshalb kommt das auf der Seite auch kaum vor. Hier geht es ja auch nicht um Mengen an Fließtext, sondern nur um mehrere kurze Abschnitte. Trotzdem bin ich für linksbündigen Fließtext in den Karten durchaus offen.
Bei der Überschrift sieht das aber mMn in sämtlichen sinnvollen Kombination aus font-size und font-weight bei mindestens einer Karte sehr ungünstig aus.

Würde das für den Fließtext also so umsetzen. Falls der UX/UI-Mensch Zeit hat für die Karten insgesamt etwas neues zu bauen, was linksbündig schick ist, bin ich für PRs offen.

Update
Alles zurück: Das geht gar nicht auf den Text sondern richtet die Cards vertikal aus. Sollte definitiv raus und ist ohne besser. Den linksbündigen Fließtext schieb ich trotzdem noch nach.

> Soweit ich sehe hat die Zeile, die du gelöscht hast keinerlei Auswirkung und kann deshalb ohnehin weg. Die Ausrichtung des Texts wird für die Überschrift und den Fließtext in den Karten jeweils separat überschrieben. > > Das zentrierter Text von der Lesbarkeit her schlecht ist, ist bekannt, deshalb kommt das auf der Seite auch kaum vor. Hier geht es ja auch nicht um Mengen an Fließtext, sondern nur um mehrere kurze Abschnitte. Trotzdem bin ich für linksbündigen Fließtext in den Karten durchaus offen. > Bei der Überschrift sieht das aber mMn in sämtlichen sinnvollen Kombination aus `font-size` und `font-weight` bei mindestens einer Karte sehr ungünstig aus. > > Würde das für den Fließtext also so umsetzen. Falls der UX/UI-Mensch Zeit hat für die Karten insgesamt etwas neues zu bauen, was linksbündig schick ist, bin ich für PRs offen. **Update** Alles zurück: Das geht gar nicht auf den Text sondern richtet die Cards vertikal aus. Sollte definitiv raus und ist ohne besser. Den linksbündigen Fließtext schieb ich trotzdem noch nach.

muli added 1 commit 2024-01-13 19:49:35 +01:00

feat: Left aligned body text of card.

continuous-integration/drone/pr Build is passing

Details

59ee17c4ac

muli merged commit 4abb52075d into main

2024-01-13 19:51:37 +01:00

muli referenced this issue from a commit

2024-01-13 19:51:38 +01:00

Merge pull request 'remove justify content 'center'' (#239) from text-align into main

muli commented

2024-01-13 19:56:50 +01:00