Skip to content

fix(Address): Disable icon control in Storybook#42

Merged
garrity-miepub merged 1 commit intomainfrom
feature/address-update
Jan 30, 2026
Merged

fix(Address): Disable icon control in Storybook#42
garrity-miepub merged 1 commit intomainfrom
feature/address-update

Conversation

@garrity-miepub
Copy link
Copy Markdown
Contributor

ReactNode props cannot be controlled via Storybook UI controls. Setting control: false removes the broken icon control from the docs page controls panel.

ReactNode props cannot be controlled via Storybook UI controls.
Setting control: false removes the broken icon control from the
docs page controls panel.
Copilot AI review requested due to automatic review settings January 30, 2026 23:04
@garrity-miepub garrity-miepub marked this pull request as ready for review January 30, 2026 23:04
@cloudflare-workers-and-pages
Copy link
Copy Markdown

Deploying ui with  Cloudflare Pages  Cloudflare Pages

Latest commit: 0f80dee
Status: ✅  Deploy successful!
Preview URL: https://7aafa707.ui-6d0.pages.dev
Branch Preview URL: https://feature-address-update.ui-6d0.pages.dev

View logs

Copy link
Copy Markdown

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

Disables the broken Storybook UI control for the icon prop on the Address component, since ReactNode props aren’t meaningfully controllable via standard Storybook controls.

Changes:

  • Added an argTypes override to disable the icon control in the Address Storybook docs/controls panel.

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

@garrity-miepub garrity-miepub merged commit 737cce5 into main Jan 30, 2026
16 checks passed
@garrity-miepub garrity-miepub deleted the feature/address-update branch January 30, 2026 23:09
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.

2 participants