From c8182dd5bf2d8df626fac09c629f97eede69c71f Mon Sep 17 00:00:00 2001
From: Ray Schamp <ray@scratch.mit.edu>
Date: Tue, 15 May 2018 10:22:18 -0400
Subject: [PATCH] Use bowser[browser name] for detection
Also, fix detection of IE so it works. Apparently bowser.name !== 'IE' on IE with bowser. bowser.msie works though.
---
src/lib/audio/shared-audio-context.js | 2 +-
src/lib/supported-browser.js | 7 +++----
2 files changed, 4 insertions(+), 5 deletions(-)
diff --git a/src/lib/audio/shared-audio-context.js b/src/lib/audio/shared-audio-context.js
index b459e9e8f..919920834 100644
--- a/src/lib/audio/shared-audio-context.js
+++ b/src/lib/audio/shared-audio-context.js
@@ -2,7 +2,7 @@ import StartAudioContext from 'startaudiocontext';
import bowser from 'bowser';
let AUDIO_CONTEXT;
-if (bowser.name !== 'IE') {
+if (!bowser.msie) {
AUDIO_CONTEXT = new (window.AudioContext || window.webkitAudioContext)();
StartAudioContext(AUDIO_CONTEXT);
diff --git a/src/lib/supported-browser.js b/src/lib/supported-browser.js
index 891a67f05..181fb1cb6 100644
--- a/src/lib/supported-browser.js
+++ b/src/lib/supported-browser.js
@@ -5,10 +5,9 @@ import bowser from 'bowser';
* @returns {boolean} False if the platform is definitely not supported.
*/
export default function () {
- if (bowser.name === 'IE' ||
- bowser.name === 'Opera' ||
- bowser.name === 'Opera Mini' ||
- bowser.name === 'Silk') {
+ if (bowser.msie ||
+ bowser.opera ||
+ bowser.silk) {
return false;
}
// @todo Should also test for versions of supported browsers
--
GitLab