refactor: Simplify thumbnails implementation in Artplayer

zhw2590582 · Aug 15, 2024 · 65b0b2c · 65b0b2c
1 parent 3e3b38f
commit 65b0b2c
Show file tree

Hide file tree

Showing 5 changed files with 17 additions and 41 deletions.
diff --git a/docs/uncompiled/artplayer/index.js b/docs/uncompiled/artplayer/index.js
diff --git a/docs/uncompiled/artplayer/index.js.map b/docs/uncompiled/artplayer/index.js.map
diff --git a/packages/artplayer/src/control/index.js b/packages/artplayer/src/control/index.js
@@ -8,7 +8,6 @@ import progress from './progress';
 import time from './time';
 import volume from './volume';
 import setting from './setting';
-import thumbnails from './thumbnails';
 import screenshot from './screenshot';
 import airplay from './airplay';
 
@@ -81,13 +80,11 @@ export default class Control extends Component {
             );
         }
 
-        this.add(
-            thumbnails({
-                name: 'thumbnails',
-                position: 'top',
-                index: 20,
-            }),
-        );
+        this.add({
+            name: 'thumbnails',
+            position: 'top',
+            index: 20,
+        });
 
         this.add(
             playAndPause({

diff --git a/packages/artplayer/src/control/thumbnails.js b/packages/artplayer/src/control/thumbnails.js
diff --git a/packages/artplayer/src/player/thumbnailsMix.js b/packages/artplayer/src/player/thumbnailsMix.js
@@ -46,14 +46,15 @@ export default function thumbnailsMix(art) {
 
     art.on('setBar', async (type, percentage, event) => {
         const $thumbnails = art.controls?.thumbnails;
-        if (!$thumbnails) return;
+        const { url } = option.thumbnails;
+        if (!$thumbnails || !url) return;
 
         const isMobileDroging = type === 'played' && event && isMobile;
 
         if (type === 'hover' || isMobileDroging) {
             if (!loading) {
                 loading = true;
-                image = await loadImg(option.thumbnails.url);
+                image = await loadImg(url);
                 isLoad = true;
             }