Skip to content

wayvr: add missing ALTTAB.KEYBOARD translation key - #643

Open
sandikodev wants to merge 1 commit into
wayvr-org:mainfrom
sandikodev:fix/alttab-keyboard-translation
Open

wayvr: add missing ALTTAB.KEYBOARD translation key#643
sandikodev wants to merge 1 commit into
wayvr-org:mainfrom
sandikodev:fix/alttab-keyboard-translation

Conversation

@sandikodev

Copy link
Copy Markdown

Summary

The ALTTAB.KEYBOARD translation key is referenced by the Alt-Tab help overlay for the T binding (wayvr/src/assets/gui/alttab-help.xml), but it was missing from the language files. At runtime this logs

ERROR wgui::i18n: missing translation for key "ALTTAB.KEYBOARD"

and the "T" row in the help overlay shows an empty label.

Changes

  • Added the ALTTAB.KEYBOARD key (English: "Toggle keyboard") to the source template wayvr/src/assets/lang/en.csv
  • Added the same key to en.json and all localized files (de, es, it, ja, pl, zh_CN) so the existing lang_json_files_contain_all_keys test keeps passing

Testing

  • cargo test --all-features passes
  • cargo build --all-features passes

@galister

Copy link
Copy Markdown
Collaborator

please don't add english for non-english locale files.

either:

  • run the translator in scripts/translator if you have a chat-completions compatible LLM
  • leave those files unchanged and we'll run the translator for you

@galister

Copy link
Copy Markdown
Collaborator

also, thanks for the PR!

"GRAB": "Einzelnes Overlay verschieben",
"DASH": "Dashboard umschalten",
"EDIT": "Bearbeitungsmodus umschalten",
"KEYBOARD": "Toggle keyboard",

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Suggested change
"KEYBOARD": "Toggle keyboard",

"GRAB": "Mover {$DEFINITIONS.OVERLAY} individual",
"DASH": "Alternar panel de control",
"EDIT": "Alternar modo edición",
"KEYBOARD": "Toggle keyboard",

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Suggested change
"KEYBOARD": "Toggle keyboard",

"GRAB": "Sposta singolo overlay",
"DASH": "Attiva/disattiva dashboard",
"EDIT": "Attiva/disattiva modalità modifica",
"KEYBOARD": "Toggle keyboard",

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Suggested change
"KEYBOARD": "Toggle keyboard",

"GRAB": "個別のオーバーレイを移動",
"DASH": "ダッシュボードの切り替え",
"EDIT": "編集モードの切り替え",
"KEYBOARD": "Toggle keyboard",

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Suggested change
"KEYBOARD": "Toggle keyboard",

"GRAB": "Przesuń pojedynczą nakładkę",
"DASH": "Przełącz panel sterowania",
"EDIT": "Przełącz tryb edycji",
"KEYBOARD": "Toggle keyboard",

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Suggested change
"KEYBOARD": "Toggle keyboard",

"GRAB": "移动单个覆盖层",
"DASH": "切换控制面板",
"EDIT": "切换编辑模式",
"KEYBOARD": "Toggle keyboard",

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Suggested change
"KEYBOARD": "Toggle keyboard",

@galister galister left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

accept the suggestions and i'll merge it (test will fail but don't worry about that)

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.

2 participants