rodneyrehm / viewport-units-buggyfill

Making viewport units (vh|vw|vmin|vmax) work properly in Mobile Safari.

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Viewport units fixed in iOS8

bratvanov opened this issue · comments

I can confirm that viewport units are finally fixed in iOS 8 that was just released.

Maybe we can update the UA test to exclude iOS 8 and only load the buggyfill in <iOS7, as from what I can tell at the moment "isMobileSafari" targets all versions which isn't optimal?

For a reference:

This is the UA on my iPhone 4S running iOS 7.0.3 which I purposely haven't yet updated to iOS8:

Mozilla/5.0 (iPhone; CPU iPhone OS 7_0_3 like Mac OS X) 
AppleWebKit/537.51.1 (KHTML, like Gecko) Version/7.0 
Mobile/11B511 Safari/9537.53

And here is the UA of my iPad Air running iOS 8.0:

Mozilla/5.0 (iPad; CPU OS 8_0 like Mac OS X) 
AppleWebKit/600.1.4 (KHTML, like Gecko) Version/8.0 
Mobile/12A365 Safari/600.1.4

You're welcome to send a PR ;)

Unfortunately writing efficient regular expressions isn't my strong suit, so I'd prefer if someone more knowledgeable was to help out. :)

Hello, any updates on this one?

Anyone is welcome to send a PR (Pull Request).

You're welcome to merge the PR, hint hint ;)

merged "manually", sorry… thx!