Skip to content

fix(i18n): apply missing translations to MobileNav component#3935

Open
pranjal270 wants to merge 1 commit intoprocessing:developfrom
pranjal270:fix/mobile-nav-translations
Open

fix(i18n): apply missing translations to MobileNav component#3935
pranjal270 wants to merge 1 commit intoprocessing:developfrom
pranjal270:fix/mobile-nav-translations

Conversation

@pranjal270
Copy link

Issue

Fixes #3928

Some MobileNav UI labels ("My Stuff", "Settings", "Log Out", "by…") were hardcoded and did not respond to language changes. Additionally, the corresponding translation keys — including "Select Language" — were missing from non-English locale files.


Demo

image image

Changes

  • Replaced hardcoded strings in MobileNav.jsx with t() calls.
  • Added missing translation keys to all locale JSON files using English values as placeholders.
  • Added a script to sync missing keys from the base English translations file.
  • Added verified Hindi translations for key labels for local testing.

I have verified that this pull request:

  • has no linting errors (npm run lint)
  • has no test errors (npm run test)
  • has no typecheck errors (npm run typecheck)
  • is from a uniquely-named feature branch and is up to date with the develop branch.
  • is descriptively named and links to an issue number.
  • meets the accessibility guidelines.

@yugalkaushik
Copy link
Contributor

@pranjal270 Thank you for the PR. Please review the contribution guidelines, as we prefer translation changes only in languages you are proficient in or native to. Additionally, very large translation updates are generally not preferred since they are difficult to review properly.

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.

Bug: Missing translation in MobileNav Language Select Overlay

2 participants