Skip to content

fix(macos): 修复插件市场详情越过分隔线 - #1447

Merged
xiaoyumuxi merged 2 commits into
1lck:previewfrom
Mucheen:codex/plugin-marketplace-split-bounds
Oct 23, 2026
Merged

xiaoyumuxi merged 2 commits into
1lck:previewfrom
Mucheen:codex/plugin-marketplace-split-bounds

Conversation

@Mucheen

@Mucheen Mucheen commented Oct 23, 2026

Copy link
Copy Markdown
Collaborator

插件市场右侧详情在窗口变窄或拖动分隔条后,会被内容最小宽度撑开并向左偏移,导致分隔线穿过 Cargo 图标、名称和说明,详情遮住左侧插件列表。之前 #1420 覆盖的是“已安装”详情,这个市场入口使用另一套视图。

本次调整:

  • 让市场详情的滚动视口严格使用分栏分配的宽度,并裁剪到右栏边界;宽面板中的正文正常换行,极窄面板允许横向滚动查看完整内容。
  • 从现有内边距、元数据标签、来源选择器和提示卡尺寸推导内容最小宽度,保留来源选择器的可用空间;窗口高度变化后,短详情仍保持顶部对齐。
  • 继续使用共享分隔条及原生拖动路径。几何测量局限于详情视口,不在拖动期间刷新索引或构建业务模型。
  • 为名称、安装按钮和说明补充辅助功能标识,新增真实市场页的原生布局回归:中英文、深浅主题、不同窗口宽高、鼠标拖动、释放后刷新不移位,以及横向滚动到文档末端。测试使用私有偏好域,并清理窗口和偏好。
  • 更新插件市场 Agent Note、跨平台能力记录,并加入相同测试条件下的深浅主题前后截图与复现命令:布局验证记录。整体产品实机验收状态仍保持 pending。

验证结果:

  • macOS 相关 6 个测试套件共 66 项测试通过,覆盖市场布局、已安装详情、插件设置路径、市场 FeatureModel、仓库呈现与共享菜单;新增布局用例含 4 个参数组合,耗时 829ms,无超时或残留测试进程。
  • 同一份新增回归在 preview 1fba19412303ae120dad46831be38c04fd4189c4 的原布局上失败(仅补齐相同辅助功能标识),修复后通过。
  • 已通过测试稳定性静态检查、服务边界检查、Agent Notes 校验、运行时 bundle 只读检查,以及能力矩阵完整校验和上述精确 base/head 的变更门禁。
  • 截图来自隔离测试宿主的原生市场视图;未运行完整发行应用验收。本次只修改 macOS SwiftUI 布局,未新增运行时文件、缓存或下载资源。

本地测试命令:

./.agents/skills/write-stable-tests/scripts/test-stability-macos.sh -- \
  --filter 'PluginMarketplaceLayoutTests|PluginDetailLayoutTests|PluginSettingsLayoutTests|PluginMarketplaceFeatureModelTests|ContextMenuCoverageTests|PluginRepositoryPresentationTests'

@Mucheen
Mucheen marked this pull request as ready for review October 23, 2026 08:06
@lithe-code-review

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

Copy link
Copy Markdown

Lithe Review

结论: ⚠️ 有建议
范围: preview ← codex/plugin-marketplace-split-bounds · head: aeeb182

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

Scope Check

  • 关联 Issue 或需求是否完成:未找到可验证的关联 Issue;按 PR 描述,本次针对市场详情越过分隔线的回归。
  • 已完成范围:详情视口受分栏宽度约束;窄视口保留横向滚动;新增真实页面布局回归,并更新 Agent Note 与功能矩阵证据。
  • 未完成范围:完整发行应用的实机验收仍为 pending;本次未独立验证截图中的视觉效果。
  • 是否复用了已有上游能力,或对自研边界给出充分证据:不适用;改动属于 SwiftUI 页面布局,没有新增 IDE 基础能力。
  • 每个 Finding 是否由本次变更引入或加剧:无代码 Finding。
  • 是否属于本次变更职责范围:检查范围限于给定完整清单中的五个语义文件;四个截图资源未读取。

Suggestions

  1. PR 正文链接了含前后截图的验证文档,但没有直接附上截图。请在 PR 描述或评论中直接附上受影响界面的前后对比,方便评审者确认视觉效果。

Summary

布局改动将详情内容宽度与滚动视口分开,并保留共享分隔条实现;新增测试覆盖窗口尺寸变化、原生拖动、刷新后位置稳定和横向滚动边界。未发现明确的正确性或架构问题。未发现需要阻塞合并的问题。

Verification

  • 实际检查:给定 base...head 中的 PluginMarketplaceView.swift、PluginMarketplaceLayoutTests.swift、插件市场 Agent Note、布局验证文档和功能矩阵;并追踪了共享分隔布局、滚动视图样式及其调用路径。
  • 已提供结果:PR 描述报告 macOS 相关测试套件 66 项通过及静态检查通过;提供的 head Check Run 状态仍有 queued / in_progress 项,不能据此认定 CI 已完成。
  • 本次未运行测试、脚本或应用,符合只读审查范围。
  • 非阻塞验证缺口:完整应用实机验收仍为 pending;截图资源未作视觉检查,界面效果尚未由本审查独立确认。

@ghfind-review ghfind-review Bot added the review: medium ghfind author score; see https://ghfind.com label Oct 23, 2026
@Mucheen

Mucheen commented Oct 23, 2026

Copy link
Copy Markdown
Collaborator Author

@xiaoyumuxi 合并一下

@xiaoyumuxi
xiaoyumuxi merged commit c1a1f98 into 1lck:preview Oct 23, 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