Skip to content

Conversation

@therobrob
Copy link
Member

Description

Checklist

  • Changed the date in updatet content pages

The content was modified in the following languages:

  • English
  • German

@therobrob therobrob requested a review from lenderom March 5, 2025 18:45
@lenderom
Copy link
Member

lenderom commented Mar 5, 2025

Sieht echt gut aus :)
Hast du ne Idee wie wir, auf kleineren Devices den Footer noch besser machen können? Also ich meine, dass die Datenschutzerklärung abgeschnitten wird oder zu nah an den Rand ragt.
image

Signed-off-by: lennartrommeiss <lennart.rommeiss@deutschebahn.com>
@therobrob
Copy link
Member Author

schau mal bitte, ob der Footer jetzt besser aussieht. Habe das Problem gar nicht erkannt vorher :o

Copy link
Member

@lenderom lenderom left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ich finde es mega gut gelungen, es sieht durch die neuen Abstände schon auch einfach viel viel stimmiger aus. Echt toll! 🥳

Was ich mich noch frage, ob die Seitenüberschrift vielleicht irgendwie größer sein sollte, als die Teilüberschriften?
image

Im Markdown sind das ja h2 (also ##), die Seitenüberschrift scheint aber auch eine h2 zu sein? Man könnte natürlich auch sagen, dass für Texte im Markdown immer h3 (###) genutzt werden soll. Aber das ist in der Nutzung vielleicht auch verwirrend 🤔

@lenderom
Copy link
Member

lenderom commented Mar 5, 2025

schau mal bitte, ob der Footer jetzt besser aussieht. Habe das Problem gar nicht erkannt vorher :o

image
Sieht deutlich besser aus :)

@therobrob
Copy link
Member Author

Das ist tatsächlich noch ein Klärungspunkt. Wir müssen die Überschriftenhierarchie einhalten, da gibt es momentan z. B. bei /news/2/ einen merkwürdigen Zustand. Natürlich können wir die sichtbare Seitenüberschrift optisch etwas größer machen :)

Signed-off-by: lennartrommeiss <lennart.rommeiss@deutschebahn.com>
@lenderom
Copy link
Member

lenderom commented Mar 5, 2025

Das ist tatsächlich noch ein Klärungspunkt. Wir müssen die Überschriftenhierarchie einhalten, da gibt es momentan z. B. bei /news/2/ einen merkwürdigen Zustand. Natürlich können wir die sichtbare Seitenüberschrift optisch etwas größer machen :)

Ja das wäre ganz gut glaube ich :)

Die News waren einfach falsch benutzt. Das habe ich gerade mal gefixt 😅

…if statement instead

Signed-off-by: lennartrommeiss <lennart.rommeiss@deutschebahn.com>
@therobrob therobrob requested a review from lenderom March 5, 2025 20:40
@lenderom
Copy link
Member

lenderom commented Mar 5, 2025

😍😍😍

@lenderom
Copy link
Member

lenderom commented Mar 5, 2025

Was hältst du von einer Sticky Navbar? :)

@therobrob therobrob merged commit 281fd6a into main Mar 5, 2025
1 check passed
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.

3 participants