Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(87)

Issue 13465021: Flipping the direction of WheelEvent.deltaX/Y to follow standards (Closed)

Created:
7 years, 8 months ago by blois
Modified:
7 years, 8 months ago
CC:
reviews_dartlang.org
Visibility:
Public.

Description

Flipping the direction of WheelEvent.deltaX/Y to follow standards We're trying to follow the W3C spec for WheelEvent, but the standard has the direction in the opposite direction from what we have it now. BUG=6215 Committed: https://code.google.com/p/dart/source/detail?r=20879

Patch Set 1 : #

Total comments: 4

Patch Set 2 : #

Total comments: 2

Patch Set 3 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+96 lines, -16 lines) Patch
M sdk/lib/html/dart2js/html_dart2js.dart View 7 chunks +29 lines, -6 lines 0 comments Download
M sdk/lib/html/dartium/html_dartium.dart View 1 chunk +19 lines, -2 lines 0 comments Download
M tools/dom/templates/html/impl/impl_WheelEvent.darttemplate View 1 2 8 chunks +48 lines, -8 lines 0 comments Download

Messages

Total messages: 7 (0 generated)
blois
7 years, 8 months ago (2013-04-02 23:17:16 UTC) #1
Emily Fortuna
lgtm with comment addition https://codereview.chromium.org/13465021/diff/5001/tools/dom/templates/html/impl/impl_WheelEvent.darttemplate File tools/dom/templates/html/impl/impl_WheelEvent.darttemplate (right): https://codereview.chromium.org/13465021/diff/5001/tools/dom/templates/html/impl/impl_WheelEvent.darttemplate#newcode25 tools/dom/templates/html/impl/impl_WheelEvent.darttemplate:25: // If polyfilling, then flip ...
7 years, 8 months ago (2013-04-02 23:59:08 UTC) #2
Andrei Mouravski
https://codereview.chromium.org/13465021/diff/5001/tools/dom/templates/html/impl/impl_WheelEvent.darttemplate File tools/dom/templates/html/impl/impl_WheelEvent.darttemplate (right): https://codereview.chromium.org/13465021/diff/5001/tools/dom/templates/html/impl/impl_WheelEvent.darttemplate#newcode8 tools/dom/templates/html/impl/impl_WheelEvent.darttemplate:8: Could you add some minimal dartdoc documentation?
7 years, 8 months ago (2013-04-03 00:10:31 UTC) #3
blois
https://codereview.chromium.org/13465021/diff/5001/tools/dom/templates/html/impl/impl_WheelEvent.darttemplate File tools/dom/templates/html/impl/impl_WheelEvent.darttemplate (right): https://codereview.chromium.org/13465021/diff/5001/tools/dom/templates/html/impl/impl_WheelEvent.darttemplate#newcode8 tools/dom/templates/html/impl/impl_WheelEvent.darttemplate:8: On 2013/04/03 00:10:31, Andrei Mouravski wrote: > Could you ...
7 years, 8 months ago (2013-04-03 00:35:34 UTC) #4
Andrei Mouravski
lgtm Err... I thought that I'd sent this already. https://codereview.chromium.org/13465021/diff/8002/tools/dom/templates/html/impl/impl_WheelEvent.darttemplate File tools/dom/templates/html/impl/impl_WheelEvent.darttemplate (right): https://codereview.chromium.org/13465021/diff/8002/tools/dom/templates/html/impl/impl_WheelEvent.darttemplate#newcode92 tools/dom/templates/html/impl/impl_WheelEvent.darttemplate:92: ...
7 years, 8 months ago (2013-04-03 15:22:00 UTC) #5
blois
Committed patchset #3 manually as r20879 (presubmit successful).
7 years, 8 months ago (2013-04-03 17:52:20 UTC) #6
blois
7 years, 8 months ago (2013-04-03 18:00:49 UTC) #7
Message was sent while issue was closed.
https://codereview.chromium.org/13465021/diff/8002/tools/dom/templates/html/i...
File tools/dom/templates/html/impl/impl_WheelEvent.darttemplate (right):

https://codereview.chromium.org/13465021/diff/8002/tools/dom/templates/html/i...
tools/dom/templates/html/impl/impl_WheelEvent.darttemplate:92: * * [W3C
WheelEvent](http://dev.w3.org/2006/webapi/DOM-Level-3-Events/html/DOM3-Events.html#events-WheelEvent-deltaY)
On 2013/04/03 15:22:00, Andrei Mouravski wrote:
> Nit: we have been annotating these as [Foo](link.com) from the W3C.

Done.

Powered by Google App Engine
This is Rietveld 408576698