Fix bvid convert (the1812#4007)

szlysms · Apr 8, 2023 · 315d830 · 315d830
1 parent 508b791
commit 315d830
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 15 deletions.
diff --git a/registry/lib/components/video/bvid-convert/BvidConvert.vue b/registry/lib/components/video/bvid-convert/BvidConvert.vue
@@ -1,19 +1,17 @@
 <template>
  <div class="bvid-convert">
- <template v-if="aid && bvid">
- <div class="bvid-convert-item">
- {{ aid }}
- <div class="bvid-convert-item-copy" title="复制链接" @click="copyLink('aid')">
- <VIcon :size="16" :icon="aidCopied ? 'mdi-check' : 'mdi-link'" />
- </div>
+ <div v-if="aid" class="bvid-convert-item">
+ {{ aid }}
+ <div class="bvid-convert-item-copy" title="复制链接" @click="copyLink('aid')">
+ <VIcon :size="16" :icon="aidCopied ? 'mdi-check' : 'mdi-link'" />
  </div>
-  <div class="bvid-convert-item">
-  {{ bvid }}
-  <div class="bvid-convert-item-copy" title="复制链接" @click="copyLink('bvid')">
-  <VIcon :size="16" :icon="bvidCopied ? 'mdi-check' : 'mdi-link'" />
- </div>
+ </div>
+ <div v-if="bvid" class="bvid-convert-item">
+ {{ bvid }}
+ <div class="bvid-convert-item-copy" title="复制链接" @click="copyLink('bvid')">
+ <VIcon :size="16" :icon="bvidCopied ? 'mdi-check' : 'mdi-link'" />
  </div>
- </template>
+ </div>
  </div>
 </template>
 
@@ -112,7 +110,6 @@ export default Vue.extend({
  flex-direction: column;
  border-radius: 4px;
  padding: 6px 8px;
- width: 100%;
  user-select: text;
  box-sizing: border-box;
  box-shadow: 0 0 0 1px #8884;

diff --git a/src/components/video/player-adaptor/bpx.ts b/src/components/video/player-adaptor/bpx.ts
@@ -30,8 +30,8 @@ const idPolyfill = async () => {
  cid: pbp.options.cid.toString(),
  bvid: pbp.options.bvid,
  }
- if (Object.values(idData).some(it => it === '' || parseInt(it) <= 0)) {
- console.warn('[bpx player polyfill] invalid pbp data')
+ if (Object.values(idData).some(it => !it || parseInt(it) <= 0)) {
+ console.warn('[bpx player polyfill] invalid pbp data', pbp.options)
  }
  Object.assign(unsafeWindow, idData)
  }