Skip to content

poc: Experiment with prose component to automate typographic whitespace#2497

Closed
VincentSmedinga wants to merge 3 commits intodevelopfrom
poc/prose-vertical-space
Closed

poc: Experiment with prose component to automate typographic whitespace#2497
VincentSmedinga wants to merge 3 commits intodevelopfrom
poc/prose-vertical-space

Conversation

@VincentSmedinga
Copy link
Copy Markdown
Contributor

@VincentSmedinga VincentSmedinga commented Mar 24, 2026

Here’s a longer article with more types of content. Source: amsterdam.nl.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds a new, longer public Article Page Storybook story to better evaluate vertical spacing with richer/long-form content (e.g., accordions, lists, multiple headings, and inline links), based on the referenced amsterdam.nl source.

Changes:

  • Added a new CzaarPeterstraat story with extended article content and structure.
  • Extended Design System component imports to support the new content (e.g., Accordion, Link, UnorderedList).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@github-actions github-actions bot temporarily deployed to demo-prose-vertical-space March 24, 2026 14:55 Destroyed
@github-actions github-actions bot temporarily deployed to demo-prose-vertical-space March 24, 2026 15:15 Destroyed
@VincentSmedinga VincentSmedinga changed the title poc: Add longer article page to help spacing design poc: Experiment with prose component to automate typographic whitespace Apr 10, 2026
@VincentSmedinga
Copy link
Copy Markdown
Contributor Author

Closing this branch – the article could be copied later, and ams-prose has a next iteration in #2455.

@VincentSmedinga VincentSmedinga deleted the poc/prose-vertical-space branch April 17, 2026 09:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants