Use native video in mobile(close: #26); Fix UMD bug

DIYgod · Sep 25, 2016 · 9fb38b1 · 9fb38b1
1 parent ba3be09
commit 9fb38b1
Show file tree

Hide file tree

Showing 4 changed files with 16 additions and 5 deletions.
diff --git a/dist/DPlayer.min.js b/dist/DPlayer.min.js
diff --git a/dist/DPlayer.min.js.map b/dist/DPlayer.min.js.map
diff --git a/src/DPlayer.js b/src/DPlayer.js
@@ -37,7 +37,7 @@ class DPlayer {
             `;
         };
 
-        const isMobile = navigator.userAgent.match(/(iPad)|(iPhone)|(iPod)|(android)|(webOS)/i);
+        const isMobile = /mobile/i.test(window.navigator.userAgent);
         // compatibility: some mobile browsers don't suppose autoplay
         if (isMobile) {
             option.autoplay = false;
@@ -117,6 +117,16 @@ class DPlayer {
             this.element.classList.add('dplayer-no-danmaku');
         }
 
+        if (isMobile) {
+            this.element.innerHTML = `
+                <div class="dplayer-video-wrap">
+                    <video class="dplayer-video" ${option.video.pic ? `poster="${option.video.pic}"` : ``} preload="${option.preload}" controls>
+                        <source src="${option.video.url}">
+                    </video>
+                </div>`;
+            return;
+        }
+
         this.element.innerHTML = `
             <div class="dplayer-mask"></div>
             <div class="dplayer-video-wrap">
@@ -1305,4 +1315,4 @@ class DPlayer {
     }
 }
 
-export {DPlayer};
+module.exports = DPlayer;
diff --git a/webpack.config.js b/webpack.config.js
@@ -14,6 +14,7 @@ module.exports = {
     output: {
         path: BUILD_PATH,
         filename: 'DPlayer.min.js',
+        library: 'DPlayer',
         libraryTarget: 'umd',
         umdNamedDefine: true
     },