-
Notifications
You must be signed in to change notification settings - Fork 86
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
MultiActionButton support element type on text prop #6720
Comments
@harpalsingh This doesn't appear to be something actively supported in the DS. Do you think it is something we should support? @StylianosKoidakisSage Would it be possible to message Harpal and me on Slack with your use case for this? I assume it has come from a UX Designer who wishes to include an icon as part of the button? |
@nineteen88 taking a look at this and will speak with @ljemmo as well |
Hi @ljemmo @harpalsingh, were you both able to reach a conclusion on this? :) |
@Parsium on my end, i would expect the main trigger on the multi action button to turn into a loading state when required. is this what is proposed? Currently can't see a visual so its hard to know what exactly is requested |
Yep @ljemmo is a loading state in the main button that's requested here, I've asked that teams requesting speak to you and potentially generate some designs here |
FE-6770 |
Description
Same as Button component we should be able to pass an element to the text prop on the MultiActionButton, like so we can place some loaders or icons there
Suggested solution
update the types of MultiActionButton
Demo URL
No response
Alternatives
No response
Additional context
No response
Confidentiality
The text was updated successfully, but these errors were encountered: