Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WheelEvent.wheelDeltaX throws exception in dart2js #23287

Closed
DartBot opened this issue Apr 21, 2015 · 3 comments
Closed

WheelEvent.wheelDeltaX throws exception in dart2js #23287

DartBot opened this issue Apr 21, 2015 · 3 comments
Assignees
Labels
closed-duplicate Closed in favor of an existing report library-html P1 A high priority bug; for example, a single project is unusable or has many test failures

Comments

@DartBot
Copy link

DartBot commented Apr 21, 2015

This issue was originally filed by ferha...@google.com


What steps will reproduce the problem?

  1. Add event handler: div.onMouseWheel.listen
  2. Access WheelEvent member wheelDeltaX or Y in handler.

What is the expected output? What do you see instead?
In Dartium we get trackpad/mousewheel scroll amount. Dart2js compile gives exception.

What version of the product are you using?
Dart SDK version 1.10.0-dev.1.1

On what operating system?
Mac OSX Yosemite 10.10.3
What browser (if applicable)?
Chrome Version 42.0.2311.82 beta (64-bit)

Please provide any additional information below.

@dgrove
Copy link
Contributor

dgrove commented Apr 21, 2015

This has been brittle over the years - in addition to fixing this, we need to improve our testing here.


Set owner to @alan-knight.
Removed Priority-Unassigned label.
Added Priority-High, Library-Html, Triaged labels.

@alan-knight
Copy link
Contributor

Part of the brittleness here is that the underlying specs have changed. The wheelDeltaX attribute is part of the obsolete MouseWheelEvent interface https://developer.mozilla.org/en-US/docs/Web/API/MouseWheelEvent , which is deprecated in favour of WheelEvent https://developer.mozilla.org/en-US/docs/Web/API/WheelEvent

It is still present in the Chrome IDL, but listed as non-standard API, and doesn't work on Firefox. It should probably be either removed or deprecated.


Added Accepted label.

@alan-knight
Copy link
Contributor

Added Duplicate label.
Marked as being merged into #16046.

@DartBot DartBot added Type-Defect P1 A high priority bug; for example, a single project is unusable or has many test failures library-html closed-duplicate Closed in favor of an existing report labels Apr 21, 2015
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
closed-duplicate Closed in favor of an existing report library-html P1 A high priority bug; for example, a single project is unusable or has many test failures
Projects
None yet
Development

No branches or pull requests

3 participants