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

l10n: create dedicated strings for music and video libraries #458

Merged
merged 3 commits into from
Oct 1, 2024

Conversation

huynhsontung
Copy link
Owner

@huynhsontung huynhsontung commented Sep 30, 2024

Fixes #457

The translation of "Music" in some languages doesn't convey the music library in context. Create a dedicated string to accommodate for these differences

The translation of "Music" in some languages doesn't convey the music library in context. Create a dedicated string to accomodate for these differences
@huynhsontung huynhsontung force-pushed the l10n-media-library-strings branch from 49ef73c to d940883 Compare September 30, 2024 18:32
@huynhsontung huynhsontung enabled auto-merge (squash) October 1, 2024 06:52
@huynhsontung huynhsontung merged commit 585338b into main Oct 1, 2024
4 checks passed
@maisondasilva
Copy link

@huynhsontung The way you did it was very good, I believe you can cancel the reversion as it is correct with our language.

@huynhsontung huynhsontung deleted the l10n-media-library-strings branch October 5, 2024 04:19
@huynhsontung
Copy link
Owner Author

@maisondasilva Just to confirm, you meant this commit is not correct? Should the text on the nav bar and music page title be the same? I was guessing based on the inbox Media Player app.

image

@maisondasilva
Copy link

maisondasilva commented Oct 5, 2024

@huynhsontung Yes, the 3 variables are correct as you did, my question is about this
fix(l10n): revert MusicPage and VideosPage title change in #458

The image above is in pt-PT (Portugal) although it follows the same standard as pt-BR (Brazil)

@maisondasilva
Copy link

maisondasilva commented Oct 5, 2024

This commit is 100% correct

585338b

Same for Video
Captura de tela 2024-10-05 092201

Thanks Maison

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.

Music pt-BR
2 participants