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

A few changes related to language #270

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Conversation

JAGulin
Copy link

@JAGulin JAGulin commented Jul 4, 2021

These commits are meant to make the plugin and development more accessible for others.
Let me know if you prefer to have each change in a separate pull-request. Only the last patch (output) depend on the previous (tipsI).
Closes #271

Language not specifically translated is better served English.
In case any variant should get Chinese, please add those as zh.
Efficient fallback to default.
@@ -1,5 +1,5 @@
// ==UserScript==
// @name HTML5视频播放器增强脚本
Copy link
Author

Choose a reason for hiding this comment

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

There may be side effects of changing the official name. Possibly TM or GreasyFork might think it's a new script, rather than a change to the existing one.
https://greasyfork.org/en/discussions/greasyfork/75556-when-a-script-name-is-changed-greasyfork-doesn-t-recognize-the-old-script
This needs to be well tested. Changing description and other things should be safe and could be first step.

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.

Plugin name and description default language is Chinese (within TM)
1 participant