Skip to content

feat(PLAY-1452): add core.options translation key - #111

Merged
maximzavadskiy merged 3 commits into
masterfrom
feat/core-options-key
Jul 8, 2026
Merged

feat(PLAY-1452): add core.options translation key#111
maximzavadskiy merged 3 commits into
masterfrom
feat/core-options-key

Conversation

@maximzavadskiy

Copy link
Copy Markdown
Contributor

Summary

Adds an options key to the core translations so the options/settings button has a localized label outside the cc namespace. Part of the UI redesign epic.

  • Added options?: string to the CoreTranslations interface.
  • Reused the existing cc.options wording for the 5 languages that already had it: de, en, gr, hi, zh.
  • Added native translations for the other 15 languages: cs, es, fi, fr, he, nb, nl, nn, oc, pl, pt, sk, sv, tlh, tr.

Verification

  • npm run lint (tsc --noEmit) passes.
  • Pre-existing test failures (ERR_REQUIRE_CYCLE_MODULE in esm/iife specs) are unrelated to this change and reproduce on master.

Ticket

To be linked under the UI redesign epic.

🤖 Generated with Claude Code

Add an `options` key to core translations so the options/settings
button has a localized label outside the cc namespace. Reuses the
existing cc.options wording for the 5 languages that had it (de, en,
gr, hi, zh) and adds native translations for the other 15 languages.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@maximzavadskiy maximzavadskiy changed the title feat: add core.options translation key feat(PLAY-1452): add core.options translation key Jun 22, 2026
@maximzavadskiy
maximzavadskiy requested a review from Copilot June 22, 2026 11:55
@maximzavadskiy
maximzavadskiy marked this pull request as ready for review June 22, 2026 11:55
@maximzavadskiy
maximzavadskiy requested review from a team and bbbo as code owners June 22, 2026 11:55
@maximzavadskiy
maximzavadskiy requested review from alexarassat and removed request for a team June 22, 2026 11:55

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Adds a new core.options translation key so the options/settings control can be labeled outside the cc namespace (supporting the UI redesign work).

Changes:

  • Extended CoreTranslations with an optional options key.
  • Added core.options translations across many locales (reusing existing wording where available).

Reviewed changes

Copilot reviewed 21 out of 21 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
src/index.ts Adds CoreTranslations.options to support a core-level options/settings label.
src/languages/cs.ts Adds core.options translation.
src/languages/de.ts Adds core.options translation.
src/languages/en.ts Adds core.options translation.
src/languages/es.ts Adds core.options translation.
src/languages/fi.ts Adds core.options translation.
src/languages/fr.ts Adds core.options translation.
src/languages/gr.ts Adds core.options translation.
src/languages/he.ts Adds core.options translation.
src/languages/hi.ts Adds core.options translation.
src/languages/nb.ts Adds core.options translation.
src/languages/nl.ts Adds core.options translation.
src/languages/nn.ts Adds core.options translation.
src/languages/oc.ts Adds core.options translation.
src/languages/pl.ts Adds core.options translation.
src/languages/pt.ts Adds core.options translation.
src/languages/sk.ts Adds core.options translation.
src/languages/sv.ts Adds core.options translation.
src/languages/tlh.ts Adds core.options translation.
src/languages/tr.ts Adds core.options translation.
src/languages/zh.ts Adds core.options translation.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/index.ts
Comment thread src/index.ts
Comment thread src/languages/de.ts Outdated
unmute: "Ton ein",
volume: "Lautstärke"
volume: "Lautstärke",
options: "Optionen"

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Einstellungen

@bbbo bbbo left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM except German for options could be "Einstellungen", though "Optionen" is acceptable too.
Klingon is fine ;)

@maximzavadskiy

Copy link
Copy Markdown
Contributor Author

@bbbo fixed. LG?

@maximzavadskiy
maximzavadskiy requested a review from bbbo July 6, 2026 10:57

@bbbo bbbo left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@maximzavadskiy
maximzavadskiy merged commit 70844a2 into master Jul 8, 2026
2 checks passed
@maximzavadskiy
maximzavadskiy deleted the feat/core-options-key branch July 8, 2026 09:55
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.

3 participants