Skip to content

refactor(Progress): overhaul stories to fix Storybook controls#24

Merged
garrity-miepub merged 1 commit into
mainfrom
feature/progress-update
Jan 28, 2026
Merged

refactor(Progress): overhaul stories to fix Storybook controls#24
garrity-miepub merged 1 commit into
mainfrom
feature/progress-update

Conversation

@garrity-miepub

Copy link
Copy Markdown
Collaborator
  • Replace render functions with args-connected pattern for all main stories
  • Add proper argTypes with descriptions for all props:
    • value: range slider (0-100)
    • size: select (sm, md, lg, xl)
    • variant: select (default, success, warning, danger)
    • showValue, striped, animated, indeterminate: boolean toggles
  • Add individual stories for each size and variant combination
  • Add CircularProgress stories with own argTypes and render functions
  • Add decorator for consistent width wrapper
  • Disable controls on showcase/example stories (AllSizes, AllVariants, etc.)
  • Keep AnimatedProgressDemo as stateful component for live demo
progress-update.mov

- Replace render functions with args-connected pattern for all main stories
- Add proper argTypes with descriptions for all props:
  - value: range slider (0-100)
  - size: select (sm, md, lg, xl)
  - variant: select (default, success, warning, danger)
  - showValue, striped, animated, indeterminate: boolean toggles
- Add individual stories for each size and variant combination
- Add CircularProgress stories with own argTypes and render functions
- Add decorator for consistent width wrapper
- Disable controls on showcase/example stories (AllSizes, AllVariants, etc.)
- Keep AnimatedProgressDemo as stateful component for live demo
@garrity-miepub garrity-miepub marked this pull request as ready for review January 28, 2026 23:31
@garrity-miepub garrity-miepub merged commit 3bd987a into main Jan 28, 2026
2 of 7 checks passed
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