DEV: stop propagation of events on button click (PR #13993)

GitHub

This is probably fine in most cases but I do wonder if some code could be relying on this. For example, perhaps the propagation was used to close a dropdown that was open? We should be a little careful when merging.

This is probably fine in most cases but I do wonder if some code could be relying on this. For example, perhaps the propagation was used to close a dropdown that was open? We should be a little careful when merging.

Agreed it could cause changes, in my mind this would be very brittle behavior anyways and we shouldn’t rely on this if we do this somewhere.