-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
fix(Slug): updates Slug styles to match new floating-ui selectors #16039
fix(Slug): updates Slug styles to match new floating-ui selectors #16039
Conversation
✅ Deploy Preview for v11-carbon-react ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, just needs a snapshot update I think
@andreancardona just need a thumbs up here and it should be good to merge |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good!
c573f97
…rbon-design-system#16039) * fix(Slug): updates Slug styles to match new floating-ui selectors * chore: update snapshot --------- Co-authored-by: Andrea N. Cardona <[email protected]>
Closes #15935
Updates
Slug
so that only the new (for the most part) controls show up instorybook, and aligns the styles so that they display properly. I don't think there is a way to throw deprecate warnings for some items insidePropTypes.oneOf
, if there is let me know 👍Changelog
New
floating-ui
logical propertiesRemoved
.#{$prefix}--slug--enabled
which was eliminated with the removal of the filled-in slug in feat(Slug): implement phase 3 updates, adjust token values #15943Testing / Reviewing
Ensure all
align
orientations work in theunstable__slug --> callout
story