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

Fix translation using Chinese name & description by default + Added Russian translation #509

Draft
wants to merge 11 commits into
base: master
Choose a base branch
from

Conversation

Blueberryy
Copy link

No description provided.

@XIU2
Copy link
Owner

XIU2 commented Feb 9, 2025

你可以加俄语翻译(脚本名和脚本描述),但其他就别动了。。。

@Blueberryy
Copy link
Author

你可以加俄语翻译(脚本名和脚本描述),但其他就别动了。。。

i moved them because they show Chinese translation for English and Russian.
With my changes it will show English translation to users with unsupported languages

@Blueberryy
Copy link
Author

How it currently looks
image

How it should be looking
image

@XIU2
Copy link
Owner

XIU2 commented Feb 9, 2025

主要是我写的脚本基本上都是面向中国用户的,所以脚本里只是顺手加上了英语名称和描述,但没有做其他任何英语翻译(脚本选项、脚本介绍等)。

不过,前几天 greasyfork.org 已经被中国封了。。。这是个悲伤的消息,但对我影响不大,因为我写的这些脚本都是为了解决自身需求而写的,顺便分享出来,因此也不特别关心有没有更多人使用(至少现在已经没感觉了)。

@Blueberryy
Copy link
Author

主要是我写的脚本基本上都是面向中国用户的,所以脚本里只是顺手加上了英语名称和描述,但没有做其他任何英语翻译(脚本选项、脚本介绍等)。

不过,前几天 greasyfork.org 已经被中国封了。。。这是个悲伤的消息,但对我影响不大,因为我写的这些脚本都是为了解决自身需求而写的,顺便分享出来,因此也不特别关心有没有更多人使用(至少现在已经没感觉了)。

All good. I'm not forcing you to make English translation of scripts UI so i don't know what should i do with it then

@Blueberryy Blueberryy marked this pull request as draft February 9, 2025 12:15
@XIU2
Copy link
Owner

XIU2 commented Feb 9, 2025

好吧,我试了下你修改后的脚本内容,看起来好像也没什么不合适的,因此这个改动我可以接受,但目前好像无法合并 PR。
因为你发起 PR 前,我刚好更新了 GithubEnhanced-High-Speed-Download.user.js,导致分支冲突,你把这个冲突解决下。

@XIU2
Copy link
Owner

XIU2 commented Feb 9, 2025

最简单的解决方法,就是直接关闭这个 PR,然后把你修改后的这些脚本文件额外复制一份,然后同步你的仓库(或者直接删了重新 fork,这样更简单点),然后再把你修改后的脚本文件都覆盖回去(GithubEnhanced-High-Speed-Download.user.js 除外,这个你就打开文件,单独复制粘贴处理一下),最后所有改动都集合为一条 commit 提交到你的仓库,然后再向我发起 PR。

实际上,因为我很少向别人发起 PR,所以我对 PR 相关知识不是很了解,以上是我有限认知想出来的简单处理方案,是否有更简单的我也不是很清楚,所以仅供参考~

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