Skip to content

docs(divider): Add Storybook story and docs page - #1208

Merged
frankieyan merged 1 commit into
mainfrom
divider-storybook-story
Sep 8, 2026
Merged

docs(divider): Add Storybook story and docs page#1208
frankieyan merged 1 commit into
mainfrom
divider-storybook-story

Conversation

@frankieyan

@frankieyan frankieyan commented Sep 7, 2026

Copy link
Copy Markdown
Member

Short description

This adds a story for the Divider component.

image image

PR Checklist

  • Updated docs (storybooks, readme)
  • Reviewed and approved Chromatic visual regression tests in CI

Co-Authored-By: Claude <noreply@anthropic.com>
@frankieyan
frankieyan force-pushed the divider-storybook-story branch from 3ca75c3 to 0198543 Compare September 7, 2026 22:28
@frankieyan
frankieyan marked this pull request as ready for review September 8, 2026 04:06
@frankieyan frankieyan added the 👀 Show PR Used for PRs that need a review, but can be merged when CI is green. label Sep 8, 2026

@doistbot doistbot left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This PR adds a Storybook story and docs page for the Divider, following the repo's existing CSF 3 and MDX conventions.

Few things worth tightening:

  • The docs describe the weights in increasing-contrast order (secondary, primary, tertiary), but the story renders them as primary, secondary, tertiary — reorder the array so the example matches the documented comparison.

I also left one optional follow-up note in the details below.

Optional follow-up note (1)
  • P3 src/divider/divider.mdx:24: This ordering claim doesn't hold across themes. In dark mode the divider tokens flip: per the values used in prose.stories.tsx (tertiary #323839 vs #202020 background), tertiary is the faintest and secondary is the middle one. Consider scoping the ordering to light mode or dropping the absolute claim.

Share FeedbackReview Logs

Comment thread src/divider/divider.stories.tsx
@frankieyan
frankieyan requested review from a team and engfragui and removed request for a team September 8, 2026 04:46
@frankieyan
frankieyan merged commit b5af404 into main Sep 8, 2026
13 checks passed
@frankieyan
frankieyan deleted the divider-storybook-story branch September 8, 2026 04:47
@doist-release-bot

Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 37.4.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@doist-release-bot doist-release-bot Bot added the Released PRs that have been merged and released label Sep 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Released PRs that have been merged and released 👀 Show PR Used for PRs that need a review, but can be merged when CI is green.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants