Skip to content

docs(memory): note the Webflow p-margin reset and the <br> constraint - #31

Merged
jansroka merged 1 commit into
mainfrom
docs/webflow-css-traps
Aug 23, 2026
Merged

docs(memory): note the Webflow p-margin reset and the <br> constraint#31
jansroka merged 1 commit into
mainfrom
docs/webflow-css-traps

Conversation

@jansroka

Copy link
Copy Markdown
Contributor

Hält zwei Fallen fest, die in dieser Session je einen Fehlversuch gekostet haben und aus dem Code nicht ableitbar sind.

  • p { margin-bottom: 0 } (inog-website.webflow.css:216) lässt Markdown-Absätze ohne Abstand rendern. Der Fix muss auf .content-box begrenzt sein — die weiter gefasste Variante auf .block-content hat die Startseite um 150px verkürzt (fix(css): restore paragraph spacing on Markdown pages #30).
  • <br> im Adressblock ist nicht redundant: ohne die Tags rendert die Anschrift als eine Zeile, und die Markdown-Alternative (zwei Leerzeichen am Zeilenende) entfernt Prettier beim nächsten npm run format.

Dazu die Verifikationsmethode, die die Startseiten-Regression überhaupt sichtbar gemacht hat: Seitenhöhen im Headless-Browser gegen einen frischen Clone von main vergleichen, über alle Seiten.

Reine Dokumentation unter memory/ — der Ordner ist in _config.yml von der Veröffentlichung ausgeschlossen, die Website ändert sich nicht.

Zwei Fallen, die in dieser Session je einen Fehlversuch gekostet haben:

- `p { margin-bottom: 0 }` im Webflow-Export lässt Markdown-Absätze ohne
  Abstand rendern; der Fix muss auf `.content-box` begrenzt sein, sonst
  verkürzt er die Startseite (PR #30).
- `<br>` im Adressblock ist nicht redundant: ohne die Tags rendert die
  Anschrift als eine Zeile, und die Markdown-Alternative (zwei Leerzeichen
  am Zeilenende) entfernt Prettier.
@jansroka
jansroka force-pushed the docs/webflow-css-traps branch from 7fea53e to 7ad1b13 Compare August 23, 2026 22:00
@jansroka
jansroka merged commit 87cd95d into main Aug 23, 2026
1 check passed
@jansroka
jansroka deleted the docs/webflow-css-traps branch August 23, 2026 22:04
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.

1 participant