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

Improve french translation and R.string usage #364

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

trackktor
Copy link

@trackktor trackktor commented Sep 15, 2019

Hello,
This patch improve french translation and also replace some strings with more generic R.string.
I have tried to rgrep most of the strings. For example "enable/disable" should be gone.
I have tested it with AVD manager and there is no obvious bug.

However, I added new empty lines in strings.xml.
Sorry for that. If its really important I can clean it.

@Relaxo143
Copy link
Contributor

Hello,
This patch improve french translation and also replace some strings with more generic R.string.
I have tried to rgrep most of the strings. For example "enable/disable" should be gone.
I have tested it with AVD manager and there is no obvious bug.

However, I added new empty lines in strings.xml.
Sorry for that. If its really important I can clean it.

Could you open a pull request to my fork of the repo with those changes so that I can do the same for the Bulgarian translation? I will translate the french strings. I just don't know how to replace the hard coded strings with ones from the file.

@SamouraiDev
Copy link
Member

We are managing all French translations internally. Please resubmit w/only the R.string usage changes. Thank you.

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