Skip to content

add FR translation file#229

Merged
Cool04ek merged 2 commits intoYalantis:masterfrom
clemp6r:master
Nov 14, 2016
Merged

add FR translation file#229
Cool04ek merged 2 commits intoYalantis:masterfrom
clemp6r:master

Conversation

@clemp6r
Copy link
Copy Markdown
Contributor

@clemp6r clemp6r commented Nov 14, 2016

No description provided.

<resources>
<string name="ucrop_label_original">Original</string>
<string name="ucrop_label_edit_photo">Modifier la photo</string>
<string name="ucrop_menu_crop">Recarder</string>
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Is it correct translation? Because Google translate doesn't know this word.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Sorry I made I typo, I will fix this 😀

@@ -0,0 +1,5 @@
<resources>
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

It lack translation for two more strings - ucrop_error_input_data_is_absent and ucrop_mutate_exception_hint

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yes, they are exception messages. It's not common to translate this kind of messages, because they're used for logging, not for showing to the user. Everything in the logcat is in english: having translated messages in the logs might be confusing. Also you can see that they're not translated in NL and SK.

I think these strings should not be externalized at all. They should be directly found in the source code.

@Cool04ek
Copy link
Copy Markdown
Contributor

It makes sense. Thank you.

@Cool04ek Cool04ek merged commit 91a1945 into Yalantis:master Nov 14, 2016
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.

2 participants