anvaka / anvaka/wheel

deltaZ and firefox issue

Open
#6 2 comments 0 reactions 0 assignees View on GitHub
Dominant language
JavaScript
Stars
30
Forks
7
PR merge metrics
No merged PRs in 30d

Description

Hi @anvaka!

Awesome package, I use it in [pan-zoom](https://npmjs.org/package/pan-zoom), in [gl-waveform](https://github.com/audio-lab/gl-waveform) and [plot-grid](https://github.com/dfcreative/plot-grid) in particular.

One gotcha.
I used `wheelDeltaX` and `wheelDeltaY` events, but they seem to be not polyfilled, but present in chrome.
So firefox just errored. I started digging and first question - is [there](https://github.com/anvaka/wheel/blob/master/index.js#L62) a typo? Should it be `deltaY`?

Maybe we should polyfill `wheelDeltaX` and `wheelDeltaY`?

Also it seem to be inconsistend through chrome/firefox :(. Same issue as [mouse-wheel](https://github.com/mikolalysenko/mouse-wheel).

Thanks!

Contributor guide

No contributing guide indexed for this repository

Research direction

Start with index.js at the linked line around the wheelDeltaX and wheelDeltaY handling, then compare the event properties available in Firefox and Chrome. Use the related mouse-wheel issue as context and verify that the resulting behavior is consistent across both browsers.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript
Domain
frontend
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.