-
Notifications
You must be signed in to change notification settings - Fork 39
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
Translation #22
Translation #22
Conversation
@flayor sorry I must have missed the email for this PR! I'll take a look when I get home from work today. |
no problem again, take your time :) For the bug how you tell you 2-3 weeks ago, i don't know if you see but i resolved that you can read this issue #21 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall looks great! Just a few small changes requested.
Contributor : Zhilian DAI | ||
*/ | ||
/* Class = "NSTextFieldCell"; title = "Rescan For Scales:"; ObjectID = "0hr-m8-QTZ"; */ | ||
"0hr-m8-QTZ.title" = "重新扫描Emoji配置区间:"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think this string should include Emoji
as it's not referring to emoji characters specifically, it's referring to a 'scale' or theme to use when displaying battery status.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hello, sorry for lated response, it seems some network problem occur on my email. Dose the "Rescan For Scales" means reload the scale configuration file?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@DonaldDai "Rescan for Scale" it's also say "Rescan for theme"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@flayor What is the practical function?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ho escuse me i don't see the email the .plist files can be a emoji, text, img ...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey folks, so sorry for taking a while to get back to you both. the "Rescan for scales" string could equally means "Rescan for configuration files", I'd like to avoid the usage of emoji in that particular string because users can create a configuration that is not emoji based at all.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would it be easier if I changed this to say "Rescan for themes" ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For now, I'm not familiar with the .plist file, so translation "重新加载配置文件" may be the best one. I will come back when I dive into it, it will not take too long : )
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sounds good, thanks! @DonaldDai
"0hr-m8-QTZ.title" = "重新扫描Emoji配置区间:"; | ||
|
||
/* Class = "NSTextFieldCell"; title = "0 Scales Found"; ObjectID = "1EL-UD-CHx"; */ | ||
"1EL-UD-CHx.title" = "未找到Emoji配置区间"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think this string should include Emoji
as it's not referring to emoji characters specifically, it's referring to a 'scale' or theme to use when displaying battery status.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is should be "未找到配置区间"
"pSb-PT-l5y.title" = "当前区间:"; | ||
|
||
/* Class = "NSButtonCell"; title = "Add Scale"; ObjectID = "xXL-NV-2xU"; */ | ||
"xXL-NV-2xU.title" = "添加Emoji区间"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think this string should include Emoji
as it's not referring to emoji characters specifically, it's referring to a 'scale' or theme to use when displaying battery status.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this mean to add Emoji scale configuration?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It means to add a new scale which can also be though of as a theme.
Again, the scale is just a set of strings, which are not necessarily emojis at all.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Got it. So "添加配置" should be a good choice.
@flayor here's a screenshot of the Chinese and English side by side in reference to the comments I made on the PR |
Hi Sorry for the late response. I have contact the guy Which translated into Chinese No response for now Sorry I fix this ASAP |
Ho…
I'm asking you which edition do you want? The original one or the practical one? : )
Best,
Donald
On 2 Feb 2017, 21:47 +0800, Flayor <notifications@github.com>, wrote:
@flayor commented on this pull request.
________________________________
In Juice/Classes/zh-Hans.lproj/GeneralPreferencesViewController.strings<#22>:
@@ -0,0 +1,24 @@
+
+/*
+Contributor : Zhilian DAI
+*/
+/* Class = "NSTextFieldCell"; title = "Rescan For Scales:"; ObjectID = "0hr-m8-QTZ"; */
+"0hr-m8-QTZ.title" = "重新扫描Emoji配置区间:";
Ho escuse me i don't see the email the .plist files can be a emoji, text, img ...
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub<#22>, or mute the thread<https://github.com/notifications/unsubscribe-auth/ANqe34rKkJgwON4boIJrfRGta7uvehKEks5rYd5egaJpZM4LkAC->.
|
The practical :)
Bien cordialement
Benjamin Noufel
Le 3 févr. 2017 à 13:11 +0100, Zhilian Dai <notifications@github.com>, a écrit :
… Ho…
I'm asking you which edition do you want? The original one or the practical one? : )
Best,
Donald
On 2 Feb 2017, 21:47 +0800, Flayor ***@***.***>, wrote:
@flayor commented on this pull request.
________________________________
In Juice/Classes/zh-Hans.lproj/GeneralPreferencesViewController.strings<#22>:
> @@ -0,0 +1,24 @@
+
+/*
+Contributor : Zhilian DAI
+*/
+/* Class = "NSTextFieldCell"; title = "Rescan For Scales:"; ObjectID = "0hr-m8-QTZ"; */
+"0hr-m8-QTZ.title" = "重新扫描Emoji配置区间:";
Ho escuse me i don't see the email the .plist files can be a emoji, text, img ...
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub<#22>, or mute the thread<https://github.com/notifications/unsubscribe-auth/ANqe34rKkJgwON4boIJrfRGta7uvehKEks5rYd5egaJpZM4LkAC->.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub (#22 (comment)), or mute the thread (https://github.com/notifications/unsubscribe-auth/AGSA0UDi3vfrP9TNI_weVGxgkjVzJxflks5rYxlxgaJpZM4LkAC-).
|
Hi @DonaldDai, Sorry if i didn't answer, ..., i'm In full moving Tell me here in this comment for the translate, I'm lost with all these different threads lol |
@flayor |
It's done @brianmichel |
:)
On 2017年2月8日 +0800 AM12:08, Flayor <notifications@github.com>, wrote:
It's done @brianmichel<https://github.com/brianmichel>
Thanks @DonaldDai<https://github.com/DonaldDai>
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub<#22 (comment)>, or mute the thread<https://github.com/notifications/unsubscribe-auth/ANqe39CMmopSuBzu9qJR45A-EHCowPQJks5raJcBgaJpZM4LkAC->.
|
zh-Hans.lproj/Localizable.strings
Outdated
"General"="常规"; | ||
"General Settings"="常规设置"; | ||
"Credits"="开发者成员"; | ||
"Scales Found"="Emoji电量范围"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This still mentions Emoji
in it's translation and it should not.
zh-Hans.lproj/Localizable.strings
Outdated
"Error scanning application support directory"="寻找必要文件时出现错误"; | ||
"title"="标题"; | ||
"default"="默认"; | ||
"detents"="Emoji范围配置"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This still mentions Emoji
in it's translation and it should not.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"detents" = “打开配置文件”
Well all's feel good ? |
@flayor looks good to me! |
Finish to translate en to fr
Translate en to chinese thanks to Zhilian DAI :) 👍