-
Notifications
You must be signed in to change notification settings - Fork 70
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
[Feature Request] Vertical button group #659
Comments
@ruanyl thanks for your feature request - I think this proposal makes sense to iterate on (cc @BSFishy apologies - your initial tag "component iteration" was correct - I misunderstood and thought that Maps team was already aware of the Action Bar designs and was making a request for that. ) |
@ruanyl do you want to contribute this change? |
@KrooshalUX Sure, l'd love to 😃 I will get it planned and work on it |
This commit added a new prop(orientation) to the current OuiButtonGroup, the possible values are `horizontal` and `vertical`. The default value is `horizontal` which match the current UI behavior. With `vertical` prop, the button group will be oriented vertically. related issue: opensearch-project#659 Signed-off-by: Yulong Ruan <ruanyl@amazon.com>
@joshuarrrr I can take this issue if no one is working on it at the moment! |
Is your feature request related to a problem? Please describe.
Trying to add vertical button group similar to this, but couldn't find it's supported from the doc.
Describe the solution you'd like
I'd like to have
ButtonGroup
component to also supportvertical
alignment. I can see material-ui have something similar.Describe alternatives you've considered
We can use custom css styles but it would be nice if OUI can support this natively.
Additional context
dashboards-maps project needs to add a map control button group like this:
The PR: opensearch-project/dashboards-maps#372
cc @VijayanB @junqiu-lei
The text was updated successfully, but these errors were encountered: