-
Notifications
You must be signed in to change notification settings - Fork 281
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
iOS 7 (maybe all) Doesn't work unless you scroll from the very top #19
Comments
This is possibly a bug having to do with Bootstrap, I tested it on a few other sites, and it works as planned. I will look into this. |
It could be. It works as intended when I grabbed an older version of hook (specifically using the source on gethook.com) FYI. |
@marcmedina which one worked for you? I have the same problem without using bootstrap. |
I have the same problem with uikit |
I had the same issue and I think that I managed to fix it by doing the following changes: addHandler('touchmove', function(e) {
// Should substract instead of adding
//swipe = e.originalEvent.touches[0].pageY + lastY;
swipe = e.originalEvent.touches[0].pageY - lastY;
st = $(this).scrollTop();
// This prevents the user from swiping down
/*if(swipe < settings.swipeDistance) {
e.preventDefault();
}*/
// Second condition prevents swipe from anywhere
// below the first 40 pixels
if(swipe > settings.swipeDistance /*&& lastY <= 40*/) {
methods.onSwipe($this, settings);
}
}); I'm not sure whether removing those lines breaks another part of the functionality that I'm missing, but at least it seems to let you swipe from anywhere on the screen. |
It seems using the newest version, I could not get this to fire. Then I tried pulling from the top of the browser (right below the address bar) and it works. Very strange, I tried with a previous version of Hook and it works like intended. Perhaps the is a bug in the latest version?
I have a basic bootstrap page up with hook (http://medinastudios.com/test/)
Trying pulling it down, it only seems to work when you pull next to the top nav bar.
The text was updated successfully, but these errors were encountered: