Simple patch to set proper indentation for ordered lists in blocks. Also removes the redundant block ul {margin: 0;} since that is already set by core.

CommentFileSizeAuthor
#3 drupal-bartik-883764-3.patch1006 bytestim.plunkett
#1 list-indentation.png13.16 KBJeff Burnz
bartik-block-ol-list-indentation.patch1.06 KBJeff Burnz

Comments

Jeff Burnz’s picture

Jeff Burnz commented
Status: Active » Needs review
StatusFileSize
new list-indentation.png13.16 KB

Screen shot.
list-indentation.png

Jeff Burnz’s picture

Jeff Burnz commented

bartik-block-ol-list-indentation.patch queued for re-testing.

tim.plunkett’s picture

tim.plunkett
he/him
Primary language English
Location Philadelphia
commented
StatusFileSize
new drupal-bartik-883764-3.patch1006 bytes

The padding: 0 0 0.25em 1em; was added to .block ul in #885228: CSS Files are in major need of clean up!, and I'm not sure we want to remove it anymore. Instead, how about this?

Jeff Burnz’s picture

Jeff Burnz commented
Status: Needs review » Reviewed & tested by the community

Lets do it, its basically the same as the first patch.

dries’s picture

dries commented
Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD. Thanks.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.