feat(desktop): allow hiding project source labels on pinned tasks - #3268
feat(desktop): allow hiding project source labels on pinned tasks#3268nothingyuancando wants to merge 3 commits into
Conversation
Signed-off-by: nothingyuancando <1579035512@qq.com>
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 6913874638
ℹ️ 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".
|
| Filename | Overview |
|---|---|
| apps/desktop/src/renderer/hooks/useSidebarCardMode.ts | 新增默认开启、支持显式覆盖及恢复默认的置顶来源标签偏好,并同步同窗口订阅者和其他窗口。 |
| apps/desktop/src/renderer/components/settings/AppearanceSection.tsx | 新增置顶任务来源标签开关及带可访问名称的恢复默认控件。 |
| apps/desktop/src/renderer/features/cc-agent/sidebar/sections/PinnedSection.tsx | 在卡片和文本两种置顶任务展示路径中按偏好传递项目来源标签。 |
| apps/desktop/src/renderer/hooks/tests/useSidebarCardMode.test.ts | 覆盖默认值、显式关闭、持久化、恢复默认和同窗口订阅同步。 |
| apps/desktop/src/renderer/components/settings/tests/AppearanceSection.accessibility.test.tsx | 验证恢复默认按钮具有可访问名称并正确表达不可用状态。 |
Reviews (3): Last reviewed commit: "fix(desktop): clarify pinned source labe..." | Re-trigger Greptile
|
这条 PR 需要维护者确认(产品 / UI:固定任务是否显示项目来源标签)。请维护者在本 PR 上 Approve;若要修改请 Request Changes,作者改完后再 Approve。讨论 issue:#3273 |
|
命中 UI 路径(apps/desktop/src/renderer/components/settings/AppearanceSection.tsx / apps/desktop/src/renderer/features/cc-agent/sidebar/sections/PinnedSection.tsx / apps/desktop/src/renderer/hooks/useSidebarCardMode.ts)但 description 未附界面效果证据——建议补充改动后效果:截图/录屏,或改动后界面的 HTML 页面(```html 代码块、.html 附件或在线预览链接),便于确认界面符合 DESIGN.md 设计规范 |
|
@nothingyuancando 👋 这个 PR 还有 1 条 review conversation 没 resolve(apps/desktop/src/renderer/hooks/useSidebarCardMode.ts),auto-review 因此暂时跳过、没法继续审查 / 合并。 如果你已经按评论改完或回应了,请到对应 thread 上点 Resolve conversation;全部 resolve 后,下一轮 auto-review 会自动重新审查这个 PR。 |
Signed-off-by: nothingyuancando <1579035512@qq.com>
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 34fbf2040c
ℹ️ 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: nothingyuancando <1579035512@qq.com>
dashhuang
left a comment
There was a problem hiding this comment.
目前不建议把「项目来源」做成仅针对置顶任务的独立开关。请收敛为统一的侧边栏任务信息设置:在侧边栏显示设置中新增「任务信息」分组,分别控制项目来源、远程连接来源、是否挂载自动任务等信息是否显示,并让同一套规则作用于所有任务,而不是只作用于置顶区。完成统一方案后再复核。
|
补充一下:如果这个统一的「任务信息」设置超出当前 PR 的范围,或者你不打算继续扩展,也可以留给我来做。我会把它作为后续需求记下来并另行处理,不需要为了保留这次提交而勉强扩大范围。 |
|
感谢补充。这个 PR 会保持当前范围,暂不继续扩展统一的「任务信息」设置;后续如需推进,我会新开 issue 跟踪。 |
MagicLizi
left a comment
There was a problem hiding this comment.
审查通过:置顶任务隐藏项目来源标签的设置项、默认恢复、跨窗口同步和五语 i18n 与 diff 一致,定向测试/typecheck/DCO 复核通过。无 P0/P1。本轮不合并:仍有其他 reviewer 的 Request Changes。
|
感谢补充。确认本 PR 继续保持当前范围,不扩展统一的「任务信息」设置;现有改动仅覆盖置顶任务的项目来源标签显示。统一任务信息(项目来源、远程来源、自动任务等)后续由你另行处理,我不会为了保留本次提交扩大范围。 |
|
@dashhuang 根据你补充的说明,本 PR 将保持当前范围,统一「任务信息」设置留作后续需求处理。当前 review 状态仍为 Request changes,导致 PR 无法合并;麻烦 dismiss 此次 review,或提交新的 Approve 以解除合并阻塞。谢谢。 |
MagicLizi
left a comment
There was a problem hiding this comment.
审查通过:没有 P0/P1。
外观开关默认开、关掉只藏置顶来源标签;恢复默认删除 override 而不是写死 true。五语文案与术语表一致。dashhuang 的 CHANGES_REQUESTED 未代 dismiss。
|
这次 Request Changes 仍然有效,不需要 dismiss:产品结论是“不接受只针对置顶任务的独立开关”。如果不在本 PR 扩成统一的任务信息设置,就请保持本 PR 不合入;统一方案可另起需求由维护者后续实现。 |
这次改了什么
摘要
为置顶任务增加“显示项目来源”外观设置。用户关闭后,置顶任务继续保留原有标题、状态和操作,但不再显示项目来源标签,从而释放横向空间给任务标题;默认开启,保持现有用户的展示不变。
根据 review 补充“恢复默认”:恢复操作会删除
sidebar.pinnedSourceLabelVisibleoverride,重新跟随当前默认值,并同步同窗口和其他窗口的订阅者。变更类型
feat新功能fix缺陷修复refactor/perf重构或性能优化docs/test/chore文档、测试或工程维护范围
localStorageoverride 持久化和跨窗口同步、置顶区域的标签渲染门控、五语文案和回归测试。UI 变化
docs/design-rules/DESIGN.md的 Settings 行密度、语义颜色和双模式交付要求;复用现有Switch、图标恢复按钮和设置行样式。AppearanceSection.tsx。改动后界面(HTML)
恢复默认按钮只在用户已有显式 override 时可用;点击后删除 override 并恢复当前版本的默认显示。所有颜色继续使用既有语义 token,因此同一结构同时适用于 Light / Dark。
怎么验证的
自动验证
手工验证
未执行:本轮未启动 Desktop,因此未做 Light/Dark 实机检查。
未执行的验证
Electron 补齐后需重跑完整 related unit gate;需在 Light/Dark 下确认恢复默认、窄侧边栏标题空间和跨窗口同步。
风险
风险分类
影响与回滚
提交前检查
git commit -s,见 DCO)