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

Improved feature detection of touch scrolling #844

Closed
wants to merge 8 commits into from

Conversation

kriszyp
Copy link
Member

@kriszyp kriszyp commented Feb 14, 2014

I couldn't stand the broken scroll on my touch+mouse computer, so here is a possible fix.

@kriszyp
Copy link
Member Author

kriszyp commented Feb 14, 2014

I don't understand why this wasn't just kriszyp@33d59db.

@kfranqueiro
Copy link
Member

I'm wondering if your master branch has somehow diverged from SitePen/dgrid's... the last thing I see in your master branch is a merge commit, which seems wrong. I would suggest trying to rebase your master against SitePen/dgrid master, and then rebasing this branch against your master once it's fixed.

Edit: Actually, I guess if something really diverged, rebasing is still going to leave your master in a state that's inconsistent from upstream. A hard reset may be necessary.

Incidentally, I think one of the applications of this in your 2nd commit may help resolve #828, although I suspect it may also have the effect of disabling touch support for certain components.

@kfranqueiro
Copy link
Member

Fixed via fd95bfb, 86ad71d, ab64e8e, and 3e4f66e instead.

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