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

English updates and a Chinese string fault #163

Merged
merged 9 commits into from Mar 29, 2020
Merged

Conversation

CPCer
Copy link
Contributor

@CPCer CPCer commented Mar 29, 2020

其他问题:

  1. {name} 没有翻译,所以英文界面本来应该是“Favorites list”在添加歌曲时显示的还是“我的收藏”
  2. 搜索那一排“小芸搜索”之类的暗号,不清楚英文长度能否完全显示

CPCer and others added 5 commits March 29, 2020 18:27
问题:
1. {name} 没有翻译,所以英文界面本来应该是“Favorites list”在添加歌曲时显示的还是“我的收藏”
2. 搜索那一排“小芸搜索”之类的暗号,不清楚英文长度能否完全显示
"backup_part_export_list_desc": 应该是歌单而不是设置
原字符串前后的空格……
@CPCer
Copy link
Contributor Author

CPCer commented Mar 29, 2020

刚刚发现一个string语言包里好像没有……
请求异常😮,可以多试几次,若还是不行就换一首吧。。。
似乎 message.js 都不在语言包里……

@lyswhut
Copy link
Owner

lyswhut commented Mar 29, 2020

谢谢提供PR,是的,一些信息没有提取到语言包里面

@lyswhut lyswhut self-requested a review March 29, 2020 13:12
Comment on lines 1 to 4
{
"min": "Minimize",
"close": "Close"
}
Copy link
Owner

Choose a reason for hiding this comment

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

这里有什么变化吗?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

这里有什么变化吗?

没有。。翻译的时候是整个文件夹过的,可能也覆盖了……

Comment on lines 2 to 29
"theme_green": "Ocean Green",
"theme_blue": "Curious Blue",
"theme_yellow": "Confetti Yellow",
"theme_orange": "Saffron Orange",
"theme_red": "Valencia Red",
"theme_purple": "Wisteria Purple",
"theme_grey": "Slate Gray",
"theme_mid_autumn": "Chinese Goddess to the Moon",
"theme_naruto": "Naruto with Friends",
"theme_happy_new_year": "Happy Chinese New Year",

"source_kw": "Kuwo music",
"source_kg": "Kugou music",
"source_tx": "QQ music",
"source_wy": "NetEase Music",
"source_mg": "Migu Music",
"source_bd": "Baidu Music",

"source_all": "Meta Search",

"source_alias_kw": "KW Music",
"source_alias_kg": "KG Music",
"source_alias_tx": "Kyu×2 Music",
"source_alias_wy": "NY Music",
"source_alias_mg": "MG Music",
"source_alias_bd": "BD Music",

"source_alias_all": "Meta"
Copy link
Owner

@lyswhut lyswhut Mar 29, 2020

Choose a reason for hiding this comment

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

英文都是谷歌翻译翻的,主题名字太长会显示不全,所以使用尽量简即可,24行的Kyu×2 Music是什么鬼。。聚合搜索为什么翻译成meta呢?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

英文都是谷歌翻译翻的,主题名字太长会显示不全,所以使用尽量简即可,24行的Kyu×2 Music是什么鬼。。聚合搜索为什么翻译成meta呢?

Kyux2:啊 因为英文习惯者可能知道QQ音乐但不清楚QQ跟腾讯的关系,所以起花名只好用Q的音Kyu来弄了…
聚合搜索……一般在英语里指的是“元搜索”引擎吧……
这些您来定

"select_all": "Select all",
"unselect_all": "Select none",
"name": "Name",
"progress": "Downloaded",
Copy link
Owner

Choose a reason for hiding this comment

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

下载进度,用Progress即可,进展的意思

Comment on lines 9 to 10
"lossless": "Lossless",
"high_quality": "High Quality",
Copy link
Owner

Choose a reason for hiding this comment

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

这里显示的是音乐列表的音质标签,太长不好吧

Copy link
Contributor Author

Choose a reason for hiding this comment

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

这里显示的是音乐列表的音质标签,太长不好吧

好的

@lyswhut
Copy link
Owner

lyswhut commented Mar 29, 2020

我去,我好像把那几个文件给删了。,。

Copy link
Owner

@lyswhut lyswhut left a comment

Choose a reason for hiding this comment

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

谢谢修正列表导出错误的文字描述问题、英文翻译问题!

@lyswhut
Copy link
Owner

lyswhut commented Mar 29, 2020

你可以把这些更改提交到dev分支吗?

算了,这次我就直接合了,下次先提交到dev分支,后面更新版本时我会再统一推到master分支

@lyswhut lyswhut merged commit 15a7648 into lyswhut:master Mar 29, 2020
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.

None yet

2 participants