Add website footer #27
Conversation
|
Your Render PR Server URL is https://kulturdaten-website-frontend-pr-27.onrender.com. Follow its progress at https://dashboard.render.com/web/srv-cj9oqim3ttrc73cc790g. |
| const Row = styled.div({ | ||
| display: "flex", | ||
| flexWrap: "wrap", | ||
| gap: spacings.get(5), |
There was a problem hiding this comment.
Das hier ist vermutlich überfällig. Obwohl ich intuitiv in ner flexbox vermutlich schon mit nem gap anstatt mit dem spacer arbeiten würde. Vielleicht kann man da mal kurz was abquatschen
There was a problem hiding this comment.
Der gap hier definiert den horizontalen Zwischenraum zwischen z.B. dem linken Block (mit Adresse und E-mail) und dem rechten Block (mit den beiden Links). Das habe ich hinzugefügt, damit die beiden Blöcke sich nicht bei bestimmten Bildschirmbreiten "berühren", sondern dann lieber in die nächste Zeile wrappen.
Aber stimmt, statt gap könnte man innerhalb von <Row> einen <Spacer /> verwenden. 🤔
There was a problem hiding this comment.
Aber stimmt, statt gap könnte man innerhalb von
<Row>einen<Spacer />verwenden. 🤔
… sprach er, aber irrte sich. Mit gap ist es viel leichter, das Wrapping gut abzubilden. Wenn man einen <Spacer /> dafür verwendet, ist dieser ggf. noch nicht in der nächsten "Zeile", obwohl das nächste Flex-Item schon in der nächsten Zeile ist. Von daher würde ich’s gern so lassen. :D
| </Address> | ||
| <PageLinks> | ||
| <LinkInternal href="/privacy-policy">{t("link-privacy-policy")}</LinkInternal> | ||
| <Spacer size={4} /> |
There was a problem hiding this comment.
Gleicher Punkt wie oben gap vs Spacer
|
Der footer in figma ist ja etwas unaligned. Hab jetzt was die Abstuande und Positionen angeht nicht so genau geschaut. Sieht aber gut aus finde ich! |
Adds the website footer with links to the upcoming new pages (privacy-policy and imprint).
Draft for now, as this branch is based on thenewsletter-sectionbranch (#22).Screenshots