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

一些问题:歌词读取、播放界面,整体画面偏左、UI等 #12

Open
jkqxl opened this issue Oct 23, 2024 · 3 comments
Open
Labels
enhancement New feature or request

Comments

@jkqxl
Copy link

jkqxl commented Oct 23, 2024

尝试了下,看起来这个播放器还不够完善,期待未来,以下是发现的一些问题,以及建议:

  1. 一些歌曲都无法读取标签的歌词,但偶尔也行,不清楚为什么
  2. 缺乏全屏模式,且在最大化模式下,整体画面偏左,一般人个人认为还是希望可以画面居中的吧,因此希望可以增加居中的选项
  3. 播放界面,左侧封面+下面进度条+按钮,个人感觉还是稍小,在点击时,总是担心误点的问题,还有在最大化模式下,“封面+下面进度条+按钮”距离上下界面稍远,显得较为空旷,因此请求增加自适应调整,可以放大的选项,预期效果对应类似字体放大的一样
  4. 虽然播放界面的UI已经足够好,但个人认为依然可以进一步,请求增加播放界面的封面背景,即当前是动态纯色,预期是封面+动态纯属,即封面+透明度 覆盖 动态纯色+透明度
@YanJi314
Copy link
Collaborator

收到,感谢反馈!

  1. 目前的逻辑是依次读取「内嵌lrc格式歌词 -> 内嵌同步歌词 -> 目录下同名.lrc文件」,如果都未成功或无信息,则显示「暂无歌词」。如果有内嵌歌词但是无法成功读取的,请问能否提供一个文件用于复现问题?非常感谢~
  2. 如果您指的是播放页全屏显示的全屏模式,可以在播放页按 F11 或播放控制界面右侧「...」图标进入;目前如果歌词宽度与屏幕相比较窄确实会出现整体偏左的问题,下版应该会增设相关宽度调整设置项。
  3. 自适应调整在路上了,应该下版会有,开发的时候自己屏幕比较小,没有考虑到大屏幕上间距的问题,感谢建议。
  4. 下版计划添加一个专门的「播放界面」设置页,曲目封面背景浮层应该会安排上,感谢建议。

由于本人学业原因下版可能还要等挺久才能出,再次感谢支持~

@YanJi314 YanJi314 added the enhancement New feature or request label Oct 24, 2024
@jkqxl
Copy link
Author

jkqxl commented Oct 25, 2024

@YanJi314

  1. 歌词读取:本人一般习惯使用foobar2000将歌词保持到标签中,不清楚为什么无法读取,如下图

Snipaste_2024-10-25_10-11-55

歌曲文件

Whitney Houston - I Will Always Love You.zip

2.全屏:按 F11 确实可以,播放控制界面右侧「...」图标,这个就藏的太深了,个人感觉以功能分类也不太合适,还是放到右上角比较好

3.播放器界面自适应调整:期望达到的效果是,封面尽可能的要大,这样才不会感觉整个画面比较空旷,四周一片都是空白,对应的如果走极简风格,歌词标题其实也可以放到右侧歌词上方。简单的预期分布如下图,当然是件实际还要进一步的调整:

Snipaste_2024-10-25_10-18-07

最后,等有空了,记得更新就好。。。

@PYLXU
Copy link

PYLXU commented Nov 20, 2024

@jkqxl
歌词标签貌似不是直接塞lyrics的吧
我用Mp3tag是需要放入UNSYNCEDLYRICS标签的

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants