Make WordPress Core

Opened 13 years ago

Closed 13 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 13 years ago.
adding a wp-video class to video shortcodes

Download all attachments as: .zip

Change History (5)

#1 @goto10
13 years ago

  • Cc dromsey@… added

@nofearinc
13 years ago

adding a wp-video class to video shortcodes

#2 @nofearinc
13 years ago

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

#3 @SergeyBiryukov
13 years ago

  • Milestone changed from Awaiting Review to 3.8

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