Last modified: 2014-08-28 18:01:48 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 T71449, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 69449 - MediaWiki UI: VForm only partially documented
MediaWiki UI: VForm only partially documented
Status: NEW
Product: MediaWiki
Classification: Unclassified
MediaWiki UI (Other open bugs)
unspecified
All All
: Normal normal (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks: documentation
  Show dependency treegraph
 
Reported: 2014-08-12 23:37 UTC by Jon
Modified: 2014-08-28 18:01 UTC (History)
7 users (show)

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


Attachments

Description Jon 2014-08-12 23:37:36 UTC
MediaWiki UI has a concept of a mw-ui-vform but it is badly documented and it is not clear when, where and how to use it.

In the login form where it is used it also introduces mw-ui-checkbox-label, mw-ui-label, mw-ui-vform-field which are not documented anywhere in the style guide.

These classes should also be standalone without nesting and also have guidance on when they should be used.

mw-ui-checkbox-label also conflicts with the newly introduced mw-ui-checkbox so should probably be killed.
Comment 1 Matthew Flaschen 2014-08-13 23:15:40 UTC
This is something that should be cleaned up, but it is not major ("Major loss of function in an important area.", https://www.mediawiki.org/wiki/Bugzilla/Fields#Severity),

(In reply to Jon from comment #0)
> These classes should also be standalone without nesting

This is beyond the scope of a documentation bug.  It should be discussed and addressed separately.  Note even the new code (e.g. mw-ui-checkbox) still uses nesting in key places, albeit not in the same way.

> mw-ui-checkbox-label also conflicts with the newly introduced mw-ui-checkbox
> so should probably be killed.

Yes.  As discussed earlier, mw-ui-checkbox-label does not exist in the LESS (or CSS) and can simply be dropped.
Comment 2 Jon 2014-08-13 23:25:40 UTC
Not sure how it got assigned major that was not intentional :)

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


Navigation
Links