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

Workaround for Android 'inert-visual-viewport' behavior #1566

Merged
merged 1 commit into from Nov 30, 2016

Conversation

robwalch
Copy link
Contributor

@robwalch robwalch commented Nov 22, 2016

This is limited to horizontal sliders on Android where pageXOffset is not 0 - assuming this means the viewport is offset from zooming and not scrolling the page. It's not intended for all viewport settings where horizontal scrolling without zooming the viewport can occur. Most pages only scroll vertically at 100% (not zoomed), and only scroll horizontally when zoomed in so this should help compatibility with 'inert-visual-viewport' behavior.

JW7-1078

@jwplayer-robot
Copy link

Automated tests passed!

Cheers! 🍻 👐

@jw-kaurand jw-kaurand self-assigned this Nov 22, 2016
@jw-kaurand
Copy link
Contributor

This works pretty well for dragging. Can we expand this solution to also work when tapping on the time slider?

not intended for all viewport settings where horizontal scrolling without zooming the viewport can occur
JW7-1078
@robwalch robwalch force-pushed the bugfix/android-slider-viewport-zoomed branch from 15b42ea to b25b6dc Compare November 22, 2016 19:59
@robwalch
Copy link
Contributor Author

Thx @jw-kaurand. Fixed it.

@jwplayer-robot
Copy link

Automated tests passed!

Cheers! 🍻 👐

Copy link
Contributor

@jw-kaurand jw-kaurand left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like it. Let's merge this a little later for 7.8.1

@robwalch robwalch added this to the v7.8.2 milestone Nov 30, 2016
@egreaves egreaves merged commit 81cb7de into master Nov 30, 2016
@robwalch robwalch deleted the bugfix/android-slider-viewport-zoomed branch March 14, 2017 13:59
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.

None yet

4 participants