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

increase swipe length required to open navigation #835

Merged
merged 1 commit into from Apr 28, 2015

Conversation

Projects
None yet
2 participants
@mfairchild365
Member

mfairchild365 commented Apr 27, 2015

When scrolling down a long page of information on a touch enabled mobile device, it was easy to accidentally open the side navigation. This was frustrating and not expected. This commit aims to address that issue by doubling the distance required to swipe for the navigation to open. While it might not be a perfect solution, it is hopefully better than it used to be.

Maybe it would be better to remove this feature altogether?

increase swipe length required to open navigation
When scrolling down a long page of information on a touch enabled mobile device, it was easy to accidentally open the side navigation.  This was frustrating and not expected.  This commit aims to address that issue by doubling the distance required to swipe for the navigation to open.  While it might not be a perfect solution, it is hopfully better than it used to be.
@kabel

This comment has been minimized.

Show comment
Hide comment
@kabel

kabel Apr 28, 2015

Contributor

Let's accept this for now and propose to remove it in the next Gov Board meeting. I think this has been a failed experiment in used expectation.

Contributor

kabel commented Apr 28, 2015

Let's accept this for now and propose to remove it in the next Gov Board meeting. I think this has been a failed experiment in used expectation.

kabel added a commit that referenced this pull request Apr 28, 2015

Merge pull request #835 from mfairchild365/navigation
increase swipe length required to open navigation

@kabel kabel merged commit f981356 into unl:develop Apr 28, 2015

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment