web-react-components icon indicating copy to clipboard operation
web-react-components copied to clipboard

Handler gets attached twice if name already exists in DOM

Open ChristophP opened this issue 7 years ago • 1 comments

If you have a react property called onClick and use it on the web-react-component the handler will fire twice. Once for the native DOM and once for the web react component.

Example (logs twice when clicked):

<some-button onClick="console.log('Hello')"></some-button>

This is not an issue when attaching the handler with addEventListener though, because the DOM event would be click while the web react component event would be onClick

ChristophP avatar Aug 09 '17 18:08 ChristophP

We should check the events .composed property to determine if it bubbles outside of the DOM. If not we should redispatch the event, if not just use the event which bubbles up into the light DOM.

ChristophP avatar Sep 09 '17 20:09 ChristophP