Last modified: 2014-10-10 02:57:44 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T72473, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 70473 - HTMLMultiSelectField checkboxes do not use MW UI
HTMLMultiSelectField checkboxes do not use MW UI
Status: RESOLVED FIXED
Product: MediaWiki
Classification: Unclassified
MediaWiki UI (Other open bugs)
unspecified
All All
: Normal normal (vote)
: ---
Assigned To: Prateek Saxena
:
Depends on:
Blocks: 70424
  Show dependency treegraph
 
Reported: 2014-09-06 03:51 UTC by Quiddity
Modified: 2014-10-10 02:57 UTC (History)
6 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Quiddity 2014-09-06 03:51:25 UTC
The checkboxes at http://mwui.wmflabs.org/wiki/Special:EditWatchlist
are not styled
Comment 1 Matthew Flaschen 2014-09-09 04:38:57 UTC
Thanks for the report.  The underlying code has a TODO, but this will help track it better.

Special:EditWatchlist uses EditWatchlistCheckboxSeriesField, which extends HTMLMultiSelectField.

HTMLMultiSelectField has a TODO to use checkLabel. I'll make this bug about that class.
Comment 2 Florian 2014-10-03 11:28:49 UTC
(In reply to Matthew Flaschen from comment #1)
> HTMLMultiSelectField has a TODO to use checkLabel. I'll make this bug about
> that class.

I wondering, if checkLabel can be used there. The label is html content (the link to the page), and checkLabel will escape all html code from label.

And HTMLMultiSelectField has a check for UseMediaWikiUIEverywhere, this should be removed to use mw ui all the time, right? :)
Comment 3 Matthew Flaschen 2014-10-10 02:57:32 UTC
Fixed by Prateek in 358d2980cc6c817e727dfaf166bf975a57f20950 and 9b461e218ad9b804435ef3e99bdedfd3759e02d7

(In reply to Florian from comment #2)
> (In reply to Matthew Flaschen from comment #1)
> > HTMLMultiSelectField has a TODO to use checkLabel. I'll make this bug about
> > that class.
> 
> I wondering, if checkLabel can be used there. The label is html content (the
> link to the page), and checkLabel will escape all html code from label.

Yeah, maybe not.  There is some trickiness in this class's labels, which is why it doesn't already use checkLabel.

> And HTMLMultiSelectField has a check for UseMediaWikiUIEverywhere, this
> should be removed to use mw ui all the time, right? :)

Yes, I want to get rid of UseMediaWikiUIEverywhere entirely as soon as we're ready (using the true behavior always).

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links