Make WordPress Core

Opened 11 years ago

Closed 11 years ago

#25896 closed enhancement (fixed)

Wrapping div surrounding video shortcode cannot be reffered easily

Reported by: alphak's profile AlphaK Owned by: wonderboymusic's profile 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 11 years ago.
adding a wp-video class to video shortcodes

Download all attachments as: .zip

Change History (5)

#1 @goto10
11 years ago

  • Cc dromsey@… added

@nofearinc
11 years ago

adding a wp-video class to video shortcodes

#2 @nofearinc
11 years ago

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

#3 @SergeyBiryukov
11 years ago

  • Milestone changed from Awaiting Review to 3.8

#4 @wonderboymusic
11 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.