WordPress.org

Make WordPress Core

Opened 7 years ago

Closed 7 years ago

#27552 closed task (blessed) (fixed)

Smash "playlist" and "video-playlist" together

Reported by: wonderboymusic Owned by:
Milestone: 3.9 Priority: high
Severity: major Version: 3.9
Component: Media Keywords: has-patch
Focuses: javascript Cc:

Description

There shouldn't be 2 shortcodes. On top of that, there is a sweet shortcode-parsing bug related to -

Attachments (3)

27552.diff (17.2 KB) - added by wonderboymusic 7 years ago.
27552.2.diff (17.1 KB) - added by wonderboymusic 7 years ago.
27552.3.diff (17.3 KB) - added by wonderboymusic 7 years ago.

Download all attachments as: .zip

Change History (6)

@wonderboymusic
7 years ago

#1 @ocean90
7 years ago

Patch includes at the end: 'addToPlaylist' >= __( 'Add to playlist' ),.

#2 @wonderboymusic
7 years ago

In 27785:

There is no more video-playlist shortcode. To use video, it is now [playlist type="video" ....]. Also deleting core playlist styles. The style attribute is still supported, defaulting to light. Our core style support was 4-5 CSS rules.

See #27552. Leaving open for comments and potential bugs.

#3 @wonderboymusic
7 years ago

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

Congrats, everyone!

Note: See TracTickets for help on using tickets.