Skip to content

docs: add vertical spacing guidance from MWPW-201396 - #199

Open
qiyundai wants to merge 1 commit into
devfrom
MWPW-201396-spacing-guidance
Open

docs: add vertical spacing guidance from MWPW-201396#199
qiyundai wants to merge 1 commit into
devfrom
MWPW-201396-spacing-guidance

Conversation

@qiyundai

Copy link
Copy Markdown
Collaborator

Summary

  • Records the Design/Consonant/Authoring decision from MWPW-201396: vertical block spacing is a Figma/authoring concern, not something to hard-code in engineering implementations.
  • Adds a ### Vertical Spacing subsection to CLAUDE.md under Coding Standards, so future block work (and Claude Code) follows this convention: no hard-coded top/bottom padding/margin, authors keep flexibility, and any genuinely-required baked-in spacing must be documented (value + viewport + reason).
  • The specific blocks called out in the ticket (plans-hero, hub-hero, offer-hero, logo-ticker, pdf-space, brand-concierge, comparison-table-c2) are Consonant/Milo C2 blocks that don't live in this repo, so this PR ports the principle rather than a block-specific list.

Test plan

  • Docs-only change — no code touched, npm test/npm run lint not applicable
  • Reviewer confirms the guidance text accurately reflects the ticket's confirmed decisions

🤖 Generated with Claude Code

Records the Design/Consonant/Authoring decision that vertical page
spacing belongs in Figma components, not hard-coded in block
CSS/JS, so authors keep flexibility unless a baked-in value is
documented.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@aem-code-sync

aem-code-sync Bot commented Jul 23, 2026

Copy link
Copy Markdown

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 all PSI checks
  • Re-run failed PSI checks
  • Re-sync branch
Commits

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