17626 2012-06-28 09:56:40 +0000 mousemove event is really cancelable? 2012-09-13 01:22:51 +0000 1 1 1 Unclassified WebAppsWG HISTORICAL - DOM3 Events unspecified All All RESOLVED WORKSFORME needsReview P2 normal --- 1 masayuki travil bugs glenn mike ojan travil www-dom public-webapps-bugzilla oldest_to_newest 69480 0 masayuki 2012-06-28 09:56:40 +0000 I'd like to confirm that is intentional change or a bug of the draft. In DOM2, mousemove event isn't cancelable, but the D3E spec draft defines it's cancelable. If that is intentional change, we will change the behavior on Gecko. https://bugzilla.mozilla.org/show_bug.cgi?id=704423 See https://bugzilla.mozilla.org/show_bug.cgi?id=704423#c1 for the compatibility with browsers. I think that it should be cancelable. See http://lists.w3.org/Archives/Public/www-dom/2011OctDec/0253.html for the reason. 69493 1 ojan 2012-06-28 15:29:30 +0000 In the mozilla bug, you listed that Opera and IE9 allow you to cancel this event, but still update hover-state. What effect does canceling the events have that is user/developer visible? 69495 2 masayuki 2012-06-28 16:04:36 +0000 (In reply to comment #1) > In the mozilla bug, you listed that Opera and IE9 allow you to cancel this > event, but still update hover-state. What effect does canceling the events have > that is user/developer visible? I tested by the defaultPrevented attribute. Additionally, IE9 doesn't change selected item by mousemove event after mousedown in listbox styled <select> element if mousemove element is consumed. 69496 3 1147 masayuki 2012-06-28 16:11:35 +0000 Created attachment 1147 testcase for the default action if preventDefault() is called 69497 4 1148 masayuki 2012-06-28 16:14:08 +0000 Created attachment 1148 testcase for checking defaultPrevented attribute value of mousemove event 69499 5 ojan 2012-06-28 17:30:08 +0000 I don't see a point in making it cancellable if cancelling it doesn't do anything. Maybe someone from IE/Opera could comment on whether it accomplishes anything else. 69972 6 travil 2012-07-13 21:53:26 +0000 Historically the spec had mousemove as not cancelable, but sometime between 2002 (http://www.w3.org/TR/2002/WD-DOM-Level-3-Events-20020712/) and 2003 (http://www.w3.org/TR/2003/WD-DOM-Level-3-Events-20030221/events.html#Events-EventTypes-complete) this was switched to cancelable and has apparently been that way ever since. The term "mousemove" doesn't come up anywhere in the www-dom archives during that period either. So, the trail has gone cold trying to understand why the spec changed back then. In checking IE's behavior, we explicitly ignore the canceled state for hover, as has been observed. It does not appear to have other impact that we could readily see, though we didn't follow all code paths. I see the point of making every event that can be cancelable have some rational behavior when it is canceled, but then again, the spec already points out that this logic may not always hold true: file:///D:/DOM3Events/D3E/html/DOM3-Events.html#event-flow-default-cancel At this point, I'd rather not tweak the value in the spec unless there's a more substantive reason to change it. 73671 7 masayuki 2012-09-12 04:15:37 +0000 Excuse me, I don't understand what "WONTFIX" means for this bug. So, starting D3E, should the mousemove event be cancelable? I mean, should defaultPrevented be able to become true if preventDefault() is called? 73699 8 glenn 2012-09-12 15:21:31 +0000 Note that an event being cancellable is mostly legacy; I'm not aware of any real purpose to this flag today. It dates from when the "default behavior" of an event was thought of as a part of event dispatch, but with a couple isolated exceptions (onclick) that's no longer the case; the whole concept of default behavior is a misnomer. It's saner to err towards cancellable, for simplicity, to always make newly-minted events cancellable, and to only maintain existing events as not cancellable for compatibility. 73708 9 travil 2012-09-12 18:00:36 +0000 (In reply to comment #0) > I'd like to confirm that is intentional change or a bug of the draft. > > In DOM2, mousemove event isn't cancelable, but the D3E spec draft defines it's > cancelable. > > If that is intentional change, we will change the behavior on Gecko. > https://bugzilla.mozilla.org/show_bug.cgi?id=704423 > > See https://bugzilla.mozilla.org/show_bug.cgi?id=704423#c1 for the > compatibility with browsers. > > I think that it should be cancelable. See > http://lists.w3.org/Archives/Public/www-dom/2011OctDec/0253.html for the > reason. (In reply to comment #7) > Excuse me, I don't understand what "WONTFIX" means for this bug. > > So, starting D3E, should the mousemove event be cancelable? I mean, should > defaultPrevented be able to become true if preventDefault() is called? To be clear--yes, mousemove should be cancelable and defaultPrevented should become true when preventDefault() is called on this event. There is no other tangible side-effect to cancelling this event that implementers must support. 73736 10 masayuki 2012-09-13 01:22:51 +0000 (In reply to comment #9) > To be clear--yes, mousemove should be cancelable and defaultPrevented should > become true when preventDefault() is called on this event. > > There is no other tangible side-effect to cancelling this event that > implementers must support. Thanks, it's very clear for me! 1147 2012-06-28 16:11:35 +0000 2012-06-28 16:11:35 +0000 testcase for the default action if preventDefault() is called doPreventDefault.html text/html 2866 masayuki PCFET0NUWVBFIGh0bWw+CjxkaXY+CjxsYWJlbD5zZWxlY3QgZm9sbG93aW5nIGV2ZW50IG5hbWVz IHdoaWNoIHlvdSB3YW50IHRvIGNhbGwgPGNvZGU+cHJldmVudERlZmF1bHQoKTwvY29kZT4gb2Yu PGJyPgo8c2VsZWN0IG11bHRpcGxlIGlkPSJwcmV2ZW50RGVmYXVsdExpc3QiIHNpemU9IjgiIG9u Y2hhbmdlPSJkb1ByZXZlbnREZWZhdWx0KCk7Ij4KICA8b3B0aW9uPmtleWRvd248L29wdGlvbj4K ICA8b3B0aW9uPmtleXByZXNzPC9vcHRpb24+CiAgPG9wdGlvbj5rZXl1cDwvb3B0aW9uPgogIDxv cHRpb24+bW91c2Vkb3duPC9vcHRpb24+CiAgPG9wdGlvbj5tb3VzZW1vdmU8L29wdGlvbj4KICA8 b3B0aW9uPm1vdXNldXA8L29wdGlvbj4KICA8b3B0aW9uPmNsaWNrPC9vcHRpb24+CiAgPG9wdGlv bj5kYmxjbGljazwvb3B0aW9uPgo8L3NlbGVjdD48L2xhYmVsPgo8ZGl2IGlkPSJkb1ByZXZlbnRE ZWZhdWx0Ij4KPHNjcmlwdCB0eXBlPSJ0ZXh0L2phdmFzY3JpcHQiPgpmdW5jdGlvbiBwcmV2ZW50 aW5nRGVmYXVsdEhhbmRsZXIoZSkKewogIGUucHJldmVudERlZmF1bHQoKTsKfQoKZnVuY3Rpb24g ZG9QcmV2ZW50RGVmYXVsdCgpCnsKICB2YXIgbGlzdCA9IGRvY3VtZW50LmdldEVsZW1lbnRCeUlk KCJwcmV2ZW50RGVmYXVsdExpc3QiKTsKICB2YXIgZGl2ID0gZG9jdW1lbnQuZ2V0RWxlbWVudEJ5 SWQoImRvUHJldmVudERlZmF1bHQiKTsKICBmb3IgKHZhciBpID0gMDsgaSA8IGxpc3Qub3B0aW9u cy5sZW5ndGg7IGkrKykgewogICAgdmFyIG9wdGlvbiA9IHByZXZlbnREZWZhdWx0TGlzdC5vcHRp b25zW2ldOwogICAgaWYgKG9wdGlvbi5zZWxlY3RlZCkgewogICAgICBkaXYuYWRkRXZlbnRMaXN0 ZW5lcihvcHRpb24udmFsdWUsIHByZXZlbnRpbmdEZWZhdWx0SGFuZGxlciwgdHJ1ZSk7CiAgICB9 IGVsc2UgewogICAgICBkaXYucmVtb3ZlRXZlbnRMaXN0ZW5lcihvcHRpb24udmFsdWUsIHByZXZl bnRpbmdEZWZhdWx0SGFuZGxlciwgdHJ1ZSk7CiAgICB9CiAgfQp9Cgpkb1ByZXZlbnREZWZhdWx0 KCk7Cjwvc2NyaXB0Pgo8dWw+CjxsaT5ub3JtYWwgdGV4dDwvbGk+CjxsaT48YSBocmVmPSJodHRw Oi8vd3d3LmQtdG95Ym94LmNvbS9zdHVkaW8vIj5hbmNob3IgdGV4dDwvYT48L2xpPgo8bGk+PGRp diBzdHlsZT0id2lkdGg6IDEwMHB4OyBoZWlnaHQ6IDEwMHB4OyBib3JkZXI6IDFweCBzb2xpZCBi bHVlOyBvdmVyZmxvdzogYXV0bzsiPjxkaXYgc3R5bGU9IndpZHRoOiA1MDBweDsgaGVpZ2h0OiA1 MDBweDsiPnNjcm9sbGJhcnM8L2Rpdj48L2Rpdj48L2xpPgo8bGk+PGxhYmVsPnRleHQ6IDxpbnB1 dCB0eXBlPSJ0ZXh0Ii8+PC9sYWJlbD48L2xpPgo8bGk+PGxhYmVsPnBhc3N3b3JkOiA8aW5wdXQg dHlwZT0icGFzc3dvcmQiLz48L2xhYmVsPjwvbGk+CjxsaT48bGFiZWw+ZmlsZTogPGlucHV0IHR5 cGU9ImZpbGUiLz48L2xhYmVsPjwvbGk+CjxsaT48bGFiZWw+PGlucHV0IHR5cGU9InJhZGlvIiBu YW1lPSJ0ZXN0MSIvPiByYWRpbzE8L2xhYmVsPjxsYWJlbD48aW5wdXQgdHlwZT0icmFkaW8iIG5h bWU9InRlc3QxIi8+IHJhZGlvMjwvbGFiZWw+PC9saT4KPGxpPjxsYWJlbD48aW5wdXQgdHlwZT0i Y2hlY2tib3giLz4gY2hlY2tib3g8L2xhYmVsPjwvbGk+CjxsaT48bGFiZWw+YnV0dG9uOiA8aW5w dXQgdHlwZT0iYnV0dG9uIi8+PC9sYWJlbD48L2xpPgo8bGk+PGxhYmVsPmRhdGU6IDxpbnB1dCB0 eXBlPSJkYXRlIi8+PC9sYWJlbD48L2xpPgo8bGk+PGxhYmVsPmRhdGV0aW1lOiA8aW5wdXQgdHlw ZT0iZGF0ZXRpbWUiLz48L2xhYmVsPjwvbGk+CjxsaT48bGFiZWw+ZGF0ZXRpbWUtbG9jYWw6IDxp bnB1dCB0eXBlPSJkYXRldGltZS1sb2NhbCIvPjwvbGFiZWw+PC9saT4KPGxpPjxsYWJlbD5tb250 aDogPGlucHV0IHR5cGU9Im1vbnRoIi8+PC9sYWJlbD48L2xpPgo8bGk+PGxhYmVsPndlZWs6IDxp bnB1dCB0eXBlPSJ3ZWVrIi8+PC9sYWJlbD48L2xpPgo8bGk+PGxhYmVsPnRpbWU6IDxpbnB1dCB0 eXBlPSJ0aW1lIi8+PC9sYWJlbD48L2xpPgo8bGk+PGxhYmVsPm51bWJlcjogPGlucHV0IHR5cGU9 Im51bWJlciIvPjwvbGFiZWw+PC9saT4KPGxpPjxsYWJlbD5yYW5nZTogPGlucHV0IHR5cGU9InJh bmdlIi8+PC9sYWJlbD48L2xpPgo8bGk+PGxhYmVsPnNlYXJjaDogPGlucHV0IHR5cGU9InNlYXJj aCIvPjwvbGFiZWw+PC9saT4KPGxpPjxsYWJlbD50ZWw6IDxpbnB1dCB0eXBlPSJ0ZWwiLz48L2xh YmVsPjwvbGk+CjxsaT48bGFiZWw+dXJsOiA8aW5wdXQgdHlwZT0idXJsIi8+PC9sYWJlbD48L2xp Pgo8bGk+PGxhYmVsPmVtYWlsOiA8aW5wdXQgdHlwZT0iZW1haWwiLz48L2xhYmVsPjwvbGk+Cjxs aT48bGFiZWw+Y29sb3I6IDxpbnB1dCB0eXBlPSJjb2xvciIvPjwvbGFiZWw+PC9saT4KPGxpPjxs YWJlbD50ZXh0YXJlYTogPHRleHRhcmVhPjwvdGV4dGFyZWE+PC9sYWJlbD48L2xpPgo8bGk+PGxh YmVsPmNvbWJvYm94OiA8c2VsZWN0PjxvcHRpb24+Zmlyc3QgaXRlbTwvb3B0aW9uPjxvcHRpb24+ c2Vjb25kIGl0ZW08L29wdGlvbj48L3NlbGVjdD48L2xhYmVsPjwvbGk+CjxsaT48bGFiZWw+bGlz dGJveDogPHNlbGVjdCBzaXplPSI0Ij48b3B0aW9uPmZpcnN0IGl0ZW08L29wdGlvbj48b3B0aW9u PnNlY29uZCBpdGVtPC9vcHRpb24+PG9wdGlvbj50aGlyZCBpdGVtPC9vcHRpb24+PC9zZWxlY3Q+ PC9sYWJlbD48L2xpPgo8bGk+PGxhYmVsPmxpc3Rib3g6IDxzZWxlY3QgbXVsdGlwbGU9Im11bHRp cGxlIj48b3B0aW9uPmZpcnN0IGl0ZW08L29wdGlvbj48b3B0aW9uPnNlY29uZCBpdGVtPC9vcHRp b24+PG9wdGlvbj50aGlyZCBpdGVtPC9vcHRpb24+PC9zZWxlY3Q+PC9sYWJlbD48L2xpPgo8L3Vs Pgo8L2Rpdj4KPC9kaXY+Cg== 1148 2012-06-28 16:14:08 +0000 2012-06-28 16:14:08 +0000 testcase for checking defaultPrevented attribute value of mousemove event mousemove.html text/html 495 masayuki PCFET0NUWVBFIGh0bWw+CjxwPkhlcmUgaXMgdGV4dDwvcD4KPHAgaWQ9InN0YXR1cyI+PC9wPgo8 c2NyaXB0PgogIHZhciB0aW1lciA9IDA7CiAgd2luZG93LmFkZEV2ZW50TGlzdGVuZXIoIm1vdXNl bW92ZSIsCiAgICBmdW5jdGlvbiAoZSkgewogICAgICBlLnByZXZlbnREZWZhdWx0KCk7CiAgICAg IGRvY3VtZW50LmdldEVsZW1lbnRCeUlkKCJzdGF0dXMiKS5pbm5lckhUTUwgPQogICAgICAgIGUu ZGVmYXVsdFByZXZlbnRlZCA/ICJzdWNjZWVkZWQgdG8gcHJldmVudERlZmF1bHQoKSIgOgogICAg ICAgICAgICAgICAgICAgICAgICAgICAgICJmYWlsZWQgdG8gcHJldmVudGVkRGVmYWx1bHQoKSI7 CiAgICAgIGNsZWFyVGltZW91dCh0aW1lcik7CiAgICAgIHRpbWVyID0gc2V0VGltZW91dChmdW5j dGlvbiAoKSB7CiAgICAgICAgZG9jdW1lbnQuZ2V0RWxlbWVudEJ5SWQoInN0YXR1cyIpLmlubmVy SFRNTCA9ICIiOyB9LCA1MDApOyB9LCB0cnVlKTsKPC9zY3JpcHQ+