Skip to content

Fix for older mouse scrollwheels#117

Open
ryanturnerwebdev wants to merge 2 commits intoalvarotrigo:masterfrom
ryanturnerwebdev:master
Open

Fix for older mouse scrollwheels#117
ryanturnerwebdev wants to merge 2 commits intoalvarotrigo:masterfrom
ryanturnerwebdev:master

Conversation

@ryanturnerwebdev
Copy link
Copy Markdown

Changed delta operator to ‘less than or equal to’ for scrollwheels that
don’t change delta on small scrolls

Changed delta operator to ‘less than or equal to’ for scrollwheels that
don’t change delta on small scrolls
@alvarotrigo
Copy link
Copy Markdown
Owner

An else should be enough.
But in any case, have you checked this solves your problem?

@alvarotrigo
Copy link
Copy Markdown
Owner

alvarotrigo commented Nov 29, 2016

fullPage.js uses:

                        if (delta < 0) {
                            scrolling('down', scrollable);

                        //scrolling up?
                        }else {
                            scrolling('up', scrollable);
                        }

And you probably don't have any issue there.
So not quite sure if the problem is there in any case.

@ryanturnerwebdev
Copy link
Copy Markdown
Author

Yes this solves the issue I was having. Pushed new commit to as suggested.

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