Skip to content

Commit

Permalink
Merge pull request #1910 from yarysh/master
Browse files Browse the repository at this point in the history
fix(platform): ons.platform.isSafari() doesn't work in Safari 10
  • Loading branch information
asial-matagawa authored Apr 6, 2017
2 parents 72c0e9a + bf7298f commit cf0f96d
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion core/src/ons/platform.js
Original file line number Diff line number Diff line change
Expand Up @@ -230,7 +230,7 @@ class Platform {
if (this._renderPlatform) {
return this._renderPlatform === 'safari';
} else {
return (Object.prototype.toString.call(window.HTMLElement).indexOf('Constructor') > 0);
return (Object.prototype.toString.call(window.HTMLElement).indexOf('Constructor') > 0) || (function (p) { return p.toString() === '[object SafariRemoteNotification]' })(!window['safari'] || safari.pushNotification);
}
}

Expand Down

0 comments on commit cf0f96d

Please sign in to comment.