Make WordPress Core

Opened 19 years ago

Closed 17 years ago

#5614 closed defect (bug) (wontfix)

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

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

Description

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 18 years ago.

Download all attachments as: .zip

Change History (6)

#1 @lloydbudd
19 years ago

  • Milestone 2.3.32.5

#2 in reply to: ↑ description @sivel
19 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
19 years ago

  • Milestone 2.52.6

@gtim
18 years ago

#4 @gtim
18 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
17 years ago

  • Milestone 2.9
  • Resolutionwontfix
  • Status newclosed

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.