[ButtonBase] Cancel ripple if touch was interrupted

[eps1lon]

Need deploy for experiment with #23808

Closes #23808

[mui-pr-bot]

Details of bundle changes

Generated by 🚫 dangerJS against c145c59

[oliviertassinari]

@eps1lon Do you need to keep this pull request open? From what I understand, the logic removed is required to trigger the ripple when the touchstart -> touchend session span for less than 80ms.

[eps1lon]

From what I understand, the logic removed it required to trigger the ripple when the touchstart -> touchend session span for less than 80ms.

Yes, this is what the code is doing. We have #23808 for why it is doing this.

Do you need to keep this pull request open?

Yes, see #23808

[oliviertassinari]

What's the next step?

[eps1lon]

What's the next step?

You haven't really addressed #23808 i.e. why do you think the current behavior is intended? Also waiting for a response to #23808 (comment) but that might be unlikely considerig you already closed the issue.

[oliviertassinari]

why do you think the current behavior is intended?

From what I understand with "intended", you are asking if we were aware of this issue when we merged #3407? I believe we were, yes.

Also waiting for a response to #23808 (comment)

Ok, let's wait :). I have reported one important downside with the approach to the issue: #23808 (comment).

you already closed the issue

If there is an alternative tradeoff that has more potential, it would be interesting to reopen. If not, I think that we might as well ignore it, there are many other issues with high ROI we can work on :).