WordPress.org

Make WordPress Core

Opened 7 years ago

Closed 7 years ago

#25896 closed enhancement (fixed)

Wrapping div surrounding video shortcode cannot be reffered easily

Reported by: AlphaK Owned by: wonderboymusic
Milestone: 3.8 Priority: normal
Severity: minor Version: 3.6
Component: Media Keywords: has-patch
Focuses: Cc:

Description

In changeset [24789], I found it was very annoying having that div wrapping the video tag, without a class or anything to refer the div using CSS...
It makes for instance very difficult to center or to style the video without using javascript, as the wrapping div has to be targeted.

I thing adding a class to the div would greatly help.

Attachments (1)

25896.diff (621 bytes) - added by nofearinc 7 years ago.
adding a wp-video class to video shortcodes

Download all attachments as: .zip

Change History (5)

#1 @goto10
7 years ago

  • Cc dromsey@… added

@nofearinc
7 years ago

adding a wp-video class to video shortcodes

#2 @nofearinc
7 years ago

  • Cc mario@… added
  • Keywords has-patch added

#3 @SergeyBiryukov
7 years ago

  • Milestone changed from Awaiting Review to 3.8

#4 @wonderboymusic
7 years ago

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

In 26526:

Add a wp-video class to the div that wraps video shortcode output. It was previously hard to target.

Props nofearinc.
Fixes #25896.

Note: See TracTickets for help on using tickets.