Skip to content
This repository has been archived by the owner on Aug 30, 2018. It is now read-only.

Fix iOS Safari off canvas scaling issues #518

Merged
merged 2 commits into from
Mar 2, 2016

Conversation

Cam
Copy link

@Cam Cam commented Mar 2, 2016

This fixes the changes in the way Safari interprets zooming and overflow on objects. As Apple consider their changes to be the right way to do things, they probably aren’t going to be changing the behaviour any time soon.

Cam Gould added 2 commits March 2, 2016 12:41
This fixes the changes in the way Safari interprets zooming and
overflow on objects. As Apple consider their changes to be the right
way to do things, they probably aren’t going to be changing the
behaviour any time soon.
This fixes the changes in the way Safari interprets zooming and overflow on objects. As Apple consider their changes to be the right way to do things, they probably aren’t going to be changing the behaviour any time soon.
@cshold
Copy link
Contributor

cshold commented Mar 2, 2016

Thank you for bringing this up. I'd been searching for a fix but never came across shrink-to-fit somehow.

For those wondering exactly what this fixes, see what happens in our Brooklyn theme when you close the navigation drawer (on iOS 9):
brooklyn

With this fix, you get a nice smooth drawer closing:
update

cc/ @stevebosworth @t-kelly

cshold added a commit that referenced this pull request Mar 2, 2016
Fix iOS Safari off canvas scaling issues
@cshold cshold merged commit 91fd07e into Shopify:master Mar 2, 2016
@Cam
Copy link
Author

Cam commented Mar 3, 2016

@cshold Apple are apparently taking an ideological stand on how viewport overflows should work :/

@Cam Cam deleted the iOS-viewport-patch branch March 3, 2016 10:30
@cshold
Copy link
Contributor

cshold commented Mar 3, 2016

Chrome gives a little warning, but small price to pay for iOS to be happy.

image

@Cam
Copy link
Author

Cam commented Mar 4, 2016

Yeah that seems like a small price to pay. Looks like it will be addressed in Chromium :) https://bugs.chromium.org/p/chromium/issues/detail?id=569649

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants