Skip to content

Remove play button from sidebar - #332

Open
yannbf wants to merge 11 commits into
mainfrom
upgrade-storybook-v10
Open

Remove play button from sidebar#332
yannbf wants to merge 11 commits into
mainfrom
upgrade-storybook-v10

Conversation

@yannbf

@yannbf yannbf commented Oct 13, 2025

Copy link
Copy Markdown
Member

This PR removes the "play" button from the animated layout used in frontpage, and also upgrades from Storybook 8 -> 10. Additionally, sets Storybook to use the nextjs-vite framework.

Before:
image

After:
image

@yannbf
yannbf requested a review from kylegach October 13, 2025 14:34
@netlify

netlify Bot commented Oct 13, 2025

Copy link
Copy Markdown

Deploy Preview for storybook-addon-catalog failed. Why did it fail? →

Name Link
🔨 Latest commit 9380521
🔍 Latest deploy log https://app.netlify.com/projects/storybook-addon-catalog/deploys/69cfabdb9734b20008979775

@netlify

netlify Bot commented Oct 13, 2025

Copy link
Copy Markdown

Deploy Preview for storybook-frontpage failed. Why did it fail? →

Name Link
🔨 Latest commit 9380521
🔍 Latest deploy log https://app.netlify.com/projects/storybook-frontpage/deploys/69cfabdbd8e7510008a98640

@kylegach
kylegach force-pushed the upgrade-storybook-v10 branch from 0c37f34 to b2a1d4e Compare October 17, 2025 14:44

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

LGTM. We can now update deps to 10.0, I've set suggestions to make it easier.

Comment thread apps/frontpage/package.json Outdated
Comment on lines +74 to +75
"@storybook/addon-links": "^10.0.0-beta.10",
"@storybook/nextjs-vite": "^10.0.0-beta.10",

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.

Suggested change
"@storybook/addon-links": "^10.0.0-beta.10",
"@storybook/nextjs-vite": "^10.0.0-beta.10",
"@storybook/addon-links": "^10.0.0",
"@storybook/nextjs-vite": "^10.0.0",

Comment thread apps/frontpage/package.json Outdated
Comment thread apps/frontpage/package.json Outdated
Comment thread apps/frontpage/package.json Outdated
Comment thread packages/ui/package.json Outdated
Comment thread packages/ui/package.json Outdated
Comment thread packages/ui/package.json Outdated
Comment thread packages/ui/package.json Outdated
@Sidnioulz Sidnioulz self-assigned this Nov 14, 2025
@Sidnioulz
Sidnioulz force-pushed the upgrade-storybook-v10 branch from de3f769 to 9380521 Compare April 3, 2026 12:00
@chromatic-com

chromatic-com Bot commented Apr 3, 2026

Copy link
Copy Markdown

Important

UI Tests need review – Review now

🟡 UI Tests: web/frontpage: 9 changes must be accepted as baselines
🟡 UI Review: web/frontpage: Go review the new and updated UI
Storybook icon Storybook Publish: web/frontpage: 37 stories published

@chromatic-com

chromatic-com Bot commented Apr 3, 2026

Copy link
Copy Markdown

Tip

All tests passed and all changes approved!

🟢 UI Tests: web/ui: 11 tests unchanged
🟢 UI Review: web/ui: 11 stories published -- no changes
Storybook icon Storybook Publish: web/ui: 11 stories published

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.

3 participants