Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Translate board menu in puzzle and game #16551

Merged
merged 3 commits into from
Dec 10, 2024

Conversation

yafred
Copy link
Contributor

@yafred yafred commented Dec 8, 2024

Note that I have used 2 translations from preferences (which are the titles of the preference pages)

'Game display preferences' becomes 'Display' (i18n.preferences.display)
'Game behavior preferences' becomes 'Game behavior' (i18n.preferences.gameBehavior)

If you prefer, I can add 2 specific keys

Puzzle board menu:
lichess-puzzle-boardmenu-en
lichess-puzzle-boardmenu-fr

Game board menu:

lichess-game-boardmenu-en
lichess-game-boardmenu-fr

@@ -1134,6 +1133,9 @@ object I18nKey:
val `notifyWeb`: I18nKey = "preferences:notifyWeb"
val `notifyDevice`: I18nKey = "preferences:notifyDevice"
val `bellNotificationSound`: I18nKey = "preferences:bellNotificationSound"
val `blindFold`: I18nKey = "preferences:blindFold"
val `voiceInput`: I18nKey = "preferences:voiceInput"
Copy link
Collaborator

@ornicar ornicar Dec 9, 2024

Choose a reason for hiding this comment

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

a translation already exists, inputMovesWithVoice

@@ -1134,6 +1133,9 @@ object I18nKey:
val `notifyWeb`: I18nKey = "preferences:notifyWeb"
val `notifyDevice`: I18nKey = "preferences:notifyDevice"
val `bellNotificationSound`: I18nKey = "preferences:bellNotificationSound"
val `blindFold`: I18nKey = "preferences:blindFold"
Copy link
Collaborator

Choose a reason for hiding this comment

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

why the capital F? Everywhere else it's just blindfold

@@ -1134,6 +1133,9 @@ object I18nKey:
val `notifyWeb`: I18nKey = "preferences:notifyWeb"
val `notifyDevice`: I18nKey = "preferences:notifyDevice"
val `bellNotificationSound`: I18nKey = "preferences:bellNotificationSound"
val `blindFold`: I18nKey = "preferences:blindFold"
val `voiceInput`: I18nKey = "preferences:voiceInput"
val `keyboardInput`: I18nKey = "preferences:keyboardInput"
Copy link
Collaborator

Choose a reason for hiding this comment

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

a translation already exists, inputMovesWithTheKeyboard

@yafred yafred marked this pull request as draft December 9, 2024 10:14
@yafred yafred marked this pull request as draft December 9, 2024 10:14
@yafred
Copy link
Contributor Author

yafred commented Dec 9, 2024

The text of the translations for the preferences may be too long to fit the side view ...

There are a few options:

  • Request a shorter translation for preferences
  • Have a specific translation for the side view
  • Wrap the text in the side view

lichess-boardmenu-fr-issue

@yafred yafred marked this pull request as ready for review December 9, 2024 18:06
@ornicar ornicar merged commit f2f4a75 into lichess-org:master Dec 10, 2024
6 checks passed
@superuser-does
Copy link
Collaborator

I wish to highlight that we already have the string blindfoldMode on the mobile app.

I don't believe Lila has a mechanism to use the app's translations.

With that in mind, I can migrate the translations from the mobile side using Crowdin pre-translation if you are happy with that? We can then delete the key on that side to avoid duplicating effort.

@superuser-does
Copy link
Collaborator

superuser-does commented Dec 11, 2024

Thanks for the vote of confidence, Thibault.

I have copied all translations using Crowdin pre-translation from the mobile app to lila's file, and created an issue #1259 for the mobile app to replace their string with lila's.

Edit, 2024-12-12: The string has now been wholly replaced ok the mobile side.

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