-
Notifications
You must be signed in to change notification settings - Fork 16
Feat/robert #21
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
Feat/robert #21
Conversation
Signed-off-by: lennartrommeiss <lennart.rommeiss@deutschebahn.com>
|
schau mal bitte, ob der Footer jetzt besser aussieht. Habe das Problem gar nicht erkannt vorher :o |
lenderom
left a comment
There was a problem hiding this 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?

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 🤔
|
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>
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>
|
😍😍😍 |
|
Was hältst du von einer Sticky Navbar? :) |


Description
Checklist
The content was modified in the following languages: