Skip to content

fix: padding/layout update - #5848

Open
Skazitron wants to merge 2 commits into
mainfrom
fix-padding-layout
Open

Skazitron wants to merge 2 commits into
mainfrom
fix-padding-layout

Conversation

@Skazitron

@Skazitron Skazitron commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

Done

  • Fixed the padding issue referenced here
  • Implemented the layout update suggested here

QA

  • Open the demo
  • Navigate to /docs/examples/patterns/_test-article-resources
  • Ensure that the design matches the suggested changes in the figma doc.

Card

https://warthogs.atlassian.net/browse/WD-37058

Check if PR is ready for release

If this PR contains Vanilla SCSS or macro code changes, it should contain the following changes to make sure it's ready for the release:

  • PR should have one of the following labels to automatically categorise it in release notes:
    • Feature 🎁, Breaking Change 💣, Bug 🐛, Documentation 📝, Maintenance 🔨.
  • Vanilla version in package.json should be updated relative to the most recent release, following semver convention
    • if existing APIs (CSS classes & macro APIs) are not changed it can be a bugfix release (x.x.X)
    • if existing APIs (CSS classes & macro APIs) are changed/added/removed it should be a minor version (x.X.0)
    • see the wiki for more details
  • Any changes to component class names (new patterns, variants, removed or added features) or macros should be listed on the what's new page.

@webteam-app

Copy link
Copy Markdown

@mattea-turic

mattea-turic commented Sep 10, 2026

Copy link
Copy Markdown

Thanks @Skazitron !

Looking at the "specs for dev" page in the Figma, the solution was to remove the spacing from the heading and the body, but maintain the spacing for the wrapper housing them:
Screenshot 2026-09-10 at 12 57 32 PM

In the demo link you shared, I don't see this reflected

Also looping in @kim-isaac, as she's the owner of the Figma, and maybe would like to take a look too :)

@kim-isaac

Copy link
Copy Markdown

Thanks for sharing @mattea-turic ! on’t have any other comments beyond what you’ve already mentioned. @Skazitron Based on Mattea’s comment, if metadata is included as well, the padding should be structured like this:
image
Thanks all!

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants