Skip to content

feat(diff): 按 commit 查看变更范围 + 详情面板与评审界面交互优化#81

Merged
huhamhire merged 4 commits into
devfrom
feat/commit-diff-scope
Jun 19, 2026
Merged

feat(diff): 按 commit 查看变更范围 + 详情面板与评审界面交互优化#81
huhamhire merged 4 commits into
devfrom
feat/commit-diff-scope

Conversation

@huhamhire

Copy link
Copy Markdown
Owner

背景

体验优化里程碑的又一批 PR 评审界面改进,覆盖 Diff 标签的「按 commit 查看」能力,以及详情面板、合并按钮、提交/活动视图等多处交互细节。

改动

Diff 按 commit 查看变更范围

  • 文件树头部「 个文件」补充范围信息:「· 全部变更」或「· 」,整体可点击弹出范围下拉(参考 Bitbucket 两行布局:图标 + 标题 + 元信息),选「全部变更(PR base..head)」或某 commit 的 parent..sha。下拉 portal + fixed 定位,避免被文件列表 overflow 裁切 / 被 Monaco 盖住。
  • 提交 / 活动标签页点击 commit 不再跳浏览器,改为切到 Diff 标签本地渲染该 commit 变更(只读 diff)。
  • diff:listChangedFiles / getFileContent / getBlame 增加可选 base/head 范围参数。

PR 详情标签页

  • 整面板国际化(原「描述 / 时间线」中文、「Reviewers」英文写死);reviewer 行式展示(图标 + 头像 + 名 + 决断 chip);时间线精简为「远端创建 / 远端更新 / 最近更新时间」;左右布局 + 容器查询响应式(窄屏堆叠);侧栏限宽、时间小字号右对齐。

评审界面交互细节

  • 合并按钮去「常绿填充」→ 基础态 + 1s blink 突出可点击,点击后 disabled 灰显。
  • 提交表格行高加高、表头字号不小于正文。
  • 活动视图 inline 评论「文件:行号」锚点可点击跳到 Diff 对应位置。
  • 活动内容区宽度 [480, 960] 自适应、窄于 480 转横向滚动(修 ChatPane 遮盖)。
  • PR 头部「冲突」标记改为带色 chip。

删除行评论

  • Diff 支持给删除行新增行内评论 / 草稿:并排视图下 base 侧(删除 / 上下文行)可 hover「+」创建,锚定 side: 'old'(发布映射 Bitbucket lineType: removed / fileType: FROM)。统一视图删除行为 view zone,仍需切并排视图创建。

验证

lint / typecheck / build 全通过。repo-mirror 的 5 个 git-CLI 用例为既有环境失败,与本次无关。

🤖 Generated with Claude Code

huhamhire and others added 4 commits June 19, 2026 14:33
- 文件树头部「<n> 个文件」补充范围信息,变为「· 全部变更」或「· <commit>」,整体可
  点击弹出范围下拉(参考 Bitbucket 两行布局:图标 + 标题 + 元信息),选「全部变更」或
  某 commit 的 parent..sha。下拉用 portal + fixed 定位,避免被文件列表 overflow 裁切 /
  被 Monaco 盖住;菜单整体拉宽,commit 标题超宽省略。
- 提交 / 活动标签页点击 commit 不再跳浏览器,改为切到 Diff 标签本地渲染该 commit 变更。
- commit 视图为只读 diff(行内评论 / 草稿锚定在 PR 全量 diff 行号上,不套用于单 commit)。
- `diff:listChangedFiles` / `getFileContent` / `getBlame` 增加可选 base/head 范围参数。
- 提交面板时间列接入 hover 精确时间提示、表格限宽 960 居中,与详情 / 活动页一致。

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
- 整面板国际化:原「描述 / 时间线」写死中文、「Reviewers」写死英文,现按界面语言
  出文案(新增 prInfo 文案集,四语言)。
- reviewer 列表参照活动时间线行式展示:前置状态图标 + 头像 + 名 + 决断 chip
  (评审者 / 已批准 / 要求修改 / 待评审),替换原 disc bullet。
- 时间线精简为「远端创建 / 远端更新 / 最近更新时间」,移除「本地首次发现」。
- 改为左右布局(左描述、右时间线 + 评审者),按容器查询响应式:面板窄到阈值时侧栏
  堆叠到描述下方;侧栏限宽 400px、时间小字号右对齐,避免元素过散。

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
- 合并按钮:去掉「常绿填充」(易误判为已点击 / 已合并),改为与 approve 同款基础态
  (描边 + 绿图标)+ 1s blink 突出可点击;点击后沿用 .btn:disabled 灰显并停闪。
- 提交标签页:表格行高加高(td/th 纵向 padding 10px)、表头字号不小于正文($fs-md)。
- 活动视图:inline 评论「文件:行号」锚点可点击,经 pendingDiffNav 跳到 Diff 对应位置
  (新增 commentsPanel.anchorJumpTitle)。
- 活动内容区:宽度在 [480, 960] 内自适应缩放,窄于 480 转外层横向滚动;并给 tab-pane /
  comments-panel 补 min-width:0,修正窄宽下被 ChatPane 遮盖。
- PR 头部「冲突」标记改为带色 chip(琥珀 tint)展示。

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
此前 hover「+」只挂在 head 侧(新增 / 上下文行),无法在删除行起评论。把 hover-「+」
抽成 wireAdder(editor, side, occupied):modified 编辑器走 side:'new'(始终可点)、original
编辑器走 side:'old'(仅并排视图挂载,删除 / 上下文行可点);占用行按侧分桶、policy 按对应
side 判定。锚定 side:'old' 经现有发布路径映射为 Bitbucket lineType:removed / fileType:FROM。

显示 / 策略 / 发布三条链路此前已支持 old 侧,仅创建入口缺失。统一(inline)视图删除行是
view zone、无可 hover 行号,仍需切并排视图创建。

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@huhamhire huhamhire added this to the 体验优化 milestone Jun 19, 2026
@qodo-code-review

Copy link
Copy Markdown

Qodo reviews are paused for this user.

Troubleshooting steps vary by plan Learn more →

On a Teams plan?
Reviews resume once this user has a paid seat and their Git account is linked in Qodo.
Link Git account →

Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center?
These require an Enterprise plan - Contact us
Contact us →

@huhamhire huhamhire added the enhancement New feature or request label Jun 19, 2026
@huhamhire huhamhire merged commit 65d2169 into dev Jun 19, 2026
1 check passed
@huhamhire huhamhire deleted the feat/commit-diff-scope branch June 19, 2026 07:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant