关于重构的 lyricManager #579
MoYingJi
started this conversation in
讨论 General
Replies: 1 comment
-
|
重构主要是感觉没有必要用四个歌词数组来表示
|
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
目前重构的 lyricManager 用下来问题还是有很多的,有时体验还没重构前的版本好
比如关于网络的一系列问题又回来了(因为逻辑又回到了处理完才设置)、歌词在上一首歌(代码里貌似是有对此情况处理的 不知道为什么)、只要开了歌词排除就不区分 TTML 和本地了
对于本地歌词覆盖在线歌词功能,加了前缀后又不识别了,我已经提出 PR 了(#578)。我感觉及时沟通还是比较重要的,现在我也不能立即明白此举是故意的还是不小心的
对于最后一个问题我个人的设想是这样的
export interface SongLyric { lrcData: LyricLine[]; yrcData: LyricLine[]; + + isLrcLocal: boolean; + isYrcLocal: boolean; + isLrcTTML: boolean; + isYrcTTML: boolean; }我同时也是有 #532 里第三条的想法,即标注歌词 来自本地 / 是 TTML 歌词,所以何不直接标注歌词的来源?这个歌词来自本地还是网络,格式是 TTML 还是 LRC/YRC
不知道对此作者是怎么想的
Beta Was this translation helpful? Give feedback.
All reactions