Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

Replace CrMo check with Chrome to support Chrome/Android once more #13

Merged
merged 1 commit into from

2 participants

@jzaefferer

The CrMo string was removed from Chrome/Android, so for now our next-best bet is to look for just Chrome instead. The header is still fixed, so there's no point in trying to scroll anything away.

@jzaefferer jzaefferer The CrMo string was removed from Chrome/Android, so for now our next-…
…best bet is to look for just Chrome instead. The header is still fixed, so there's no point in trying to scroll anything away.
994f88d
@pbakaus pbakaus merged commit 62f7034 into zynga:master
@pbakaus

Thanks Jörn!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on May 3, 2012
  1. @jzaefferer

    The CrMo string was removed from Chrome/Android, so for now our next-…

    jzaefferer authored
    …best bet is to look for just Chrome instead. The header is still fixed, so there's no point in trying to scroll anything away.
This page is out of date. Refresh to see the latest.
Showing with 1 addition and 1 deletion.
  1. +1 −1  src/viewporter.js
View
2  src/viewporter.js
@@ -45,7 +45,7 @@ var viewporter;
var that = this;
// Scroll away the header, but not in Chrome
- this.IS_ANDROID = /Android/.test(navigator.userAgent) && !/CrMo/.test(navigator.userAgent);
+ this.IS_ANDROID = /Android/.test(navigator.userAgent) && !/Chrome/.test(navigator.userAgent);
var _onReady = function() {
Something went wrong with that request. Please try again.