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

Add mouseWheelTimeout option #450

Closed
wants to merge 5 commits into from
Closed

Add mouseWheelTimeout option #450

wants to merge 5 commits into from

Conversation

HarryLyu
Copy link

Hello!

On some touchpads (and maybe with normal mouses) when user scrolls a lot, plugin scrolls slides twice.

I've added a 'mouseWheelTimeout' option so you can add some kind of 'debouncing' to MouseWheelHandler.

It works for me with value 700.

@alvarotrigo
Copy link
Owner

That's already been done with the default value for the scrollDelay variable in combination with setTimeout.

If you are having troubles, you can increase that value which is now set to 600 inside the plugin code.
But of course, you will find other users complaning of this delay in desktop computers without trackpad (or without the Apple magic mouse).

@alvarotrigo
Copy link
Owner

This topic is related with this another one in which it was being discussed if the variable scrollDelay should be introduced as a parameter or not.

alvarotrigo added a commit that referenced this pull request Feb 16, 2015
…blem)#785 #756 #452 #450 #314 #298 #281 #164 #73 #12

- Removed delay when using keyboard arrows to scroll #874 #333 #84
- Fixed bug with IE 11 touch compatibility in touch screens #937
- Fixed bug with destroy('all') fired when resizing #1010
- Fixed bug with focused inputs in mobile #1008 (other types besides text)
- Fixed bug with navigation bullets and scrollBar:true #974
- Fixed bug with Firefox on touch screens? #1016
- Changed jquery default easing from `easeInQuart` to `easeInOutCubic` (more fluid)
- Improved keboard accessibility controls 
- Scroll on moving the mouse when pressing the 3rd button (middle button)
- Added class in body element after destroying fullpage.js
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants