material-ui icon indicating copy to clipboard operation
material-ui copied to clipboard

Feature Request: extend trigger of TouchRipple to FormControlLabel

Open ghost opened this issue 3 years ago • 1 comments

Duplicates

  • [X] I have searched the existing issues

Latest version

  • [X] I have tested the latest version

Summary 💡

When using a component built upon MuiButtonBase, for example, CheckBox, the TouchRipple animation triggers when user clicks the component. Very good.

When such a component is wrapped with FormControlLabel to provide an adjacent label, currently the TouchRipple animation is unaffected by the label.

When label exists, would request the ability to extend the trigger for the TouchRipple animation to include the label. Of course the animation itself would not extend to the label, just the trigger.

Result: Touch the label, see the checkbox change state with a ripple.

ghost avatar Aug 19 '22 19:08 ghost

I think it's a sensible request. However, unless there's a significant number of upvotes from the community, we won't have the bandwidth to implement this. Feel free to open a PR if you'd like to give it a shot.

michaldudak avatar Aug 26 '22 10:08 michaldudak