search

choojs / nanomorph

🚅 - Hyper fast diffing algorithm for real DOM nodes
MIT License
726 stars 58 forks source link

Event Failures #89

Open him2him2 opened 7 years ago

him2him2 commented 7 years ago

I created tests for the events listed in https://github.com/choojs/nanomorph/blob/master/lib/events.js#L32

They are tested in the as follows:

  1. test if the event is parsed from html
  2. test if events are removed in morph
  3. test of events are copied in morph

The following events fail to parse from HTML attribute. I believe this is a problem in Bel. But I am not sure where this problem originates.

The first two are listed as problematic and fail

Note: these event may not work as expected in Chrome, Safari and Opera 15+ using the JavaScript HTML DOM syntax. However, it should work as an HTML attribute and by using the addEventListener() method (See syntax examples below).

onfocusin https://www.w3schools.com/jsref/event_onfocusin.asp

onfocusout https://www.w3schools.com/jsref/event_onfocusout.asp

These events are listed as Editor's Draft(Non-stable version), but should work in chrome. They also fail.

ontouchcancel https://developer.mozilla.org/en-US/docs/Web/API/GlobalEventHandlers/ontouchcancel https://developer.mozilla.org/en-US/docs/Web/Events/touchcancel

ontouchend https://developer.mozilla.org/en-US/docs/Web/API/GlobalEventHandlers/ontouchend https://developer.mozilla.org/en-US/docs/Web/Events/touchend

ontouchmove https://developer.mozilla.org/en-US/docs/Web/API/GlobalEventHandlers/ontouchmove https://developer.mozilla.org/en-US/docs/Web/Events/touchmove

ontouchstart https://developer.mozilla.org/en-US/docs/Web/API/GlobalEventHandlers/ontouchstart https://developer.mozilla.org/en-US/docs/Web/Events/touchstart

This event should only work on the body tag, but also seems to fail.
onunload https://www.w3schools.com/jsref/event_onunload.asp Should only work on the body tag. Testing on the body tag fails.

I will submit a pull request that contains the event tests.