Make WordPress Core

Context Navigation

← Previous Ticket
Next Ticket →

Opened 12 years ago

Closed 12 years ago

#27552 closed task (blessed) (fixed)

Smash "playlist" and "video-playlist" together

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

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 12 years ago.
27552.2.diff (17.1 KB ) - added by wonderboymusic 12 years ago.
27552.3.diff (17.3 KB ) - added by wonderboymusic 12 years ago.

Download all attachments as: .zip

Change History (6)

@wonderboymusic
12 years ago

Attachment 27552.diff added

#1 @ocean90
12 years ago

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

@wonderboymusic
12 years ago

Attachment 27552.2.diff added

@wonderboymusic
12 years ago

Attachment 27552.3.diff added

#2 @wonderboymusic
12 years ago

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
12 years ago

Resolution → fixed
Status new → closed

Congrats, everyone!

Note: See TracTickets for help on using tickets.

Download in other formats: