fix utils.isMobile

DIYgod · Jan 11, 2018 · 15631ec · 15631ec
1 parent 1002347
commit 15631ec
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 11 deletions.
diff --git a/src/js/DPlayer.js b/src/js/DPlayer.js
@@ -1,4 +1,4 @@
-import utils, { isMobile } from './utils';
+import utils from './utils';
 import handleOption from './options';
 import i18n from './i18n';
 import Template from './template';
@@ -48,7 +48,7 @@ class DPlayer {
         if (this.options.live) {
             this.container.classList.add('dplayer-live');
         }
-        if (isMobile) {
+        if (utils.isMobile) {
             this.container.classList.add('dplayer-mobile');
         }
         this.arrow = this.container.offsetWidth <= 500;
@@ -83,10 +83,10 @@ class DPlayer {
                         this.template.danmakuLoading.style.display = 'none';
 
                         // autoplay
-                        if (this.options.autoplay && !isMobile) {
+                        if (this.options.autoplay && !utils.isMobile) {
                             this.play();
                         }
-                        else if (isMobile) {
+                        else if (utils.isMobile) {
                             this.pause();
                         }
                     }, 0);
@@ -128,10 +128,10 @@ class DPlayer {
 
         if (!this.danmaku) {
             // autoplay
-            if (this.options.autoplay && !isMobile) {
+            if (this.options.autoplay && !utils.isMobile) {
                 this.play();
             }
-            else if (isMobile) {
+            else if (utils.isMobile) {
                 this.pause();
             }
         }

diff --git a/src/js/controller.js b/src/js/controller.js
@@ -1,12 +1,12 @@
-import utils, { isMobile } from './utils';
+import utils from './utils';
 import Thumbnails from './thumbnails';
 
 class Controller {
     constructor (player) {
         this.player = player;
 
         this.autoHideTimer = 0;
-        if (!isMobile) {
+        if (!utils.isMobile) {
             this.player.container.addEventListener('mousemove', () => {
                 this.setAutoHide();
             });
@@ -30,7 +30,7 @@ class Controller {
             this.player.toggle();
         });
 
-        if (!isMobile) {
+        if (!utils.isMobile) {
             this.player.template.videoWrap.addEventListener('click', () => {
                 this.player.toggle();
             });

diff --git a/src/js/options.js b/src/js/options.js
@@ -1,10 +1,10 @@
 /* global DPLAYER_VERSION GIT_HASH */
 import defaultApiBackend from './api.js';
+import utils from './utils';
 
 export default (options) => {
-    const isMobile = /mobile/i.test(window.navigator.userAgent);
     // compatibility: some mobile browsers don't suppose autoplay
-    if (isMobile) {
+    if (utils.isMobile) {
         options.autoplay = false;
     }