Skip to content

chore(ui): add some storybook entries - #3141

Merged
serhalp merged 7 commits into
npmx-dev:mainfrom
leekeh:feat/storybook-entries
Aug 9, 2026
Merged

chore(ui): add some storybook entries#3141
serhalp merged 7 commits into
npmx-dev:mainfrom
leekeh:feat/storybook-entries

Conversation

@leekeh

Copy link
Copy Markdown
Contributor

🔗 Linked issue

fixes some of #1841

🧭 Context

If desired, I can scaffold the stories for the remaining components, but I'd like to know if this is the direction you want to take.

I also think that it could be nice to organize the components a bit more, to store some core UI components separate from specific features. This will also help with storybook organization, as it should mirror the repo structure.

📚 Description

@vercel

vercelBot commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

ProjectDeploymentActionsUpdated (UTC)
docs.npmx.devReadyReadyPreviewAug 9, 2026 3:00pm
npmx.devReadyReadyPreviewAug 9, 2026 3:00pm
1 Skipped Deployment
ProjectDeploymentActionsUpdated (UTC)
npmx-lunariaIgnoredIgnoredAug 9, 2026 3:00pm

Request Review

@coderabbitai

coderabbitaiBot commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

📝 Walkthrough

Summary by CodeRabbit

  • Documentation
    • Added comprehensive Storybook examples and autodocs for interface components, including alerts, navigation, code viewing, loading states, pagination, progress, badges, cards, logos and skeletons.
    • Added scenarios covering default, empty, selected, loading, error, responsive and interactive states.
    • Improved Storybook theme switching to reflect the selected light or dark mode.

Walkthrough

Changes

Storybook now includes component stories, mock fixtures, decorators, controls, responsive examples, loading states, and autodocs metadata. The preview decorator applies the selected global theme class to the document root.

Storybook component coverage

Layer / File(s)Summary
Global theme-class handling
.storybook/preview.ts
The decorator maps the global theme value to light or dark, removes existing theme classes, and applies the selected class.
About and Brand component stories
app/components/About/*, app/components/Brand/Customize.stories.ts
Added stories for governance members, logos, logo lists, and logo customisation.
Code navigation stories
app/components/Code/DirectoryListing.stories.ts, app/components/Code/FileTree.stories.ts, app/components/Code/MobileTreeDrawer.stories.ts
Added mock trees, route data, responsive rendering, empty states, and selected-file states.
Code content and loading stories
app/components/Code/Header.stories.ts, app/components/Code/Viewer.stories.ts, app/components/Code/SkeletonLoader.stories.ts
Added file, directory, markdown, binary, selected-line, highlighted-code, and loading stories.
Component state stories and autodocs
app/components/Alert.stories.ts, app/components/BaseCard.stories.ts, app/components/CollapsibleSection.stories.ts, app/components/CopyToClipboardButton.stories.ts, app/components/Input/Base.stories.ts, app/components/Link/Link.stories.ts, app/components/LoadingSpinner.stories.ts, app/components/PaginationControls.stories.ts, app/components/ProgressBar.stories.ts, app/components/ProvenanceBadge.stories.ts, app/components/SkeletonBlock.stories.ts, app/components/SkeletonInline.stories.ts
Added component state stories, custom renders, controls, loading examples, and automatic documentation metadata. PaginationControls.stories.ts references watch without importing it.

Possibly related PRs

  • npmx-dev/npmx.dev#2545: Both changes modify Storybook theme and colour-mode handling in .storybook/preview.ts.

Suggested reviewers:ghostdevv

🚥 Pre-merge checks | ✅ 4
✅ Passed checks (4 passed)
Check nameStatusExplanation
Linked Issues check✅ PassedCheck skipped because no linked issues were found for this pull request.
Out of Scope Changes check✅ PassedCheck skipped because no linked issues were found for this pull request.
Title check✅ PassedThe title clearly identifies the main change: adding Storybook entries for the UI.
Description check✅ PassedThe description relates to adding Storybook stories and references the associated issue and planned follow-up work.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@leekeh

Copy link
Copy Markdown
ContributorAuthor

Question for reviewer: do we need to have code coverage for stories?

@coderabbitaicoderabbitaiBot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Actionable comments posted: 2

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In @.storybook/preview.ts:
- Around line 76-78: Update the theme application logic around themeClass so it
assigns the derived light/dark value to document.documentElement.dataset.theme,
matching the selectors in main.css. Remove the light/dark class removal and
addition unless another Storybook stylesheet explicitly requires those classes.
In `@app/components/PaginationControls.stories.ts`:
- Around line 16-33: Synchronize the local refs in the PaginationControls
story’s render setup with Storybook args after initialization. Add watchers for
args.mode, args.pageSize, and args.currentPage that update the corresponding
mode, pageSize, and currentPage refs when control-panel values change, while
preserving the existing v-model bindings.
🪄 Autofix

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: fc7b6132-12f5-4301-8e0d-8b7b782bc038

📥 Commits

Reviewing files that changed from the base of the PR and between ea245cc and 90c7fc0.

📒 Files selected for processing (23)
  • .storybook/preview.ts
  • app/components/About/GovernanceList.stories.ts
  • app/components/About/LogoImg.stories.ts
  • app/components/About/LogoList.stories.ts
  • app/components/Alert.stories.ts
  • app/components/BaseCard.stories.ts
  • app/components/Brand/Customize.stories.ts
  • app/components/Code/DirectoryListing.stories.ts
  • app/components/Code/FileTree.stories.ts
  • app/components/Code/Header.stories.ts
  • app/components/Code/MobileTreeDrawer.stories.ts
  • app/components/Code/SkeletonLoader.stories.ts
  • app/components/Code/Viewer.stories.ts
  • app/components/CollapsibleSection.stories.ts
  • app/components/CopyToClipboardButton.stories.ts
  • app/components/Input/Base.stories.ts
  • app/components/Link/Link.stories.ts
  • app/components/LoadingSpinner.stories.ts
  • app/components/PaginationControls.stories.ts
  • app/components/ProgressBar.stories.ts
  • app/components/ProvenanceBadge.stories.ts
  • app/components/SkeletonBlock.stories.ts
  • app/components/SkeletonInline.stories.ts

Comment thread.storybook/preview.ts
Comment threadapp/components/PaginationControls.stories.ts
leekehand others added 2 commits August 5, 2026 14:50
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
@serhalpserhalp assigned serhalp and unassigned serhalpAug 9, 2026
@serhalpserhalp added chore needs review This PR is waiting for a review from a maintainer labels Aug 9, 2026

@serhalpserhalp 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.

Amazing, thank you for contributing!

@serhalpserhalp changed the title feat: add some storybook entrieschore(ui): add some storybook entriesAug 9, 2026
@serhalp
serhalp enabled auto-merge August 9, 2026 15:01
@serhalp
serhalp added this pull request to the merge queueAug 9, 2026
Merged via the queue into npmx-dev:main with commit 8a1bec9Aug 9, 2026
24 of 25 checks passed
@github-actionsgithub-actionsBot mentioned this pull request Aug 9, 2026
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

choreneeds reviewThis PR is waiting for a review from a maintainer

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@leekeh@serhalp