It appears that on an element having more than one event handlers (on* attributes), th

According to this <a href="https://codepen.io/stephanebisson/pen/yLrWgzL" rel="nofollo

According to this <a href="https://codepen.io/stephanebisson/pen/yLrWgzL"

Second event handler not correctly registered about reef HOT 4 CLOSED

stephanebisson commented on May 28, 2024

Second event handler not correctly registered

from reef.

Comments (4)

hueitan commented on May 28, 2024

The issue causing the second item to be skipped is that the previous attributes are being removed while the loop is still running. To fix this, we should wait until after the loop has finished to remove the attributes.

reef/src/components/render.js

Lines 187 to 194 in ca459f0

 for (let {name, value} of elem.attributes) { 

 // If the attribute should be skipped, remove it 

 if (skipAttribute(name, value)) { 

 removeAttribute(elem, name); 

 listen(elem, name, value, events); 

 continue; 

 }

from reef.

cferdinandi commented on May 28, 2024

Fixed!

from reef.

stephanebisson commented on May 28, 2024

According to this codepen, v13.0.4 broke this again

from reef.

hueitan commented on May 28, 2024

According to this codepen, v13.0.4 broke this again

fixed on #204

from reef.

Related Issues (20)

License? HOT 2
Add support for on* events that does so safely HOT 2
12.3 has broken my app HOT 2
support disabled and readonly attributes with @ syntax? HOT 5
jsdoc issue HOT 2
Listeners remove html attributes HOT 4
Event functions with parameters HOT 3
Rename store(), setters() to align with industry norms (breaking change) HOT 1
Add a way to ignore certain elements when diffing the DOM HOT 1
How to read a signal's data built via a store? HOT 5
Add getter methods to store() object
tbody element not supported as a component root HOT 3
Add option to keep attributes on nodes with [reef-keep] HOT 9
onclick attribute removed by reef HOT 3
Reef types HOT 2
to do demo bug
Component composition HOT 2
13.0.3 has broken attributes on html HOT 2
Is the store function doesn't work with component? HOT 1

Second event handler not correctly registered about reef HOT 4 CLOSED

Comments (4)

Related Issues (20)

Recommend Projects

React

Vue.js

Typescript

TensorFlow

Django

Laravel

D3

Recommend Topics

javascript

web

server

Machine learning

Visualization

Game

Recommend Org

Facebook

Microsoft

Google

Alibaba

D3

Tencent

	for (let {name, value} of elem.attributes) {

	// If the attribute should be skipped, remove it
	if (skipAttribute(name, value)) {
	removeAttribute(elem, name);
	listen(elem, name, value, events);
	continue;
	}