Skip to content

fix(i18n): key ObjectView 的 create/edit/view 表单标题 (#3462) - #3468

Merged
yinlianghui merged 1 commit into
mainfrom
claude/issue-3462-form-title-i18n
Aug 6, 2026
Merged

fix(i18n): key ObjectView 的 create/edit/view 表单标题 (#3462)#3468
yinlianghui merged 1 commit into
mainfrom
claude/issue-3462-form-title-i18n

Conversation

@yinlianghui

Copy link
Copy Markdown
Collaborator

Fixes #3462

#3426 / PR #3457#3459 / PR #3464 同族的又一个调用点。

前提复核(先于实现)

origin/main (84bd39e) 上确认 issue 前提仍然成立:getFormTitle()(行号已漂到 903)仍在用模板字符串自拼三个英文动词,三个消费点也都还读它 —— renderDrawerFormDrawerTitle(937)、renderModalFormDialogTitle(954)、popover 分支传给 NavigationOverlaytitle prop(1276)。

case 'create': return `Create ${objectLabel}`;
case 'edit':   return `Edit ${objectLabel}`;
case 'view':   return `View ${objectLabel}`;

zh 会话打开抽屉读到的是 "View 联系人" —— 英文动词贴在本地化标签前。这不是推断:反向验证时把 ObjectView.tsx 还原成 main 的版本再跑新测试,渲染出的 DOM 里字面量出现了 View 联系人 / Create 联系人 / Edit 联系人,与报告人的复现完全一致。

可达性门槛比 #3459 的 split 分支还低:ObjectViewSchema.layout 默认即 'drawer',而 navigationobject-view 注册块上已声明的可编写输入,其 mode 联合类型含 drawer / modal / popover;任一模式下点行即 setFormMode('view') 并打开容器。app-shell 包装器钉 layout: 'page' 是宿主覆盖已注册块,不能证明分支已死。

改法:复用既有 key 家族,而不是新造

按 PM 裁定「先在十个语言包里找现成的 Create/Edit/View 形状的 key,找不到再造 console.objectView.*」—— 找到了:

  • form.createTitle = Create {{object}}
  • form.editTitle = Edit {{object}}

这两个 key 十包全有,而且 app-shell 的 page 模式记录表单标题用的正是它们(RecordFormPage.tsx:131-132AppContent.tsx:787-788)。抽屉 / 弹窗 / 浮层的标题只是同一个标题的另一个承载面,所以解析同一组 key;另造一套 console.objectView.* 只会保证两处拼写迟早漂开(zh 本身就区分 新建 与 创建),而这恰是本系列 issue 要治的病。

只有第三个动词没有兄弟,新增 form.viewTitle 到十个语言包,按各包 create/edit 双胞胎的既有语序排布,而非「翻译动词再拼标签」:

createTitle(既有) editTitle(既有) viewTitle(新增)
en Create {{object}} Edit {{object}} View {{object}}
zh 新建{{object}} 编辑{{object}} 查看{{object}}
de {{object}} erstellen {{object}} bearbeiten {{object}} anzeigen
ja {{object}}を作成 {{object}}を編集 {{object}}を表示
ko {{object}} 생성 {{object}} 편집 {{object}} 보기
fr Créer {{object}} Modifier {{object}} Afficher {{object}}
es Crear {{object}} Editar {{object}} Ver {{object}}
pt Criar {{object}} Editar {{object}} Ver {{object}}
ru Создать {{object}} Редактировать {{object}} Просмотреть {{object}}
ar إنشاء {{object}} تعديل {{object}} عرض {{object}}

VIEW_DEFAULT_TRANSLATIONS(#3464 在本文件建好的兜底表)补上三条英文,供无 I18nProvider 的宿主使用。

两个分支刻意保持原样,并各有测试钉住:schema.form?.title 覆盖(作者写了标题就用作者的,任何语言下都是)、以及 default 分支(只回标签,没有动词可译)。

工具栏那个「新建」按钮仍解析 console.objectView.new,刻意没有拿来当标题:按钮动词与标题是两种语境,把它们并成一个 key 正是下一次同形漂移的起点。

与 PM 裁定的一处偏差(需知会)

插值变量是 {{object}} 而非派发单里写的 {{label}}。这是复用既有 form.*Title 家族的机械后果 —— 该家族声明的就是 {{object}},而 all-locales-key-parity 的占位符守卫是按 key 逐个比对占位符集合的;给 viewTitle 单独用 {{label}} 会让它与两个兄弟不一致。若维护者更希望另起 console.objectView.* 家族,改回来的成本很小,但代价是与 app-shell 的 page 模式标题分家。

测试

新增两个文件,分文件的理由沿用 #3464:createI18n 会把实例注册成 react-i18next 的模块级全局默认,该注册在 cleanup() 后仍在,所以「无 provider」断言必须独占一个文件。

  • ObjectView.formTitleI18n.test.tsx —— 13 例,三个动词 × en/zh/de(+ view 的 ja),外加 modal DialogTitle、popover NavigationOverlay 两个消费点,以及 schema.form.title 覆盖。
  • ObjectView.formTitleNoProviderFallback.test.tsx —— 4 例,无 provider 时三个动词的英文兜底 + 无 label 时回落到 objectName

de 是承重断言:德语动词在标签之后(Kontakte anzeigen),任何「翻译动词再拼接」的写法在任何动词取值下都产不出这个语序 —— 将来若有人把 key 退化成动词查表,这条会红。

反向验证(方向在跑之前就定好了)

预测:zh/de/ja 各动词 —— 改前红、改后绿;en、无 provider、form.title 覆盖 —— 两侧皆绿(逐字节钉死)。

git checkout origin/main -- packages/plugin-view/src/ObjectView.tsx 还原实现、保留新测试与语言包后实测:

 Test Files  1 failed | 1 passed (2)
      Tests  9 failed | 8 passed (17)

9 条红的全部且仅为 zh/de/ja 用例(view/edit/create 三态 + modal + popover);8 条绿的是 3 条 en + 1 条 form.title 覆盖 + 4 条无 provider 兜底。方向与预测完全一致。恢复实现后 17/17 全绿。

完整跑批(仓库根,flock 串行,--maxWorkers=2)

npx vitest run --maxWorkers=2 packages/plugin-view packages/i18n
 Test Files  32 passed (32)
      Tests  363 passed (363)

all-locales-key-parity.test.ts(十包 key 与占位符全量对齐)。

turbo run type-check --filter=@object-ui/plugin-view --filter=@object-ui/i18n
 Tasks:    16 successful, 16 total

turbo run lint --filter=@object-ui/plugin-view --filter=@object-ui/i18n
 ✖ 138 problems (0 errors, 138 warnings)   ← 全为既有 warning

消费半径清扫

按规则用 key 的消费半径而非改动包来扫 fixture:全仓 grep 旧标题字面量(View Contacts 等)与 getFormTitle,除本 PR 自己的测试外,没有任何 e2e spec 或宿主测试按旧英文串寻址这块 chrome —— 英文输出逐字节不变,所以本来也无需改动。

字节自查

node scripts/check-control-bytes.mjs 通过(扫 3668 个文件);另按规则做了超出闸门的自查 grep -naP '[\x00-\x08\x0b\x0c\x0e-\x1f]',改动文件全部干净。

Changeset:.changeset/object-view-form-title-i18n-3462.md(@object-ui/plugin-view + @object-ui/i18n,均为 patch)。


Generated by Claude Code

`ObjectView.getFormTitle()` 用模板字符串自拼三个英文动词
(`Create/Edit/View ${objectLabel}`),zh 会话打开抽屉读到的是
"View 联系人" —— 英文动词贴在本地化标签前。三个消费点全是可见标题:
`renderDrawerForm` 的 DrawerTitle、`renderModalForm` 的 DialogTitle,
以及 popover 分支传给 `NavigationOverlay` 的 title prop。

三个动词分支改走 `form.createTitle` / `form.editTitle` /
`form.viewTitle`。没有新造 key 家族:前两个十包已有,且 app-shell 的
page 模式记录表单(RecordFormPage / AppContent)用的正是它们 —— 抽屉、
弹窗、浮层只是同一个标题的另一个承载面,复用同一组 key 才不会漂。
只有第三个动词没有兄弟,`form.viewTitle` 按各包既有排布补进十个语言包。

`schema.form?.title` 覆盖分支与 `default` 分支保持原样。英文输出逐字节
不变,无 provider 时由 `VIEW_DEFAULT_TRANSLATIONS` 兜底。

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01GTRjn8xBqp75dk7kFupVRt
@vercel

vercel Bot commented Aug 6, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
objectui Ignored Ignored Aug 6, 2026 7:57am

Request Review

@github-actions

github-actions Bot commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

✅ Console Performance Budget

Metric Value Budget
Main entry (gzip) 28.1 KB 350 KB
Entry file index-B_1r7HOU.js
Status PASS

📦 Bundle Size Report

Package Size Gzipped
app-shell (index.js) 8.47KB 3.09KB
app-shell (runtime-config.js) 7.42KB 2.32KB
app-shell (types.js) 0.01KB 0.04KB
app-shell (urlParams.js) 7.57KB 2.97KB
auth (AuthContext.js) 0.31KB 0.24KB
auth (AuthGuard.js) 1.17KB 0.53KB
auth (AuthProvider.js) 22.10KB 4.37KB
auth (AuthShell.js) 3.49KB 1.40KB
auth (ForgotPasswordForm.js) 12.21KB 3.45KB
auth (LoginForm.js) 18.13KB 5.39KB
auth (PreviewBanner.js) 0.90KB 0.50KB
auth (RegisterForm.js) 6.64KB 2.21KB
auth (SocialSignInButtons.js) 9.60KB 3.89KB
auth (UserMenu.js) 3.40KB 1.22KB
auth (auth-gate-events.js) 1.29KB 0.66KB
auth (authStyles.js) 5.04KB 1.72KB
auth (createAuthClient.js) 35.76KB 9.11KB
auth (createAuthenticatedFetch.js) 4.37KB 1.69KB
auth (index.js) 2.35KB 1.07KB
auth (org-roles.js) 6.66KB 2.78KB
auth (phone-identifier.js) 1.11KB 0.66KB
auth (types.js) 0.59KB 0.35KB
auth (useAuth.js) 4.91KB 0.87KB
auth (useIsWorkspaceAdmin.js) 1.61KB 0.85KB
collaboration (CommentThread.js) 21.35KB 5.70KB
collaboration (LiveCursors.js) 3.17KB 1.27KB
collaboration (PresenceAvatars.js) 6.49KB 2.64KB
collaboration (PresenceProvider.js) 2.79KB 1.13KB
collaboration (index.js) 1.65KB 0.73KB
collaboration (useCollaborationTranslation.js) 5.30KB 2.24KB
collaboration (useCommentSearch.js) 1.98KB 0.88KB
collaboration (useConflictResolution.js) 7.75KB 1.86KB
collaboration (useMentionNotifications.js) 1.81KB 0.68KB
collaboration (usePresence.js) 6.33KB 1.84KB
collaboration (useRealtimeSubscription.js) 7.91KB 2.01KB
components (index.js) 479.63KB 105.43KB
core (index.js) 2.47KB 0.91KB
create-plugin (index.js) 9.28KB 2.98KB
data-objectstack (index.js) 136.23KB 34.75KB
fields (index.js) 229.92KB 56.48KB
i18n (LocalizationContext.js) 1.76KB 0.96KB
i18n (currency.js) 1.22KB 0.64KB
i18n (i18n.js) 4.32KB 1.77KB
i18n (index.js) 2.65KB 1.06KB
i18n (pickLocalized.js) 1.70KB 0.83KB
i18n (provider.js) 9.48KB 3.27KB
i18n (useObjectLabel.js) 26.14KB 6.07KB
i18n (useSafeTranslation.js) 3.26KB 1.44KB
layout (index.js) 38.53KB 10.71KB
mobile (MobileProvider.js) 0.92KB 0.49KB
mobile (ResponsiveContainer.js) 0.94KB 0.38KB
mobile (breakpoints.js) 1.51KB 0.70KB
mobile (createOfflineDataSource.js) 5.61KB 1.74KB
mobile (index.js) 1.50KB 0.62KB
mobile (offlineQueue.js) 3.91KB 1.35KB
mobile (pwa.js) 0.97KB 0.49KB
mobile (serviceWorker.js) 1.48KB 0.62KB
mobile (serviceWorkerSource.js) 3.41KB 1.48KB
mobile (useBreakpoint.js) 1.54KB 0.65KB
mobile (useGesture.js) 6.96KB 1.98KB
mobile (useOfflineSync.js) 1.99KB 0.72KB
mobile (usePullToRefresh.js) 2.53KB 0.85KB
mobile (useResponsive.js) 0.71KB 0.42KB
mobile (useResponsiveConfig.js) 1.36KB 0.63KB
mobile (useSpecGesture.js) 4.05KB 1.53KB
mobile (useTouchTarget.js) 1.01KB 0.54KB
permissions (MePermissionsProvider.js) 8.75KB 3.06KB
permissions (PermissionContext.js) 0.31KB 0.25KB
permissions (PermissionGuard.js) 0.89KB 0.45KB
permissions (PermissionProvider.js) 3.67KB 1.12KB
permissions (evaluator.js) 4.41KB 1.44KB
permissions (index.js) 0.91KB 0.41KB
permissions (store.js) 0.91KB 0.42KB
permissions (useFieldPermissions.js) 1.28KB 0.52KB
permissions (usePermissions.js) 1.55KB 0.71KB
plugin-ai (index.js) 15.71KB 3.79KB
plugin-calendar (index.js) 44.98KB 12.37KB
plugin-charts (index.js) 61.04KB 17.31KB
plugin-chatbot (index.js) 180.09KB 42.72KB
plugin-dashboard (index.js) 112.03KB 28.88KB
plugin-designer (index.js) 210.51KB 42.51KB
plugin-detail (index.js) 232.53KB 57.37KB
plugin-editor (index.js) 2.46KB 1.10KB
plugin-form (index.js) 111.54KB 26.97KB
plugin-gantt (index.js) 162.55KB 39.57KB
plugin-grid (index.js) 185.25KB 49.08KB
plugin-kanban (index.js) 48.03KB 13.22KB
plugin-list (index.js) 105.19KB 25.39KB
plugin-map (index.js) 16.81KB 5.24KB
plugin-markdown (index.js) 13.72KB 4.69KB
plugin-report (index.js) 40.58KB 10.58KB
plugin-timeline (index.js) 25.76KB 7.33KB
plugin-tree (index.js) 8.50KB 2.88KB
plugin-view (index.js) 84.03KB 20.55KB
providers (DataSourceProvider.js) 0.75KB 0.39KB
providers (MetadataProvider.js) 1.37KB 0.59KB
providers (ThemeProvider.js) 1.90KB 0.85KB
providers (UploadProvider.js) 11.71KB 3.53KB
providers (index.js) 0.44KB 0.22KB
providers (types.js) 0.01KB 0.04KB
react-runtime (index.js) 5.67KB 2.37KB
react (LazyPluginLoader.js) 3.77KB 1.33KB
react (SchemaRenderer.js) 19.28KB 6.38KB
react (data-invalidation.js) 5.05KB 2.08KB
react (index.js) 1.02KB 0.55KB
react (spec-input.js) 0.20KB 0.18KB
sdui-parser (codegen.js) 4.09KB 1.74KB
sdui-parser (index.js) 4.47KB 2.03KB
sdui-parser (parse.js) 10.04KB 2.82KB
sdui-parser (types.js) 0.29KB 0.24KB
sdui-parser (validate.js) 4.69KB 1.48KB
types (ai.js) 0.20KB 0.17KB
types (api-types.js) 0.20KB 0.18KB
types (app.js) 2.87KB 0.99KB
types (base.js) 0.20KB 0.18KB
types (blocks.js) 0.20KB 0.18KB
types (complex.js) 0.20KB 0.18KB
types (crud.js) 0.20KB 0.18KB
types (data-display.js) 0.20KB 0.18KB
types (data-protocol.js) 0.20KB 0.19KB
types (data.js) 0.20KB 0.18KB
types (designer.js) 1.87KB 0.85KB
types (disclosure.js) 0.20KB 0.18KB
types (error-code.js) 1.54KB 0.88KB
types (feedback.js) 0.20KB 0.18KB
types (field-types.js) 0.20KB 0.18KB
types (form.js) 0.20KB 0.18KB
types (http-retry.js) 4.32KB 2.02KB
types (index.js) 2.46KB 1.21KB
types (layout.js) 0.20KB 0.18KB
types (managed-by.js) 0.19KB 0.18KB
types (mobile.js) 0.20KB 0.18KB
types (navigation.js) 0.20KB 0.18KB
types (objectql.js) 0.20KB 0.18KB
types (overlay.js) 0.20KB 0.18KB
types (permissions.js) 0.20KB 0.18KB
types (plugin-scope.js) 0.20KB 0.18KB
types (record-components.js) 0.20KB 0.19KB
types (record-semantics.js) 1.28KB 0.67KB
types (registry.js) 0.20KB 0.18KB
types (reports.js) 0.20KB 0.18KB
types (spec-report.js) 5.05KB 1.93KB
types (system-fields.js) 3.33KB 1.54KB
types (theme.js) 0.20KB 0.18KB
types (ui-action.js) 3.40KB 1.71KB
types (views.js) 0.20KB 0.18KB
types (widget.js) 0.20KB 0.18KB

Size Limits

  • ✅ Core packages should be < 50KB gzipped
  • ✅ Component packages should be < 100KB gzipped
  • ⚠️ Plugin packages should be < 150KB gzipped

@yinlianghui
yinlianghui marked this pull request as ready for review August 6, 2026 08:02
@yinlianghui
yinlianghui added this pull request to the merge queue Aug 6, 2026
Merged via the queue into main with commit 28b2e65 Aug 6, 2026
19 checks passed
@yinlianghui
yinlianghui deleted the claude/issue-3462-form-title-i18n branch August 6, 2026 08:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

2 participants