Make WordPress Core

Opened 12 years ago

Closed 12 years ago

#27637 closed enhancement (fixed)

Remove <br>s for HTML5 galleries

Reported by: obenland Owned by: nacin
Priority: normal Milestone: 3.9
Component: Media Version:
Severity: normal Keywords: has-patch
Cc: Focuses:

Description

HTML5 support for galleries was introduced recently (r27302). Shortly after that, we decided to seize the opportunity and disable default gallery styles, when a theme supports HTML5 galleries (r27396).

Last week, Nacin wondered if it would make sense to also remove the <br> element, that gets inserted to force line breaks. After we weighed pros and cons I asked some colleagues for their opinion, and they unanimously agreed that we should remove them. So let's do it and improve the markup even further.

Attachments (2)

27637.diff (591 bytes ) - added by obenland 12 years ago.
27637.2.diff (2.4 KB ) - added by obenland 12 years ago.
Includes support for 2013 and 2014

Download all attachments as: .zip

Change History (7)

@obenland
12 years ago

This ticket was mentioned in IRC in #wordpress-dev by nacin. View the logs.


12 years ago

@obenland
12 years ago

Includes support for 2013 and 2014

#2 @nacin
12 years ago

  • Owner set to nacin
  • Resolutionfixed
  • Status newclosed

In 27914:

Remove <br> elements for HTML5 galleries.

props obenland.
fixes #27637, see #26697.

#3 @obenland
12 years ago

  • Resolution fixed
  • Status closedreopened

Twenty Thirteen and Fourteen need a little attention on this.

#4 @lancewillett
12 years ago

In 27926:

Twenty Thirteen and Fourteen: style to support the new HTML5 line-break-less galleries. Props obenland, see #27637.

#5 @nacin
12 years ago

  • Resolutionfixed
  • Status reopenedclosed
Note: See TracTickets for help on using tickets.