Skip to content

A few changes related to language - #270

Open
JAGulin wants to merge 4 commits into
xxxily:masterfrom
JAGulin:locale
Open

A few changes related to language#270
JAGulin wants to merge 4 commits into
xxxily:masterfrom
JAGulin:locale

Conversation

@JAGulin

@JAGulin JAGulin commented Jul 4, 2021

Copy link
Copy Markdown

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

JAGulin added 4 commits July 3, 2021 16:41
Language not specifically translated is better served English.
In case any variant should get Chinese, please add those as zh.
Efficient fallback to default.
Comment thread src/h5player/comment.js
@@ -1,5 +1,5 @@
// ==UserScript==
// @name HTML5视频播放器增强脚本

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