Skip to content

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

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

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

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