Skip to content

Conversation

@grumlu
Copy link

@grumlu grumlu commented Dec 17, 2024

Following a feature request on Discord, to add Chinese language to the app, I looked into how to support different languages.

I have added a localization for French as well.

Changes are in

  • Localizable.xcstrings file for French translation
  • SettingsView to support localization on Defaults.Toggle and LaunchAtStartup.Toggle
  • generic in enums to support adding localization there also

For now there is apparently no conflicting keys, however in the future it may be interesting to refactor the code to add "backend" names to each string depending on where they're used.

overflow-sudo and others added 22 commits November 29, 2024 00:48
If the artwork data cannot be found, it will be replaced with a placeholder provided by media player
…e is this is sometimes 0 even though something is playing
…p button is pressed or the source modifies the elapsedTime after being paused
…edDisplay so that boringNotch remembers the last manually selected screen and tries to use it if it is available
…el to BoringView Coordinator. This will help isolate the singleton-required code from the ViewModel that is now specific to each window.
…creens (seperate control from non-notched displays)
@grumlu grumlu changed the title Add the ability to localise the application Add the ability to localize the application Dec 17, 2024
@grumlu
Copy link
Author

grumlu commented Dec 18, 2024

I have updated all the strings to use a key instead of the English translation.
This allows to be more precise in the translation in different context.

The French translation has also been updated to match this change and fix some display issues.

@theboringhumane
Copy link
Member

@theboringhumane , are you setting up some AI review bot? 🤩

I built it lol

@grumlu
Copy link
Author

grumlu commented Dec 19, 2024

@theboringhumane , are you setting up some AI review bot? 🤩

I built it lol

It seems to be an authentication / pipeline issue for Cori, right? Or did I do something wrong?

@theboringhumane
Copy link
Member

Yes action was for that only.

@richardkunkli
Copy link
Member

The app is already localizable, no need to change texts, Xcode handles it.

@grumlu
Copy link
Author

grumlu commented Dec 22, 2024

The app is already localizable, no need to change texts, Xcode handles it.

One commit earlier does the translation in French without changing all the texts.
However, the same text in two different contexts uses the same translation in this case.
This doesn’t work in all languages. The context matters and the translation would be then different.

@piersandro
Copy link

@richardkunkli why don't you create a string database on Crowdin or Lokalise to enable localisation without having to start from source and use Xcode?

@richardkunkli
Copy link
Member

richardkunkli commented Jan 15, 2025

@piersandro I have set up everything (including Crowdin) months ago, but haven't published yet, due to the extensive changes in the settings that includes the most strings. I believe soon we can launch it.

@piersandro
Copy link

Thank you @richardkunkli. Excellent news. I'll be ready for the Italian localisation 👍

@grumlu
Copy link
Author

grumlu commented Jan 15, 2025

@theboringhumane / @richardkunkli , is it worth that I update my work to match with the Jellyfin update or shall I close the PR?

@richardkunkli
Copy link
Member

Will check the PR more deeply next week. Currently so busy, I have no time to work on these side projects. Maybe worth to ping me again with this if it's not a problem you.

@theboringhumane
Copy link
Member

Close this PR probably since, @richardkunkli already setted up Crowdin

@FirleFrantz
Copy link

I prepared the German translation for myself today and just wanted to make it available here. But I just realized that this is not possible at all? Maybe someone can help me?

@Alexander5015 Alexander5015 deleted the branch TheBoredTeam:dev August 2, 2025 06:21
@Alexander5015 Alexander5015 reopened this Aug 2, 2025
@Alexander5015
Copy link
Member

I’ve created a Crowdin project for localization: https://crowdin.com/project/boring-notch
Please submit all localization contributions through this project.

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.

9 participants