Skip to content

fix(web): 变更视图三处交互优化(删冗余标题行 / 转到文件按钮 / 折叠全部 diff) - #16

Merged
shaohaozhi286 merged 1 commit into
masterfrom
shz_vide/interface-design-fixes-28c216
Jul 11, 2026
Merged

shaohaozhi286 merged 1 commit into
masterfrom
shz_vide/interface-design-fixes-28c216

Conversation

@shaohaozhi286

Copy link
Copy Markdown
Collaborator

改动内容

Web UI 变更(diff)视图的三处交互修复:

  1. 删除详情栏顶部冗余标题行 — 中间详情栏「变更」页签原有的「变更 branch → origin/master +A -D ↻」整行移除(分支与基线信息在右侧 SidePanel 头部已有,纯重复)。+A -D 汇总与加载 spinner 并入「N 个文件已更改」行;右键菜单锚点 data-desktop-review-kind="summary" 迁移到新行,「展开/折叠/复制全部 diff」右键菜单行为不变。
  2. 文件行新增「转到文件」按钮 — 左侧详情栏(GitChangeReview)与右侧变更列表(GitChangesPanel)的每个文件行,hover / 选中时浮现 OpenFile 图标按钮,点击打开该文件的源码预览页签(VS Code 风格)。通路:复用 ChatView 已有的 openFilePreview,经 PreviewDetailsPanel / SidePanel 分别下传。
  3. 「折叠全部 diff」按钮 — 详情栏「N 个文件已更改」行最右端新增 CollapseAll 图标按钮,一键收起全部已展开的 diff(与右键菜单 COLLAPSE_ALL_DIFFS 同语义)。

审阅要点

  • 已删除(D)状态的文件磁盘上不存在,「转到文件」按钮渲染为隐形占位(is-hidden):不可点,同时保证各行 ± 计数列对齐。
  • 右侧文件行是 3 列 CSS grid,新增第 4 个子元素后补了 grid-auto-flow: column + grid-auto-columns: max-content,否则默认 row 流会把按钮折到第二行。非 git 的会话级 ChangesList 复用同一 class,仅 3 子元素,不受影响。
  • 折叠按钮未用 Tailwind ! important 前缀(v4 下行为不确定),改为专属 CSS 类 .ace-review-collapse-all-btn。
  • 详情栏原刷新按钮随标题行一并移除(用户明确要求整行干掉);手动刷新仍可从右侧 SidePanel 头部触发,回合结束 / checkout 后的自动失效重拉不受影响。

验证

  • pnpm build 通过;pnpm test(前端 Node 纯逻辑测试)全过。
  • 用隔离 daemon(独立端口/run-dir + --static-dir 指向新 dist)在浏览器逐项实测:标题行消失、两侧「转到文件」均打开源码预览页签、折叠全部一键收起。

🤖 Generated with Claude Code

- 详情栏「变更」页签删除顶部「变更 branch → base」标题行(与右侧
  面板头部重复),+A -D 汇总与加载 spinner 并入「N 个文件已更改」行
- 左右两侧变更文件行 hover 浮现「转到文件」按钮,点击直接打开文件
  内容预览页签;已删除(D)文件隐形占位保持 ± 计数对齐
- 「N 个文件已更改」行最右新增「折叠全部 diff」按钮

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@shaohaozhi286
shaohaozhi286 merged commit 5a2b5bd into master Jul 11, 2026
10 of 11 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