-
-
Notifications
You must be signed in to change notification settings - Fork 60
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
Make content-language-bounded messages display in user interface language #695
base: master
Are you sure you want to change the base?
Conversation
Codecov Report
@@ Coverage Diff @@
## master #695 +/- ##
=========================================
Coverage 50.23% 50.23%
Complexity 840 840
=========================================
Files 77 77
Lines 2817 2817
=========================================
Hits 1415 1415
Misses 1402 1402 Continue to review full report at Codecov.
|
I'm not sure this is correct. Especially |
I was considering using "page view language" rather than just using "user interface language" in namespaces with wikitext content modal, but there's no "page language"-related variables passed in, causing the complexity to implement it. |
It seems the CI already failed for "PHPUnit: MW master, PHP 8.1" before this commit?
|
|
Yes sure. But is the change in this PR correct or not?
Indeed, don't worry about that failure |
I can't tell because I am not familiar with the code. Most likely not though, if this is ending up in the parser cache without fragmenting it by language. |
I would prefer only by language variants, but I have no idea how to pass the current language variant based on page language. |
8960b58
to
6a829a7
Compare
…uage This fix the issue that the system message "maps-geo-json-create-page-button", "maps-loading-map" cannot display in language variants ("page view language" aka "user language variant", see Manual:Language on MediaWiki.org ) Bug: ProfessionalWiki#694
Make content-language-bounded messages display in user interface language
This fix the issue that the system message "maps-geo-json-create-page-button", "maps-loading-map" cannot display in language variants ("page view language" aka "user language variant", see Manual:Language on MediaWiki.org )
Bug: #694