The UploadWizard has a form for writing file descriptions. It has long lists of languages. Instead it should use a ULS component.
(Tagging good first task. While not a trivial one-line change, something like this was done in many other places across a bunch of extensions like Translate, Wikibase, and others, so the code for this can come here as well.)
| Subject | Repo | Branch | Lines +/- | |
|---|---|---|---|---|
| Add ULS for language dropdown in file description | mediawiki/extensions/UploadWizard | master | +219 -28 |
A couple notes:
(Removing good first task until sufficient info has been provided for a new contributor)
TLDR, you call the .uls() method as a jQuery plugin on the element, clicking which will show the selector. So instead of the current dropdown there should be a different element, e.g.:
$trigger = $( '<a>' ).attr( 'id', 'mw-uploadwizard-language-selector' );
$trigger.uls( {
onSelect: function ( language ) {
// Yadda yadda,
// whatever happens when the language is selected in the dropdown.
// Possibly very short.
},
languages: LIST // The languages that the dropdown shows
} );Real documentation:
https://github.com/wikimedia/jquery.uls/blob/master/README.md
A simple example from the Translate extension:
https://phabricator.wikimedia.org/diffusion/ETRA/browse/master/resources/js/ext.translate.special.translate.js
(Search for the setupLanguageSelector function.)
The OPTIONS must define a callback function for onSelect.
- ULS may support languages that we don't - i.e., languages for which there is no template. See e.g. https://commons.wikimedia.org/wiki/Template:fr for i18n templates that we support in UW.
The list of languages can (and should) be given in the languages parameter in the options. See above.
- Making UW depend on ULS may not be a good solution. Some non-WMF wikis use this extension (against our recommendation), and we're trying to make that easier for them, but adding a dependency is something we need to seriously think about.
It can be optional.
OK, thanks @Amire80 for the helpful response, I think we can leave this open for new people now :)
Hi, I'm very new to WMF code, so please forgive my ignorance ...
By "it can be optional" did you mean that the UploadWizard should detect that ULS is installed, and if it is then it should use that instead of the dropdown?
Yes. One easy way to do it in JavaScript is to check whether mw.uls is a defined object, although @santhosh may suggest something better.
Ok cool thanks
Another quick question - should the selected language be connected/related to the IME in the description field? As far as I can make out the IME is intended to be independent of the actual language (so you can enter, for example, a Chinese quote within English text), but I'm not sure my understanding is correct
@Amire80 would you have any insight into the following?
On this page http://thottingal.in/projects/js/jquery.uls/examples/ if I type the following into the js console in the browser
$('h1').uls();... then when I click on the h1 a universal language selector pops up
But on this page https://commons.wikimedia.org/wiki/Special:UploadWizard if I enter the same into the console I get this
TypeError: $(...).uls is not a function
Any ideas?
Ugh. That's because the ULS extension's JS code is not fully loaded when you initially open the page.
At the top of the page near your username there's a link with a name of a language (if you haven't changed it, it will be "English"). Click it, and a language selection will appear. On most pages, for optimizing performance, this code that shows it is loaded only after you click it.
After the panel appears, try running $(...).uls again, and it should work.
The JS module that has the .uls() function is called "ext.uls.mediawiki". To make sure that it's pre-loaded so you'll be able to use it, try doing something along these lines:
mw.loader.using( "ext.uls.mediawiki", function () {
$(...).uls( ... )
} );Change 377425 had a related patch set uploaded (by Cparle; owner: Cparle):
[mediawiki/extensions/UploadWizard@master] Add ULS for language dropdown in file description
Change 377425 merged by jenkins-bot:
[mediawiki/extensions/UploadWizard@master] Add ULS for language dropdown in file description
@Amire80 this is released now on beta if you would like to check it https://commons.wikimedia.beta.wmflabs.org/wiki/Special:UploadWizard