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(overflow-menu): add storybook controls #12474

Merged
merged 3 commits into from
Nov 10, 2022

Conversation

aledavila
Copy link
Contributor

Closes #12226

Testing / Reviewing

Make sure stories are working as intended
Make sure controls work as intended

@aledavila aledavila requested a review from a team as a code owner November 3, 2022 06:20
@netlify
Copy link

netlify bot commented Nov 3, 2022

Deploy Preview for carbon-elements ready!

Name Link
🔨 Latest commit d3a3c28
🔍 Latest deploy log https://app.netlify.com/sites/carbon-elements/deploys/636d47325e409e0009df7e0a
😎 Deploy Preview https://deploy-preview-12474--carbon-elements.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@netlify
Copy link

netlify bot commented Nov 3, 2022

Deploy Preview for carbon-components-react ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit d3a3c28
🔍 Latest deploy log https://app.netlify.com/sites/carbon-components-react/deploys/636d4732de3ea80008f2a84c
😎 Deploy Preview https://deploy-preview-12474--carbon-components-react.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@alisonjoseph
Copy link
Member

The flipped boolean doesn't seem to be working for me in the playground. Also not sure why tests are failing?

@aledavila
Copy link
Contributor Author

@alisonjoseph thanks will look into it 🤔

@tw15egan
Copy link
Collaborator

tw15egan commented Nov 8, 2022

@aledavila pushed up a change, looks like it was just that the snapshots needed to be updated because you've removed the v11 conditional flag in the export.

@alisonjoseph
Copy link
Member

What is the flipped prop supposed to do? Appears to not do anything, but I could be missing something.

@kodiakhq kodiakhq bot merged commit e389729 into carbon-design-system:main Nov 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Storybook controls: components/OverflowMenu/OverflowMenu-story.js
3 participants