Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(content-switcher): update story to Storybook v7 #11315

Conversation

m4olivei
Copy link
Contributor

@m4olivei m4olivei commented Jan 3, 2024

Related Ticket(s)

Closes #11313

Description

Adjustments to content-switcher component to be compatible with Storybook v7.

Changelog

Changed

  • Updated content-switcher Storybook component for Storybook v7
  • Slight adjustment to imports to ensure content-switcher has all of its dependencies imported
  • Fix an issue with a CSS selector

@m4olivei m4olivei requested a review from a team as a code owner January 3, 2024 22:01
@m4olivei m4olivei requested review from emyarod and IgnacioBecerra and removed request for a team January 3, 2024 22:01
@ibmdotcom-bot
Copy link
Contributor

ibmdotcom-bot commented Jan 3, 2024

Copy link

@RichKummer RichKummer left a comment

Choose a reason for hiding this comment

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

Hey @m4olivei

Looks good, but just found one issue. When looking at the with Layer version with a smaller width screen, the content switchers break out of the layering set up. In React, the content switchers more naturally condense in size.

React version:
Screenshot 2024-01-08 at 2 05 53 PM

This build:
Screenshot 2024-01-08 at 2 05 56 PM

@m4olivei m4olivei force-pushed the chore/content-switcher-sb-v7 branch from 9491e17 to c5146b7 Compare January 10, 2024 20:20
Copy link
Member

@kennylam kennylam left a comment

Choose a reason for hiding this comment

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

LGTM!

@kennylam kennylam merged commit 2e29308 into carbon-design-system:feat/cwc-storybook-7-vite Jan 10, 2024
5 of 12 checks passed
kennylam added a commit to kennylam/carbon-for-ibm-dotcom that referenced this pull request Jun 11, 2024
…ystem#11315)

* chore(content-switcher): update content-switcher to sb v7

* fix(content-switcher): add imports for content-switcher dependencies

* fix(content-switcher): fix css selector for icon only styles

* fix(content-switcher): restore the size options

* chore(content-switcher): include storybook layers components

---------

Co-authored-by: kennylam <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants