Skip to content

feat(player): redesign danmaku source selector - #2557

Merged
Predidit merged 4 commits into
Predidit:mainfrom
moonrailgun:feat/danmaku-source-selector
Sep 21, 2026
Merged

Predidit merged 4 commits into
Predidit:mainfrom
moonrailgun:feat/danmaku-source-selector

Conversation

@moonrailgun

@moonrailgun moonrailgun commented Sep 12, 2026 •

Copy link
Copy Markdown
Contributor

描述

切换弹幕选择器的设计,包括桌面端和移动端。

优化了对海贼王这种超多集数番剧的使用体验

关联的 Issues

无。

PR 类型

  • Bug 修复
  • 添加新功能
  • 重构实现
  • 文档或格式
  • 其它

AI 辅助开发

  • 没有使用 AI 辅助
  • AI 辅助开发了部分功能,但是我已检查并理解 AI 所写的代码
  • 基本上都由 AI 实现,但是我非常了解 AI 的代码与写完的产物

AI 辅助模型: OpenAI GPT-6(Codex)。本次改动使用了 AI 辅助开发。

提交前检查

  • 我已经填写了 PR 模板中的所有内容
  • 我已经阅读了 贡献指引 ,并遵守指引进行开发
  • 这个 PR 的功能已经经过我的测试,并且我完全理解我(或 AI)做出的改动

附注

对提交 7b2a0738 的独立副本执行了本地检查:

  • flutter test --no-pub --reporter expanded:201 项测试通过。
  • flutter analyze --no-pub --no-fatal-infos --fatal-warnings:通过;5 项 avoid_print 信息提示,无警告或错误。

移动端

搜索弹幕源 选择番剧 选择弹幕分集
移动端搜索弹幕源 移动端选择番剧 移动端选择弹幕分集

桌面端

选择番剧:

桌面端选择番剧

选择弹幕分集:

桌面端选择弹幕分集

长列表的搜索、集号跳转与区段筛选:

桌面端长列表分集选择

@Predidit

Copy link
Copy Markdown
Owner

此PR包含了过多自定义样式,考虑复用已有的封装,以适应 m3e 设计规范并减少重复代码

@moonrailgun
moonrailgun force-pushed the feat/danmaku-source-selector branch from d704fd1 to 1bccde5 Compare September 14, 2026 15:38
@moonrailgun

moonrailgun commented Sep 14, 2026 •

Copy link
Copy Markdown
Contributor Author

Hi @Predidit ,我认为新的改动已经尽我所能去复用了,大概删减了200+ lines代码,然后的话辛苦再检查一下
如果有新的指导意见的话,请随时说。
感谢

@Predidit

Copy link
Copy Markdown
Owner

感谢您的工作,还有一个小问题,没有复用 KazumiDialog 来处理弹窗和相关路由

此封装保证了 Dialog 的易用与安全,在此项目中类似的场合被广泛使用

如果 KazumiDialog 在这里无法满足你的要求,考虑拓展此封装

@moonrailgun

Copy link
Copy Markdown
Contributor Author

感谢建议,已改为复用现有的 KazumiDialog 来显示和关闭弹窗,保留了横竖屏布局及弹幕源选择功能,无需扩展封装。修改已推送,麻烦再看一下。

@moonrailgun

Copy link
Copy Markdown
Contributor Author

@Predidit Hi,有什么更新吗?我迫不及待想看到新功能上线了。

@Predidit

Copy link
Copy Markdown
Owner

抱歉,我错过了这个PR,感谢你的提醒,我会在明天仔细检查这个PR

@Predidit
Predidit merged commit a2e5a58 into Predidit:main Sep 21, 2026
2 of 3 checks passed
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.

2 participants