Skip to content

fix(LanguagePicker): use the language-specific empty-state key#642

Merged
krokosik merged 1 commit intooss-apps:mainfrom
mvanhorn:fix/635-language-picker-no-language-found
May 8, 2026
Merged

fix(LanguagePicker): use the language-specific empty-state key#642
krokosik merged 1 commit intooss-apps:mainfrom
mvanhorn:fix/635-language-picker-no-language-found

Conversation

@mvanhorn
Copy link
Copy Markdown
Contributor

@mvanhorn mvanhorn commented May 8, 2026

Closes #635.

What

When the language picker's search filter matched nothing, the picker rendered the raw i18n key ui.change_language_details.no_currency_found instead of a translated string.

LanguagePicker was looking up account.change_language_details.no_currency_found, which is not defined in the language sub-tree of public/locales/en/common.json (only title, placeholder, and messages.language_changed exist there). no_currency_found is a sibling of the currency picker's locale tree, not the language picker's.

Change

  • src/components/Account/LanguagePicker.tsx: switch noOptionsText lookup to account.change_language_details.no_language_found.
  • public/locales/en/common.json: add "no_language_found": "No language found." next to title/placeholder in account.change_language_details. Per CONTRIBUTING.md, the other locales pick this key up through the existing Weblate flow.

Test plan

  1. pnpm dev, sign in, open Account → Change language.
  2. Type asdasd in the picker.
  3. Expect "No language found." instead of ui.change_language_details.no_currency_found.

AI disclosure

Drafted with AI assistance (Claude). The diff is minimal and reviewed manually.

The 'No language found' placeholder did not exist (oss-apps#635). The component
looked up account.change_language_details.no_currency_found, which is
not defined in the language sub-tree, so the picker rendered the raw
i18n key when a search filter matched nothing.

Switch the lookup to account.change_language_details.no_language_found
and add the matching English string. Other locales pick this key up
through the existing Weblate flow.

Closes oss-apps#635
@krokosik krokosik merged commit c35684c into oss-apps:main May 8, 2026
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.

"No language found" placeholder does not exist

2 participants