pointer-events: none; on icons within buttons?

[trailsnail]

Describe the bug

Hi,
(slotted) Icons within sl-button or sl-icon-button should not catch the click on the button itself.
It is a little bit confusing, if the cursor points on the icon while you try to click a submit button.

Demo: https://codepen.io/TrailSnail/pen/WNZOGwm

1. Click the button within its default area: form will be submitted (see console)
2. Click the button with cursor on the icon - nothing happens

Button	Icon

So the question: would it be correct, to add a pointer-events: none; to the icons by default?
At the moment we do theses workaround manually in our projects.
Greetings 😉

[claviska]

That's probably the easiest workaround. Perhaps a better fix would be reworking the event listener so it accounts for the click event bubbling up from icons and anything else that might get slotted in.

Thanks for reporting this!

[claviska]

This was fixed in dd65778.