Skip to content

fix(desktop): 未登录时账户菜单显示登录 - #3973

Merged
MagicLizi merged 1 commit into
mainfrom
dash/fix-signed-out-login-label
Sep 6, 2026
Merged

fix(desktop): 未登录时账户菜单显示登录#3973
MagicLizi merged 1 commit into
mainfrom
dash/fix-signed-out-login-label

Conversation

@dashhuang

Copy link
Copy Markdown
Member

这次改了什么

摘要

未登录时,桌面侧边栏账户菜单误显示“登录更多账号”。改为复用已有五语 login.signIn 文案,简体中文显示“登录”。

变更类型

  • feat 新功能
  • fix 缺陷修复
  • refactor / perf 重构或性能优化
  • docs / test / chore 文档、测试或工程维护
  • 其他:

范围

  • 关联 Issue / 需求:用户反馈未登录账户菜单文案不准确。
  • 本 PR 包含:替换未登录菜单项的翻译 key,并同步现有测试断言。
  • 明确不包含:登录流程、账号切换逻辑及已登录状态的添加账号入口。
  • 用户可见变化:未登录时显示“登录”。
  • 是否存在 breaking change:无。

UI 变化

Desktop 侧边栏账户菜单:“登录更多账号” → “登录”。复用现有五语翻译、菜单与主题样式。

  • 引用的设计规范:docs/design-rules/DESIGN.md §10 Theme System & Token Reference、§11 Voice & Content;沿用现有 Light/Dark 语义样式,使用与当前登录状态一致的简洁动作文案。

怎么验证的

自动验证

以下检查均通过,复用本次改动完成后且代码未再变化的结果:

pnpm test:unit:related
pnpm --filter desktop run --if-present typecheck
pnpm --filter desktop exec prettier --check src/renderer/components/sidebar/UserInfoSection.tsx src/renderer/__tests__/userInfoSectionMobileDownload.test.tsx src/renderer/__tests__/userInfoSectionHover.test.ts
pnpm check:i18n
pnpm check:i18n-glossary
git diff --check
pnpm check:dco

相关测试覆盖未登录菜单项、菜单顺序及点击后仍进入原登录流程。国际化检查仅有存量非阻断警告。

手工验证

已检查完整 diff 和五语 login.signIn 翻译。

未执行的验证

未启动 Desktop 做 Light/Dark 实机目检或截图;本次只替换已有翻译 key,不改布局或样式。未跑全量单测,由 CI 执行。

风险

风险分类

  • 无已知风险
  • SQLite / migration
  • system prompt
  • 协议兼容
  • 权限 / 安全 / 用户数据
  • 存量插件兼容(批准状态 / 指纹 / manifest 校验 / 安装布局 / 包格式)
  • 原生层 / fingerprint / OTA
  • 跨平台差异
  • 其他:

影响与回滚

  • 影响范围:Desktop 未登录时的账户菜单文案。
  • 回滚 / 降级方式:还原本次翻译 key 与测试断言即可。

提交前检查

  • 已 review 完整 diff
  • 每个 commit 都带 DCO 签名(git commit -s,见 DCO
  • UI 改动已在「UI 变化」注明引用的设计规范章节(不涉及 UI 则跳过)
  • 未提交凭证、令牌或授权文件
  • 已补充必要文档(本次无需额外文档)
  • 已确认测试结果或说明未执行原因

Signed-off-by: Dash <125997726+dashhuang@users.noreply.github.com>
@dashhuang
dashhuang requested a review from a team as a code owner September 5, 2026 10:26
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 5, 2026

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review Completed 2026-09-05T10:28:55.327019Z ab9da32 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to 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" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@greptile-apps

greptile-apps Bot commented Sep 5, 2026

Copy link
Copy Markdown

Greptile Summary

此 PR 修正了 Desktop 侧边栏在未登录状态下的账户菜单文案:

  • 将“添加更多账号”对应的翻译键替换为已有的 login.signIn
  • 同步更新菜单顺序和点击导航相关测试。
  • 不改变菜单显示条件、登录流程或 /add-account 导航行为。

Confidence Score: 5/5

此 PR 看起来可以安全合并,文案变更与实际未登录状态一致,相关测试也已同步更新。

未发现由本次改动引入的行为、安全或本地化问题;五种受支持语言均包含目标翻译键,菜单显示条件和点击后的登录导航保持不变。

Important Files Changed

Filename Overview
apps/desktop/src/renderer/components/sidebar/UserInfoSection.tsx 未登录的本地模式账户菜单改用五语齐全的“登录”翻译键,原有显示条件和导航行为保持不变。
apps/desktop/src/renderer/tests/userInfoSectionMobileDownload.test.tsx 更新未登录菜单项的名称、变量和断言,同时继续覆盖菜单顺序及登录入口导航。
apps/desktop/src/renderer/tests/userInfoSectionHover.test.ts 更新源码结构测试,使其检查新的登录翻译键仍位于设置项之前。

Reviews (1): Last reviewed commit: "fix(desktop): 未登录时账户菜单显示登录" | Re-trigger Greptile

@MagicLizi MagicLizi added the touches:product-ui 改动碰到产品 / UI 面(review-pr 自动维护,仅展示) label Sep 6, 2026
@MagicLizi

Copy link
Copy Markdown
Contributor

命中 UI 路径(apps/desktop/src/renderer/components/sidebar/UserInfoSection.tsx)但 description 未附界面效果证据——建议补充改动后效果:截图/录屏,或改动后界面的 HTML 页面(```html 代码块、.html 附件或在线预览链接),便于确认界面符合 DESIGN.md 设计规范

@MagicLizi MagicLizi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

代码重审通过:未登录账户菜单改为「登录」,当前 head 没有 P0/P1,conversation 已 resolve。

@MagicLizi
MagicLizi merged commit 2051f6b into main Sep 6, 2026
20 checks passed
@MagicLizi
MagicLizi deleted the dash/fix-signed-out-login-label branch September 6, 2026 02:54
@MagicLizi

Copy link
Copy Markdown
Contributor

合了。未登录时侧栏还写「添加账号」,点进去才发现其实是登录——现在菜单直接说登录,这条路径终于对得上。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

touches:product-ui 改动碰到产品 / UI 面(review-pr 自动维护,仅展示)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants