Make WordPress Core

Opened 16 years ago

Closed 15 years ago

#5614 closed defect (bug) (wontfix)

Extra line in function end_el causes an extra gap in horizontal lists

Reported by: jashnu's profile jashnu Owned by:
Milestone: Priority: low
Severity: minor Version: 2.3.2
Component: General Keywords: has-patch
Focuses: Cc:


wp-includes/classes.php lines 535-541:

function end_el($output, $page, $depth) {

$output .= "</li>\n";

return $output;



\n causes an extra gap if you create a horizontal list so \n should be removed.

Attachments (1)

end_el-nonl.patch (329 bytes) - added by gtim 16 years ago.

Download all attachments as: .zip

Change History (6)

#1 @lloydbudd
16 years ago

  • Milestone changed from 2.3.3 to 2.5

#2 in reply to: ↑ description @sivel
16 years ago

I'm not sure I agree with making this change. Making this change will place all li tags on one line which does not follow best practice for lists.

#3 @lloydbudd
16 years ago

  • Milestone changed from 2.5 to 2.6

16 years ago

#4 @gtim
16 years ago

  • Keywords has-patch added

Added a patch in case it is decided that the advantages of removing \n outweighs not following the best practice.

#5 @Denis-de-Bernardy
15 years ago

  • Milestone 2.9 deleted
  • Resolution set to wontfix
  • Status changed from new to closed

the \n or a space is needed for accessibility. if it is not present, a text reader will potentially read the entire list as "foobar" (single word) instead of "foo" "bar".

Note: See TracTickets for help on using tickets.