-
Notifications
You must be signed in to change notification settings - Fork 518
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
Deprecate KS macros never used in en-US #3063
Comments
@escattone @chrisdavidmills What do you think about this? A "fear" I have is that some ancient translation might have used these like this: (imagine this is NOT in English) <p>See list link for a thing about: {{LXRSearch("something")}}</p> If you render that to become an empty string you end up with: <p>See list link for a thing about: </p> which looks bad. Perhaps a more industrious solution would be to mark all of these macros as deprecated but keep their functionality.
|
@escattone How about we just delete them? I.e. if a translated page still uses Also, once the translators have #3106 they'll clearly see which old busted KS macros they're using that needs to be changed. |
Having already started for This will reduce the list already, and will make it possible to worry about the others macros which are less present and which may sometimes require replacing by text, links, etc... |
There are four macros used in Japanese translation only. I agree with to remove them and to replace with other expressions. kumascript/macros/英語版章題.ejs 439 |
I think that's best, to just delete them, unless they're useful as a means to identify translated pages that need to updated to match their English equivalent? I would vote for deleting them. |
How about we start with that (apart from the Japanese ones that @mfuji09 would like to leave for a little while longer). So, once @tristantheb 's many PRs have been merged we can start |
When I worked on #1274 I noticed there are a lot more macros that would be considered never-used if you focus on the en-US content alone. For example:
So, that macro is never used in the en-US content, but it's used in greater-than-zero places in the translated-content.
What we could do is edit these macros so they're marked as deprecated and just return an empty string.
Here's a complete list of macros that exist but are only used in
translated-content
:The text was updated successfully, but these errors were encountered: