event.target should match event.currentTarget at time of dispatch

[leeoniya]

https://github.com/developit/undom/blob/master/src/undom.js#L140

should be

let t = event.currentTarget = event.target = this,

[leeoniya]

err, no i think this is wrong too.

currentTarget (and this) should always point to the element that the listener is attached to while target is always the source of the event. the actual impl that passes my browser-tested tests is this one:

Element.prototype.dispatchEvent = function(event) {
	let t = event.target = this,
		c = event.cancelable,
		l, i;
	do {
		l = t.__handlers && t.__handlers[(event.type).toLowerCase()];
		if (l && (event.currentTarget = t)) for (i=l.length; i--; ) {
			if ((l[i].call(t, event)===false || event._end) && c) break;
		}
	} while (event.bubbles && !(c && event._stop) && (t=t.parentNode));
	return !event.defaultPrevented;
};

i think this is correct, but plz doublecheck.

https://stackoverflow.com/a/10086501/973988

[developit]

That is correct, yes.

[leeoniya]

FWIW, i had to make some other additions/tweaks to undom to get all my tests fully working (that already passed in browsers), though i cannot guarantee my impls are anything close to DOM spec compliant :D

feel free to cherry pick anything you find useful. value, textContent and createComment are fairly low weight and non-contrived for domvm's test cases.

https://github.com/leeoniya/domvm/blob/3.x-dev/test/init-spec.js

[developit]

Whoops, had those properties reversed!