Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

添加新组件“视频页默认定位”(videoDefaultLocation) #2960

Merged
merged 14 commits into from
Feb 13, 2022

Conversation

timongh
Copy link
Contributor

@timongh timongh commented Jan 30, 2022

与“播放器定位”组件的关系

新组件与原来的“播放器定位”(playerFocus)是差不多的。但“播放器定位”的缺点是:

  1. 与“自定义顶栏”配合地不是很好
  2. 与带定位的功能配合地也不太好
  3. 在新版播放器下似乎失效了

总的来说就是:在未使用其他插件时,“播放器定位”开箱即用,比较方便。但与其他功能配合时,自定义比较麻烦。

新组件介绍

新组件同样是用于播放器定位,但添加了比较灵活的自定义方式。下面是其使用演示:

org.mp4

@timongh
Copy link
Contributor Author

timongh commented Jan 30, 2022

我有点担心,我弄的这几个按钮,用户会不会搞不懂它们的用法?

本来想多写点简介把用法写清楚;但是简介多了就难看了,用户要是一打开面板就看见一大堆字,估计连看下去的欲望都没有了。

@the1812
Copy link
Owner

the1812 commented Feb 1, 2022

简介就是把用法写清楚的地方, 长一点没关系.

class 建议用具体一些的名称, 即使是使用 scoped, b 站的样式依然能够产生影响

@timongh
Copy link
Contributor Author

timongh commented Feb 4, 2022

先别合并,我还想再研究一下等待加载的逻辑。再优化优化

@timongh
Copy link
Contributor Author

timongh commented Feb 4, 2022

突然想到,视频页面、番剧页面等各个页面,都是不一样的。全部使用同一个默认位置不太合理。

似乎原来的“播放器定位”的逻辑才是正确的。应该按照播放器定位,而不是页面顶部定位。

所以我白写了?🤔

@the1812
Copy link
Owner

the1812 commented Feb 5, 2022

你可以先限制为只对视频生效

@timongh
Copy link
Contributor Author

timongh commented Feb 12, 2022

Ok,功能方面写完了。但关于内部的 ExtendBox,还有一部分样式没写:

  • 对夜间模式的支持
  • :hover 时的高亮
  • 有关 focus 的样式和方法
  • @/ui 中用的图标是 svg 写的,而这个里面使用的是 mdi 图标,样式不统一

不过最近写这个写累了,近期可能都不想再改了,就这样先合并了吧。

@timongh timongh changed the title 添加新组件“播放页默认定位”(videoDefaultLocation) 添加新组件“视频页默认定位”(videoDefaultLocation) Feb 12, 2022
@the1812 the1812 merged commit 22916ea into the1812:preview-features Feb 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants