Skip to content

fix(mobile): 分享选择模式补标题行并对齐桌面文案 - #3755

Open
Michael-Yu2978 wants to merge 2 commits into
makecindy:mainfrom
Michael-Yu2978:codex/pr6-1
Open

fix(mobile): 分享选择模式补标题行并对齐桌面文案#3755
Michael-Yu2978 wants to merge 2 commits into
makecindy:mainfrom
Michael-Yu2978:codex/pr6-1

Conversation

@Michael-Yu2978

@Michael-Yu2978 Michael-Yu2978 commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

这次改了什么

摘要

移动端分享选择模式(底部操作条)原先只有 [关闭][已选 N 条][分享],没有标题头;桌面端同一入口渲染 title(分享对话)+ subtitle(已选 N 条消息)两行。本 PR 把规划好但一直没接线的死 key session.shareImage.title(五个语言文件都有,rg 全仓零引用)接成真实 UI,并新增 session.shareImage.subtitle 复数 key,文案与桌面逐字对齐。

变更类型

  • fix 缺陷修复

范围

  • 关联 Issue / 需求:无
  • 本 PR 包含:mobile ShareSelectionBar 增加标题行;五个 locale 新增 session.shareImage.subtitle 复数 key;删除随之闲置的 selectedCount key;en title 大小写对齐桌面;补充 UI 复刻证据(docs/design-previews/share-selection-bar-title/
  • 明确不包含:桌面端任何改动;分享导出 / 截图链路
  • 用户可见变化:移动端分享选择模式底部多出「分享对话」标题;已选数量行文案与桌面一致(zh-CN 由「已选择 N 条消息」改为「已选 N 条消息」,en 由「N selected」改为「N message(s) selected」)
  • 是否存在 breaking change:无

UI 变化

  • 引用的设计规范:DESIGN.md §3 Typography Rules(字号/字重走两端共享 token 梯子)与 §11 Voice & Content(微文案语气),移动端配套 apps/mobile/docs/mobile-design-guide.md(文字层级:主=textPrimary、次=textSecondary;字号角色:正文 body、metadata/标签 caption)。标题行用 textPrimary / typeScale.body / fontWeight.medium,副标题用 textSecondary / typeScale.caption,颜色全部走语义 token,Light/Dark 双模式由 token 系统同源覆盖;两行布局与桌面 ShareSelectionBar 的 title + subtitle 结构对齐。

怎么验证的

自动验证

corepack pnpm check:i18n                                    ✅ 8525 key 全部一致(1045 处存量警告)
corepack pnpm check:i18n-glossary                           ✅ 无新增违规(harness/lead 为存量 proposed 告警)
corepack pnpm --filter mobile exec vitest run \
  src/__tests__/i18nCatalogParity.test.ts \
  src/__tests__/conversationShareWebViewHtml.test.ts        ✅ 25 tests 全过
corepack pnpm --filter mobile run typecheck                 ✅
corepack pnpm test:unit:related                             ✅ mobile related 全过

注:环境默认 pnpm 为 11.x,与仓库 engines.pnpm(>=10.7 <11)冲突,上述命令均用 corepack pnpm(10.33.2,即 packageManager 锁定版本)执行。

手工验证

渲染复刻证据(明确标注:非真机/模拟器截图,是基于真实 token 的复刻渲染):

  • docs/design-previews/share-selection-bar-title/evidence/light.png(zh-CN Light)
  • docs/design-previews/share-selection-bar-title/evidence/dark.png(zh-CN Dark)
  • docs/design-previews/share-selection-bar-title/evidence/en-light.png(en Light,展示复数 2 messages selected
  • docs/design-previews/share-selection-bar-title/evidence/en-dark.png(en Dark)

复刻页 docs/design-previews/share-selection-bar-title/index.html 的配色/字号/间距全部取自 apps/mobile/src/theme/tokens.tslightColors / darkColorssurfaceTranslucentborderTranslucenttextPrimarytextSecondaryctactaTexttypeScale.body/captionlineHeight.body/captionspacingradius.pilliconSizeiconStroke),文案为本次改动的真实译文(zh-CN「分享对话 / 已选 2 条消息」,en「Share Conversation / 2 messages selected」)。

未执行的验证

  • 移动端真机 / 模拟器目检未执行:分享导出链路为 iOS-only(nativeConversationShareAvailable = Platform.OS === 'ios'),本机为 Windows、无 iOS 环境;也未安装 Android SDK / maestro,Android 上分享入口不可达,故无法产出真机截图。
  • 双模式(Light / Dark)未实机目检:复刻证据已覆盖两套语义 token 的实际渲染结果,按要求如实标注非实机。

风险

风险分类

  • 无已知风险

影响与回滚

  • 影响范围:仅移动端分享选择模式底部操作条的展示文案 / 排版;不触及原生模块、app.json / eas.json / config plugin,不改变 runtime fingerprint(无冷更)。
  • 回滚 / 降级方式:revert 本 commit 即可,无数据或状态变更。

提交前检查

  • 已 review 完整 diff
  • 每个 commit 都带 DCO 签名(git commit -s,本地 pnpm check:dco 通过)
  • UI 改动已在「UI 变化」注明引用的设计规范章节
  • 未提交凭证、令牌或授权文件
  • 已补充必要文档(含 UI 复刻证据,见「手工验证」)
  • 已确认测试结果或说明未执行原因

- ShareSelectionBar 增加标题行(session.shareImage.title),已选数量改走
  session.shareImage.subtitle 复数 key,文案对齐桌面 chat.shareImage
- 移除不再使用的 session.shareImage.selectedCount key
- en title 大小写对齐桌面(Share conversation → Share Conversation)

Signed-off-by: Michael-Yu2978 <ymt2978@163.com>
@Michael-Yu2978
Michael-Yu2978 requested a review from a team as a code owner September 1, 2026 18:04
@greptile-apps

greptile-apps Bot commented Sep 1, 2026

Copy link
Copy Markdown

Greptile Summary

移动端分享选择底栏新增标题行,并将已选消息数量改为与桌面一致的复数化副标题。

  • 更新五种语言的分享标题与数量文案
  • 调整标题、副标题的排版层级和语义颜色

Confidence Score: 5/5

未发现阻碍合并的问题,此 PR 看起来可以安全合并。

新增文案采用项目当前支持的复数键约定,双行底栏布局也保留了弹性高度和文本截断能力,没有发现可达的功能、安全或构建故障。

Important Files Changed

Filename Overview
apps/mobile/src/session/ShareSelectionBar.tsx 新增标题和副标题双行布局;现有弹性布局、行高及截断设置未显示出明确的功能问题。
apps/mobile/src/i18n/locales/en/session.json 将英文数量文案改为符合当前 i18next 配置的单复数键,并统一标题大小写。
apps/mobile/src/i18n/locales/ja/session.json 日文分享数量文案迁移到与其他语言一致的复数键结构。
apps/mobile/src/i18n/locales/ko/session.json 韩文分享数量文案迁移到与其他语言一致的复数键结构。
apps/mobile/src/i18n/locales/zh-CN/session.json 简体中文分享数量文案迁移到复数键并与桌面措辞对齐。
apps/mobile/src/i18n/locales/zh-TW/session.json 繁体中文分享数量文案迁移到复数键并与桌面措辞对齐。

Reviews (1): Last reviewed commit: "fix(mobile): 分享选择模式补标题行并对齐桌面文案" | Re-trigger Greptile

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

Copy link
Copy Markdown
Contributor

命中 UI 路径(apps/mobile/src/session/ShareSelectionBar.tsx)但 description 未附界面效果证据。建议补充改动后效果:截图/录屏,或改动后界面的 HTML 页面,便于确认界面符合 DESIGN.md。缺 UI 证据不阻断合并。

@MagicLizi

Copy link
Copy Markdown
Contributor

这个 PR 改了移动端分享选择模式的可见文案和布局(补标题行、对齐桌面副标题),需要维护者确认后再合并。

请维护者直接在本 PR 上 Approve;若要改,请 Request Changes。讨论 issue:#3756

@MagicLizi MagicLizi added the awaiting-discussion 等待维护者讨论(review-pr) label Sep 1, 2026
Signed-off-by: Michael Yu <michael.yu2978@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting-discussion 等待维护者讨论(review-pr) touches:product-ui 改动碰到产品 / UI 面(review-pr 自动维护,仅展示)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants