Page MenuHomePhabricator

Missing indentation for lists
Closed, ResolvedPublic

Description

Due to the reset stylesheet, there is 0 indentation for list items, because padding is set to 0. This can be a tad confusing at times, especially with multi level lists.


Version: unspecified
Severity: normal

Details

Reference
bz41543

Event Timeline

bzimport raised the priority of this task from to Needs Triage.Nov 22 2014, 12:55 AM
bzimport set Reference to bz41543.
bzimport added a subscriber: Unknown Object (MLST).

Please could you remove the change to Mobile.css - if we are to change this I'd rather we did this directly in the MobileFrontend extension. Whilst it is here there is a danger that it will be copied to other wikis which won't get updated when any change propagates to core.

Playing devils advocate why do we need indents on list?

Also we should limit this to lists in #content as there is a danger this will break existing interfaces.

(the original rationale was to have no margin on list items due to limited space on mobile and making the most of the mobile width) - regardless this should be discussed rather than pushed on to Mobile.css - we might want to only add the margin with a media query on larger screens for example.

The problem with no margin is in

ol
li

ol
 li

which now renders as:

  • *

instead of

  • *

Very true.
Lets add this to core.
Feel free to send a patch - otherwise I'll get one in sometime today

Less css rule would look something like this:
#content {

ul, ol {
  ul, ol {
     margin-left: 8px;
  }

}

Hmm, we also have dl.

And there is an issue on talk pages (or pages that are used as talk pages like WP:VP/T), where dl, ul and ol are expected by the editors to create initial indentation. This is an unusual usage of ul/ol/dl of course, but a very widely accepted usage throughout the wiki's.

Opening of discussion
:reply
*reply
**reply
#reply

Let's just apply https://en.wikipedia.org/w/index.php?title=MediaWiki%3AMobile.css&diff=520586981&oldid=520583307 to mf-typography.css - we can always visit this later.

Let's make sure we limit to lists in #content element however so as we do not interfere with lists in overlays, search, settings page etc...