WordPress.org

Make WordPress Core

Opened 2 years ago

Closed 2 years ago

#23871 closed defect (bug) (fixed)

Twenty Thirteen: add support and styling for Audio structured post format

Reported by: lancewillett Owned by: lancewillett
Milestone: 3.6 Priority: normal
Severity: normal Version:
Component: Bundled Theme Keywords: has-patch
Focuses: Cc:

Description

Take a stab at splitting out the_audio() and use the_extra_content() for the rest of post content (like explanatory text, if any entered).

Attachments (2)

23871.diff (5.0 KB) - added by obenland 2 years ago.
23871.1.diff (5.0 KB) - added by obenland 2 years ago.

Download all attachments as: .zip

Change History (9)

comment:1 @lancewillett2 years ago

Probably also changing "audio-content" to "entry-audio" to match video and image class values.

comment:2 @lancewillett2 years ago

Sorry—"entry-media" for the new wrapper div, not "entry-audio".

@obenland2 years ago

comment:3 @obenland2 years ago

  • Keywords has-patch added; needs-patch removed

comment:4 @lancewillett2 years ago

Patch note: Why do we need the extra div wrapper "audio-content"? Is that to create the visual effect?

comment:5 @obenland2 years ago

Yes, the dotted line separator.

Let me update the patch real quick, I just saw I forgot to remove something.

@obenland2 years ago

comment:6 @obenland2 years ago

updated.

comment:7 @lancewillett2 years ago

  • Owner set to lancewillett
  • Resolution set to fixed
  • Status changed from new to closed

In 23828:

Twenty Thirteen: add support and styling for Audio structured post format. Moves audio content out from the main post content, using the_audio() and the_extra_content(). Props obenland, fixes #23871.

Note: See TracTickets for help on using tickets.