Skip to content
This repository was archived by the owner on Jul 17, 2026. It is now read-only.

feat(lyric): 优化歌词排除 - #514

Merged
imsyy merged 10 commits into
SPlayer-Dev:devfrom
MoYingJi:feat
Oct 24, 2025
Merged

imsyy merged 10 commits into
SPlayer-Dev:devfrom
MoYingJi:feat

Conversation

@MoYingJi

@MoYingJi MoYingJi commented Oct 24, 2025 •

Copy link
Copy Markdown
Collaborator

此 PR 已经编辑完成

主要变动:

  • 增加 excludeRegexes 设置项,同时增加 getExcludeRegexes 函数
  • 增加 isLyricExcluded 函数,并将所有排除歌词的逻辑替换为此函数
  • ExcludeKeywords 改名为 ExcludeLyrics
  • 在 ExcludeLyrics 中编辑 excludeRegexes 设置项
  • 增加 enableExcludeTTML 设置项
  • 增加 enableExcludeLocalLyrics 设置项
  • 为所有调用了 isLyricExcluded 的函数增加 skipExclude 参数,调用这些函数的时候必须先判断 enableExcludeTTML 和 enableExcludeLocalLyrics 并以此传入 skipExclude 参数

@MoYingJi

Copy link
Copy Markdown
Collaborator Author

#513

@MoYingJi
MoYingJi marked this pull request as ready for review October 24, 2025 10:41
@MoYingJi

Copy link
Copy Markdown
Collaborator Author

现在有个很奇怪的 BUG,重启程序或者热重载后的第一次歌词解析时,TTML 歌词必定也会被排除 不受设置影响🤔

@MoYingJi

Copy link
Copy Markdown
Collaborator Author

现在有个很奇怪的 BUG,重启程序或者热重载后的第一次歌词解析时,TTML 歌词必定也会被排除 不受设置影响🤔

BUG 原因:投机取巧的两个人

图片 图片

@MoeFurina

Copy link
Copy Markdown
Contributor

自从我通过PR加入ttml格式歌词功能后, 你的代码贡献挺勤奋的

@MoeFurina

Copy link
Copy Markdown
Contributor

#481

@MoYingJi

Copy link
Copy Markdown
Collaborator Author

自从我通过PR加入ttml格式歌词功能后, 你的代码贡献挺勤奋的

因为加入这个功能之后体验要好不少,使用强度也更高,因此能发现更多的问题,也更愿意贡献

之前我还不会 TypeScript/Vue/Electron,最近我在研究 VitePress (部署在我的 GitHub Pages),所以也会了一小点的 TypeScript 和 Vue,仿照这个项目原来的代码就能写写了

@imsyy
imsyy merged commit 1d4aa43 into SPlayer-Dev:dev Oct 24, 2025
@MoYingJi
MoYingJi deleted the feat branch October 24, 2025 15:12
@MoYingJi

MoYingJi commented Oct 24, 2025 •

Copy link
Copy Markdown
Collaborator Author

奇怪,开发环境没问题,为啥一编译出来就报错一堆 🤔
是我的问题吗?

以后就在开发环境用吧(bushi

图片

@imsyy

imsyy commented Oct 24, 2025

Copy link
Copy Markdown
Member

没事,这是 store 的报错,试试清除下缓存

@MoYingJi

Copy link
Copy Markdown
Collaborator Author

程序的设置里好像没有清除缓存的选项
是要清除所有配置吗?还是直接删文件

@imsyy

imsyy commented Oct 24, 2025

Copy link
Copy Markdown
Member

程序的设置里好像没有清除缓存的选项 是要清除所有配置吗?还是直接删文件

image

可以在控制台的这里删除

@MoYingJi

Copy link
Copy Markdown
Collaborator Author

好的 谢谢 问题解决了

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants