-
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
feat: add fluid button set #15843
base: main
Are you sure you want to change the base?
feat: add fluid button set #15843
Conversation
✅ Deploy Preview for v11-carbon-react ready!Built without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for v11-carbon-react ready!Built without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify site configuration. |
This is great, @lee-chase. Thanks for taking the time to add this in. I had a question: Should the single button max out at |
Not sure I have an answer to that. The sizes and behaviour chosen were very much a reflection of the ActionSet used in Given that custom CSS properties cannot be used in media queries (container or otherwise) perhaps these value could be defined as SASS variables or part of a mixin call to allow a consumer could override should they need to. |
Hi, sorry for the late review. This looks great. I have a few questions and concerns:
|
@thyhmdo apologies for the lack of clarity.
The wrapper around the button story is something done in IBM Products and could probably do with a UX tidy up. The intent is to show the component but provide a simple mechanism to allow the user to see how the component behaves with different container sizes. |
|
Hey @lee-chase |
that looks good! thanks @guidari |
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.
Tested and that looks good to me. 🚀
The documentation on buttons https://carbondesignsystem.com/components/button/usage/
mentions fluid buttons and how they're handled. However, this does not appear to be implemented.
This PR adds the
fluid
property to theButtonSet
component.With the
fluid
property set to true the button set exhibits the following behavior$min-inline-button-size: convert.to-rem(176px)
withDisplayBox
to allow control of container width in storybookSetOfButonsFluid
storyFurther work
Changelog
New
Changed
Testing / Reviewing
Checked functional in storybook