Skip to content

fix(macos): 修复 Git Log 筛选文字重叠 (#1342) - #1489

Merged
xiaoyumuxi merged 2 commits into
1lck:previewfrom
Mucheen:codex/issue-1342-git-log-filter-label
Oct 25, 2026
Merged

xiaoyumuxi merged 2 commits into
1lck:previewfrom
Mucheen:codex/issue-1342-git-log-filter-label

Conversation

@Mucheen

@Mucheen Mucheen commented Oct 25, 2026 •

Copy link
Copy Markdown
Collaborator

macOS Git Log 选择用户后,窄工具栏会把筛选名称与选中值分别压缩,造成文字挤压重叠。现在用同一段单行文字布局并统一尾部截断,让名称保持可读。

  • Branch、User、Date、Paths 共用修复,保留名称/选中值颜色、动态值原文与固定选项本地化、4pt 留白、下拉与清除行为。
  • 新增明暗主题的实际渲染回归,覆盖普通及长用户名、中文、长分支和路径;更新现有 Agent Note 与 Git 历史功能矩阵。
  • 同步最新 preview,复用已合入的 AgentDesktopCardView 媒体权限拒绝委托和覆盖清单修复,解决之前 Swift CI 中唯一失败的 WebMediaCaptureCoverageTests。PR 相对 preview 的功能差异仍为上述四个文件。

验证:

  • 76 项相关 Swift 测试通过,包含 WebMediaCapture、AgentDesktopCard、Git Log 与菜单/工作台回归。权限覆盖扫描 329ms,新增渲染回归 79ms;同一渲染断言在旧实现的明暗两组场景中共报告 10 处失败。
  • 按 CI 范围执行 3,125 项 Swift 单元测试:3,124 项通过;既有 draggedPopupKeepsItsTopLeftDuringLayoutAndResetsOnReopen 在本机 1280×720 屏幕上因窗口顶边被可见区域限制为 690pt(测试预期 716pt)失败,单独执行同样复现。该测试及其产品实现与 preview 完全一致。
  • 测试稳定性、运行时 bundle 只读、Agent Notes、平台功能矩阵及变更门禁通过。服务边界脚本发现 preview 既有 RunLaunchEnvironment.swift:50 的系统 PATH 字面值;该文件不在本 PR 差异中。
  • 当前分支预览曾构建并启动;完整 Git Log 手动验收仍待完成。PR 保持 Open、可评审状态。

Fixes #1342

@ghfind-review ghfind-review Bot added the review: medium ghfind author score; see https://ghfind.com label Oct 25, 2026
@xiaoyumuxi
xiaoyumuxi marked this pull request as ready for review October 25, 2026 10:26
@lithe-code-review

lithe-code-review Bot commented Oct 25, 2026 •

Copy link
Copy Markdown

Lithe Review

结论: ⚠️ 有建议
范围: preview ← codex/issue-1342-git-log-filter-label · head: 9a77f5f

未发现需要阻塞合并的问题。

Scope Check

  • 关联 [Bug] git显示问题 #1342:改动针对筛选用户后文字挤压重叠的问题,并覆盖共用标签的 Branch、User、Date、Paths。
  • 已检查全部 4 个变更文件及直接调用链。单段文字布局保留名称和值的颜色、本地化分支、留白及原有下拉和清除入口。
  • 修改位于 macOS 视图层,复用 SwiftUI 文字布局及现有下拉组件,未引入并行领域实现;Agent Note 和功能矩阵同步更新,验证状态保留 pending。
  • 完整 Git Log 手动验收仍待完成,当前证据不足以确认实际界面效果。

Suggestions

  1. 请在 PR 描述或评论中补充受影响工具栏的修复前后对比截图,展示明暗主题下窄宽度、长用户名的布局及清除按钮。现有上下文只有 Issue 的原问题截图链接,没有修复后的对比截图;视觉效果尚未验证。

Summary

修复范围集中,布局方式与问题原因一致。新增渲染测试能检查名称前缀是否保持可读,但组件测试不能代替四项筛选并列时的布局、下拉与清除操作验收。

Verification

  • 静态检查 a9264ca...9a77f5f 的全部差异:GitLogView.swift、GitLogFilterLabelTests.swift、git-history.json 及共享下拉样式 Agent Note;追踪了四项筛选调用、清除入口及字体和颜色定义。
  • 作者报告:49 项相关 Swift 测试通过、新增渲染回归耗时 62ms、旧实现出现 10 处断言失败;本次未独立复验。
  • 已提供的 Check Run 中,macOS CI gate、Agent Notes 和矩阵检查有成功记录;Swift tests 为 skipped,另有 queued/cancelled 检查,不能据此确认完整 CI 已通过。
  • 本次未运行测试、构建、校验脚本或应用,未访问网络、读取截图。剩余非阻塞验证缺口为完整 macOS 工作流验收及修复前后截图。

@xiaoyumuxi
xiaoyumuxi merged commit 80101d8 into 1lck:preview Oct 25, 2026
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

review: medium ghfind author score; see https://ghfind.com

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants