Skip to content
This repository has been archived by the owner on Jul 29, 2019. It is now read-only.

[Timeline]: Horizontal and Vertical scroll by mouse wheel issue in Mozilla Firefox #3792

Open
gam0ny opened this issue Jan 26, 2018 · 7 comments

Comments

@gam0ny
Copy link

gam0ny commented Jan 26, 2018

Hi,
Thank you for the great tool! It covers really great part of functionality.

I'm experiencing an issue with timeline scroll by mouse wheel in Mozilla Firefox browser only.
The scrolling is really slow. It's clearly visible in the sample by the link:
http://plnkr.co/edit/zGzVxkhSsAngcLMH1xhV?p=preview

Could you please assist?
OS: Windows 8
FF: 58.0 (64-bit)

@scriptkiddielvl2
Copy link

vis.js // New school wheel delta (wheel event) :

var diff = delta / 120 * (this.range.end - this.range.start) / 20;

might become as a kind of fix :

if (navigator.userAgent.indexOf("Firefox") >= 0) {
var diff = delta / 3 * (this.range.end - this.range.start) / 20;
}
else var diff = delta / 120 * (this.range.end - this.range.start) / 20;

?!

@gam0ny
Copy link
Author

gam0ny commented Feb 15, 2018

Hi! Thank you so much for the idea of the fix! It really helped. Any idea how the fix can be requested in order to be included into the nearest build?

@scriptkiddielvl2
Copy link

Good it helps but I would not feel comfortable pushing this quick&dirt fix into pull request.
Mousewheel event is deprecated, moving to wheel is probably the right fix long term but much more work to achieve. /my2cents

@Sujay-shetty
Copy link

@gam0ny which line or method you inserted the @scriptkiddielvl2 code

@gam0ny
Copy link
Author

gam0ny commented Feb 20, 2018

@Sujay-shetty attaching screenshot:
image

@deltragon
Copy link

Is there any update on this?

yotamberk pushed a commit that referenced this issue Jul 1, 2018
* Normalize mouse wheel deltas for #3792

* include support for IE6/7/8
@deltragon
Copy link

The issue has been fixed, but since there hasn't been a new release with it and we only use the timeline, we have now switched to @yotamberk's https://github.com/yotamberk/timeline-plus.

@yotamberk yotamberk reopened this Jul 18, 2018
mojoaxel pushed a commit to visjs/vis_original that referenced this issue Jun 9, 2019
* Normalize mouse wheel deltas for almende#3792

* include support for IE6/7/8
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants