fix(desktop): 避免步骤胶囊与底部导航重叠 - #3319
Conversation
Signed-off-by: david <david@xd.com>
|
| Filename | Overview |
|---|---|
| apps/desktop/src/renderer/components/chat/messageStreamIndicatorPosition.ts | 新增中央区域和计划浮层的边界测量及观察目标收集,完整覆盖了此前报告的浮层避让问题。 |
| apps/desktop/src/renderer/features/cc-agent/CCAgentSessionView.tsx | 同步观察动态挂载、卸载和尺寸变化的中央控件,并将实测避让偏移传给消息流。 |
| apps/desktop/src/renderer/components/chat/MessageStream.tsx | 改用底部中央实际占用边界计算导航胶囊位置。 |
| apps/desktop/src/renderer/components/chat/tests/messageStreamIndicatorPosition.test.ts | 覆盖空中央区域、步骤胶囊、展开计划浮层及观察目标选择。 |
| apps/desktop/src/renderer/tests/controlledBannerPlacement.test.ts | 补充动态浮层挂载监听的结构回归断言。 |
Reviews (2): Last reviewed commit: "fix(desktop): 避让展开的计划详情浮层" | Re-trigger Greptile
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: ff6986facd
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
Signed-off-by: DavidShen <david@xd.com>
反馈已处理已处理:
变更:
验证:
备注:
|
|
命中 UI 路径(MessageStream / messageStreamIndicatorPosition / CCAgentSessionView)但 description 未附界面效果证据——建议补充改动后效果:截图/录屏,或改动后界面的 HTML 页面,便于确认界面符合 DESIGN.md 设计规范。本条不阻断合并。 |
反馈已处理已处理:
变更:
验证:
备注:
|
|
合了。步骤胶囊跟着计划浮层一起量高度,底部导航终于不用再跟中间那条叠在一起。 |
这次改了什么
摘要
修复任务执行期间,composer 上方的步骤胶囊会与“跳到底部”或新消息提示重叠的问题。
原实现只按输入框栈顶计算底部导航位置,没有把动态出现的中央步骤/接管胶囊算进避让边界;同时中央行内容变化不一定改变外层 overlay 高度,原 ResizeObserver 也可能收不到更新。本次改为测量底部中央区域最上方的实际占用层,并同时观察中央胶囊行与输入框栈,使底部导航始终叠放在它们上方。
变更类型
feat新功能fix缺陷修复refactor/perf重构或性能优化docs/test/chore文档、测试或工程维护范围
UI 变化
docs/design-rules/DESIGN.md§5 Spacing System:使用现有 6px 间距尺度。docs/design-rules/DESIGN.md§8 Desktop Window:聊天流与 composer 随窗口布局动态重排,控件尺寸不缩减。docs/design-rules/DESIGN.md§10 Light / Dark Dual-Mode Delivery Gate:本次仅调整几何定位,不增加颜色或单主题条件分支,继续复用现有语义 token;实际目检完成 Light,Dark 未单独目检。怎么验证的
自动验证
手工验证
pnpm restart:desktop:remote --region=global -- --isolated=@worktree,得到DESKTOP_DEV_VERDICT=ready。#/cc-agent,侧栏显示“未登录”。overlap=false;Light 模式截图已本地目检。未执行的验证
风险
风险分类
影响与回滚
提交前检查
git commit -s,见 DCO)