Skip to content

fix: add top margin between header and first content section#28

Merged
benpeter merged 1 commit intomainfrom
fix/post-index-header-spacing
Mar 14, 2026
Merged

fix: add top margin between header and first content section#28
benpeter merged 1 commit intomainfrom
fix/post-index-header-spacing

Conversation

@benpeter
Copy link
Owner

Summary

  • The first section's margin-block-start was explicitly set to 0, causing the post index to sit flush against the header with no spacing
  • Changed to var(--section-spacing) (48px) for proper breathing room

Test URLs

Test plan

  • Verify spacing between header and post index on homepage
  • Check mobile, tablet, and desktop breakpoints
  • Verify post detail pages still look correct

🤖 Generated with Claude Code

The first section's margin-block-start was explicitly zeroed, causing the
post index to sit flush against the header border with no breathing room.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Signed-off-by: Ben Peter <bp@ben-peter.com>
@aem-code-sync
Copy link

aem-code-sync bot commented Mar 14, 2026

Hello, I'm the AEM Code Sync Bot and I will run some actions to deploy your branch and validate page speed.
In case there are problems, just click a checkbox below to rerun the respective action.

  • Re-run PSI checks
  • Re-sync branch
Commits

@aem-code-sync
Copy link

aem-code-sync bot commented Mar 14, 2026

Page Scores Audits Google
📱 / PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI
🖥️ / PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI

@benpeter benpeter merged commit 4ca4aa2 into main Mar 14, 2026
3 checks 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.

2 participants