Skip to content

fix: add i18n to 3 more TUI dialog components#807

Open
MrRealORG wants to merge 1 commit into
XiaomiMiMo:mainfrom
MrRealORG:fix/auto-0616-dialog-i18n-coverage
Open

fix: add i18n to 3 more TUI dialog components#807
MrRealORG wants to merge 1 commit into
XiaomiMiMo:mainfrom
MrRealORG:fix/auto-0616-dialog-i18n-coverage

Conversation

@MrRealORG

Copy link
Copy Markdown

Summary

Continues i18n coverage expansion for TUI dialog components.

Changes

dialog-session-delete-failed.tsx (6 strings → i18n)

  • Title: "Failed to Delete Session"
  • Description with session/workspace names (uses {{session}}/{{workspace}} interpolation)
  • Recovery prompt
  • 2 option titles: "Delete workspace" / "Restore to new workspace"
  • 2 option descriptions

dialog-workspace-unavailable.tsx (4 strings → i18n)

  • Title: "Workspace Unavailable"
  • Description
  • Restore prompt
  • 2 button labels: "cancel" / "restore"

plugin-route-missing.tsx (2 strings → i18n)

  • Warning message with plugin route ID (uses {{id}} interpolation)
  • "go home" button label

i18n keys

  • 15 new keys added to en.ts and zh.ts
  • Namespaces: tui.delete_failed.*, tui.workspace_unavailable.*, tui.plugin_route_missing.*
  • Other locales (es, fr, ja, ru, zht) fall back to English

Files changed

  • 3 component files: +35 lines, -13 lines (import + t() calls)
  • 2 locale files: +40 lines (15 keys × 2 locales + 1 trailing comma)

- dialog-session-delete-failed.tsx: Replace 6 hardcoded English strings
  with t() calls (title, description, 2 option titles, 2 option descriptions)
- dialog-workspace-unavailable.tsx: Replace 4 hardcoded English strings
  with t() calls (title, description, prompt, 2 button labels)
- plugin-route-missing.tsx: Replace 2 hardcoded English strings with t() calls
- Add 15 new i18n keys to en.ts and zh.ts
  (tui.delete_failed.*, tui.workspace_unavailable.*, tui.plugin_route_missing.*)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant