Skip to content

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

Merged
XIU2 merged 12 commits into
XIU2:masterfrom
Blueberryy:master
Feb 11, 2025
Merged

Fix translation using Chinese name & description by default + Added Russian translation#509
XIU2 merged 12 commits into
XIU2:masterfrom
Blueberryy:master

Conversation

@Blueberryy

Copy link
Copy Markdown
Contributor

No description provided.

@XIU2

XIU2 commented Feb 9, 2025

Copy link
Copy Markdown
Owner

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

@Blueberryy

Copy link
Copy Markdown
Contributor 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
Copy Markdown
Contributor Author

How it currently looks
image

How it should be looking
image

@XIU2

XIU2 commented Feb 9, 2025

Copy link
Copy Markdown
Owner

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

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

@Blueberryy

Copy link
Copy Markdown
Contributor 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

XIU2 commented Feb 9, 2025

Copy link
Copy Markdown
Owner

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

@XIU2

XIU2 commented Feb 9, 2025

Copy link
Copy Markdown
Owner

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

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

@Blueberryy Blueberryy marked this pull request as ready for review February 10, 2025 14:37
@Blueberryy

Copy link
Copy Markdown
Contributor Author

ok done

@XIU2 XIU2 merged commit 43bf4fc into XIU2:master Feb 11, 2025
@XIU2

XIU2 commented Feb 11, 2025

Copy link
Copy Markdown
Owner

奇怪,为什么 简体中文 下,脚本名称和描述也显示为英文?这不符合我的预期。

01

@XIU2

XIU2 commented Feb 11, 2025

Copy link
Copy Markdown
Owner

找到原因了,我需要去 greasyfork 脚本管理中手动修改默认语言为英语即可。

@wangliang181230

Copy link
Copy Markdown

图片
漏了个地方没改。

@XIU2

XIU2 commented Feb 11, 2025

Copy link
Copy Markdown
Owner

@wangliang181230 OK,感谢提醒,已经更新修复啦!

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