Fixed safari detection bug

ijason · Apr 19, 2014 · a08f46c · a08f46c
1 parent 4c9b613
commit a08f46c
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/js/smart-app-banner.js b/js/smart-app-banner.js
@@ -4,10 +4,9 @@ $(function() {
   var iPad = navigator.userAgent.match(/iPad/i) != null; // Check if using an iPad
   var iPhone = navigator.userAgent.match(/iPhone/i) != null; // Check if using an iPhone
   var safari = navigator.userAgent.match(/Safari/i) != null; // Check if using Safari
-
-  var standalone = navigator.standalone;
+
   var appBannerID = $('meta[name=apple-itunes-app]').attr("content"); //Check if using smart app banners
-  if (!standalone && safari) { safari = false}; //Chrome is just a re-skinning of iOS WebKit UIWebView
+  if (navigator.userAgent.match('CriOS') && safari) { safari = false}; //Chrome is just a re-skinning of iOS WebKit UIWebView
   if (appBannerID != null) { 
     appBannerID = /app-id=([0-9]+)/.exec(appBannerID)[1]; // maybe developers add some parameter like that app-argument=http://domain.com/path/to/page
     if ((iPad || iPhone) && (!safari)) {

diff --git a/js/smart-app-banner.min.js b/js/smart-app-banner.min.js