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
[Fab] Add disabled
class to FAB button
#34245
[Fab] Add disabled
class to FAB button
#34245
Conversation
disabled
class to FAB button
The |
e08d0fc
to
3782714
Compare
@michaldudak, I did the same thing as in Icon Button, but I fixed multiple occurrences of Mui-disabled class. I guess Icon Button has the same behavior, FYI |
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.
I've pushed changes which changes the logic to be similar to what we have in the Button
component. The test that you've added helped with ensuring that the logic is correct :)
Thanks for raising this up.
@mnajdova , thanks for this implementation, I tested this, looks good. can this be merged? |
@meenarama will merge it today, I've updated with the latest master, let's wait for the green CI :) |
@mnajdova , could you let me know which release, this will be in? |
It's released already @meenarama v5.10.8 |
@mnajdova , Yup, saw that, thx! |
Issue:
Fixes #34227
Similar Issue in the past:
[25072] (#25072)
What does it do?
Fixes disabled class not getting added to button element.
Why is it needed?
disabled class not working in Fab button