Skip to content

bug: statusTap issue with native scroll on iOS #4743

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

Closed
andreialecu opened this issue Dec 8, 2015 · 4 comments
Closed

bug: statusTap issue with native scroll on iOS #4743

andreialecu opened this issue Dec 8, 2015 · 4 comments
Milestone

Comments

@andreialecu
Copy link

Type: bug

Platform: ios 8 webview

When tapping the status bar in the new ionic master while there is a momentum scroll happening the scroll to top doesn't complete properly. It snaps to top then back down to where the initial momentum scroll's end position was. This happens with native scrolling.

This can probably be fixed by forcing overflow: hidden on the overflow-scroll="true" div to freeze the scroll.

This was also an issue in previous ionic versions with native scrolling on iOS.

@mlynch mlynch added this to the 1.2 milestone Dec 9, 2015
@mlynch
Copy link
Contributor

mlynch commented Dec 9, 2015

Thanks, will get this fixed

@andreialecu
Copy link
Author

@mlynch I opened a PR at #4748 which fixes this.

@mlynch mlynch closed this as completed in 853453c Dec 9, 2015
@mlynch
Copy link
Contributor

mlynch commented Dec 9, 2015

Thanks for guiding this one, I did a slightly different solution. Mind testing on WKWebView and letting me know how it goes?

@andreialecu
Copy link
Author

@mlynch I just tested it and it works, however, I think .freeze should still be implemented instead of it being NOOP, otherwise when you start dragging up/down from the left margin so that the content scrolls, and then you drag to the right to enable the swipe to go back gesture, you can both scroll the view and expose the previous view which seems glitchy.

@ionitron-bot ionitron-bot bot locked and limited conversation to collaborators Sep 7, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants