Skip to content

feat(web): 在 Diff 行旁连续添加审阅意见到聊天草稿 - #471

Merged
Qiyuanqiii merged 3 commits into
mainfrom
Qiyuanqiii/diff-inline-feedback
Oct 27, 2026
Merged

Qiyuanqiii merged 3 commits into
mainfrom
Qiyuanqiii/diff-inline-feedback

Conversation

@Qiyuanqiii

@Qiyuanqiii Qiyuanqiii commented Oct 22, 2026 •

Copy link
Copy Markdown
Member

改动

审阅多处代码时,可以在 Diff 行旁点击 + 就地写意见,连续加入已有聊天草稿,再返回聊天检查并发送。添加意见后审阅面板保持打开,原有正文和附件保留;每条意见带上原始文件、编辑版本、差异片段与对应行号。输入时冻结来源,避免刷新后绑定到另一行。尚未加入草稿的行内输入仅保留在当前审阅视图。

#445 已合入 main,本 PR 的 base 已是 main。实现提交 803ac30f 已同步 #445 的文件名旁紧凑换行按钮,并保留逐行评论;没有旧版独立换行工具栏。原有文件、片段和行引用继续可用。

交互参考原设计核对的 Claude Code Desktop Diff 审阅和 Codex 行内反馈:就地连续添加意见,回到聊天显式发送。Enter 保留多行输入,Ctrl/⌘+Enter 加入草稿。

最新真实截图(2026-10-27)

旧正文的截图确实没有随 #445 更新,现已替换。变更前是已合入 #445 的 main 02ff2bcf,变更后是本 PR 实现 803ac30f。两边使用同一个真实 Go API、SQLite、项目和文件提案,分别构建生产前端,均开启长行换行。Windows / Edge 156 实拍。

视口 变更前:#445 已合入 变更后:#471 行内评论
1280 × 900 桌面变更前 桌面变更后
480 × 900 窄屏变更前 窄屏变更后

当前截图、源码版本与验证记录。

验证

  • 本次两个版本的 TypeScript 检查、生产构建通过,Go journey 后端构建通过。
  • 实际页面验证:切换换行保留未提交评论与代码原文;480px 页面无横向越界;连续两条意见保留已有草稿,正确绑定第 2、3 行及编辑记录,返回后聚焦草稿,重新加载后仍保留。
  • 同一实现合流时三个相关测试文件 26 项通过,更新后的 task-overview 6 项随后单独复跑通过。本次新增提交 fb19e3fd 只更新截图和说明。
  • 此轮是 Windows Edge 的生产页面验收;打包后的原生窗口、Git 预览与真实中文输入法没有在本次重测。2026-10-22 的旧截图与验收记录保留为历史资料。

本 PR 尚待最新提交的 CI 与合并审阅。

Preserve the original diff source and existing draft while reviewing multiple lines. Keep IME cancellation inside the inline editor.
Base automatically changed from Qiyuanqiii/review-diff-line-wrap to main October 27, 2026 05:52
@Qiyuanqiii
Qiyuanqiii merged commit 560d947 into main Oct 27, 2026
1 check passed
@Qiyuanqiii
Qiyuanqiii deleted the Qiyuanqiii/diff-inline-feedback branch October 27, 2026 06:13
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.

1 participant