Skip to content

feat: 重构歌曲评论为独立页与全屏内嵌面板 - #390

Open
WindDrift wants to merge 10 commits into
SPlayer-Dev:devfrom
WindDrift:feat/comments-v2
Open

WindDrift wants to merge 10 commits into
SPlayer-Dev:devfrom
WindDrift:feat/comments-v2

Conversation

@WindDrift

Copy link
Copy Markdown

改动类型

  • 新功能(feat)
  • 缺陷修复(fix)
  • 重构 / 优化(不改变对外行为)
  • 文档(docs)
  • 其他

是否包含破坏性变更

  • 是
  • 否

改动说明

旧的全局评论弹窗只能显示首页、翻不出第二页,主创评论基本抓不到。本 PR 将其替换为「独立评论页 + 全屏播放器内嵌评论」,并修通分页链路。

  • 交互:右键「查看评论」改为路由跳转 comments/:source/:id;全屏播放器评论按钮改为内嵌面板(有歌词占左半屏,无歌词铺满),与播放队列互斥;瀑布流 + 无限滚动
  • 主创说:匹配歌手绑定的网易云账号,抓取其在评论区的发言及回复;热评按需翻页、命中即停,热评无命中才补捞普通评论;失败时显示错误块与重试
  • 分页修复:网易云 legacy 端点的分页信号在顶层 more 而非 data.hasMore,原实现恒为 false 导致第二页起全部丢弃;MusicCommentPage 新增显式 hasMore;QQ 游标端到端接通
  • 清理:删除废弃评论弹窗与 store 字段,新增方法补齐中文 JSDoc

本 PR 为已关闭未合并的旧 PR #101 的重写实现,代码由 DeepSeek v4.1 Flash 1017 Max Effort 生成,由 Qwen 3.8 Max xhigh Effort 撰写计划和辅助审查。

关联 Issue

暂无

测试情况

  • pnpm typecheck / pnpm lint / pnpm test 全部通过(67 文件 / 517 用例,新增 41 条评论用例)
  • 5 个提交逐个 checkout 验证 0 类型错误

自查清单

  • 本 PR 只包含一个主要功能 / 修复,没有夹带无关改动
  • 已在本地完整测试通过;AI 生成的代码同样自行测试并审阅过
  • 已运行 prettier,并确认 pnpm typecheck、pnpm lint 通过
  • 涉及原生模块时已 pnpm build:native 验证
  • 已向 dev 分支提交

截图

image image image

@WindDrift WindDrift changed the title feat: 主进程评论服务新增主创说并修复分页判定 feat: 重构歌曲评论为独立页与全屏内嵌面板 Oct 17, 2026

@imsyy imsyy left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • 目前暂不考虑将非播放页的评论转为单独页面,会造成强打断当前用户操作,后续会基于弹窗优化评论展示以及翻页相关体验
  • 暂不考虑添加主创说相关功能,会增加额外的接口请求以及不必要的内容缓存
  • 播放器内评论样式待后续布局优化完成后处理

This branch has not been deployed

No deployments
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