Tighten mobile content rail padding to px-2 #22

Closed

michalvankodev wants to merge 2 commits from mobile-content-padding into main

pull from: mobile-content-padding

merge into: michalvankodev:main

michalvankodev:main

michalvankodev:redesign/10-mobile-readability

michalvankodev:redesign/00-foundation

michalvankodev:redesign/modern-technical

Conversation 2 Commits 2 Files changed 5 +12 -12

michalvankodev commented 2026-09-12 14:28:13 +02:00

Owner

Copy link

Tightens the shared horizontal page rail to px-2 (0.5rem) — the old px-5 (1.25rem) felt too wide on phones.

Changed everywhere the rail lives, so edges stay aligned on every page:

• base.html — the standard page content container (portfolio, blog list, showcase, contact, ...)
• blog_post.html — article header, body container, and footer
• index.html — all homepage sections
• site_header.html / site_footer.html — masthead nav and footer (same rail)

Interior article typography (.article-body px-4) is untouched — it exists for breakout geometry (figures/code/tables overflow the text column symmetrically).

On wide screens nothing visually shifts: containers are centered by max-width, the padding only applies while the viewport is narrower than the rail.

Tightens the shared horizontal page rail to `px-2` (0.5rem) — the old `px-5` (1.25rem) felt too wide on phones. **Changed everywhere the rail lives, so edges stay aligned on every page:** - `base.html` — the standard page content container (portfolio, blog list, showcase, contact, ...) - `blog_post.html` — article header, body container, and footer - `index.html` — all homepage sections - `site_header.html` / `site_footer.html` — masthead nav and footer (same rail) Interior article typography (`.article-body` px-4) is untouched — it exists for breakout geometry (figures/code/tables overflow the text column symmetrically). On wide screens nothing visually shifts: containers are centered by max-width, the padding only applies while the viewport is narrower than the rail.

michalvankodev added 1 commit 2026-09-12 14:28:13 +02:00

Increase mobile content rail padding to 2rem ... f2faeed051

Bump the shared page rail from px-5 (1.25rem) to px-8 (2rem) across
all page containers — base content div, blog post header/body/footer,
index sections, and the site header/footer — so content edges stay
aligned while phones get a more comfortable ~2rem gutter.

forgejo-actions commented 2026-09-12 14:30:21 +02:00

Copy link

🪦 Live preview

removed (PR closed)

<!-- preview-link --> ## 🪦 Live preview removed (PR closed)

michalvankodev added 1 commit 2026-09-12 17:36:53 +02:00

Tighten the rail to px-2 (0.5rem) ... 7482bcc4fa

2rem was wider than intended; drop the shared page rail to px-2.

michalvankodev changed title from Increase mobile content rail padding to 2rem to Tighten mobile content rail padding to px-2 2026-09-12 17:37:17 +02:00

michalvankodev commented 2026-09-13 21:29:50 +02:00

Author

Owner

Copy link

Superseded by #23, which takes a different approach to the mobile content rail.

Superseded by #23, which takes a different approach to the mobile content rail.

michalvankodev closed this pull request 2026-09-13 21:29:56 +02:00

All checks were successful

Hide checks

preview / preview hostname (push) Successful in 0s

Details

preview / teardown preview (push) Has been skipped

Details

test / cargo test (push) Successful in 58s

Details

preview / build + deploy preview (push) Successful in 1m13s

Details

preview / preview hostname (pull_request) Successful in 0s

Details

preview / build + deploy preview (pull_request) Has been skipped

Details

preview / teardown preview (pull_request) Successful in 0s

Details

Pull request closed

Please reopen this pull request to perform a merge.

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels

No items

No labels

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

2 participants

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference

michalvankodev/michalvankodev-site!22

No description provided.