Make WordPress Core

Opened 4 years ago

Closed 20 months ago

#11436 closed enhancement (wontfix)

Adding wmode transparent to flash videos

Reported by: aldolat Owned by: Viper007Bond
Milestone: Priority: high
Severity: normal Version: 2.9
Component: Embeds Keywords: has-patch
Focuses: Cc:


WP 2.9 should add this line

<param name='wmode' value='transparent' />

among other params inside the object tag when inserting a video from Youtube and others sites.

Also, it should add

<embed ... wmode="transparent" ... >

in the the embed tag.

While these parameters are already present in the press-this.php file, the standard writing panel doesn't include them.

If these declarations are not present, the video overlaps another element in the page that should be on top of others elements (such as a dropdown menu that would be hidden by a flash video).

Attachments (3)

11436.patch (524 bytes) - added by SergeyBiryukov 3 years ago.
11436.2.patch (681 bytes) - added by SergeyBiryukov 3 years ago.
Added check if wmode is already set.
11436.3.patch (638 bytes) - added by hakre 3 years ago.

Download all attachments as: .zip

Change History (25)

comment:1 Denis-de-Bernardy4 years ago

  • Milestone changed from Unassigned to 3.0
  • Priority changed from normal to high

+1. It's a true annoyance when it's not present. We should also add it when code returned by oembed doesn't add it already.

comment:2 hakre4 years ago

Keep in mind that wmode="transparent" does use more reources on the client's system than the opaque variant.

comment:3 hakre4 years ago

  • Keywords reporter-feedback added

Please provide steps on how to reproduce that it's not inserted right now.

comment:4 aldolat4 years ago

Step 1
When you want to publish a video from Youtube (for example), you paste the URL to the video. So paste this into the writing panel:

Step 2
Publish the post.

Step 3
Go to your blog. The video appears above any other element, such as a dropdown menu. If your wordpress theme has a dropdown menu, it should appear above the video, not below. Here a screenshot:

comment:5 Denis-de-Bernardy4 years ago

  • Keywords needs-patch added; reporter-feedback removed

comment:6 hakre4 years ago

Could reproduce. Is this only for oembed or are there other areas that should be considered to care by a patch.

comment:7 Denis-de-Bernardy4 years ago

  • Type changed from defect (bug) to enhancement

there are plentiful other areas. basically, anything that potentially adds an object tag is subject to this.

an additional improvement, while we're at it, could be to automatically use SWFObject.

comment:8 nacin4 years ago

  • Component changed from Media to Embeds
  • Owner set to Viper007Bond
  • Status changed from new to assigned

comment:9 nacin4 years ago

I'm strongly tempted to wontfix this. I'm not sure we should try to mess with oembed responses and should leave it to a plugin for when the oembed provider doesn't include it.

comment:10 Viper007Bond4 years ago

Yes, opaque is better than transparent.

I'm on the fence on this one though. oEmbed exists so we don't have to mess around with the embed HTML. YouTube also doesn't provide this parameter in their embed HTML anywhere, even on the standard video page where you copy/paste it. They used to have it, but removed it as it is generally considered a harmful parameter unless you absolutely need it. This is because the wmode parameter makes the player inaccessible to keyboard users (they can't tab into it and it's controls).

On the other hand, this can be a really annoying issue. I'm making the new version of my Video Quicktags plugin have an option to add the wmode parameter for example

I think I'm going to recommend wontfix with the reasoning being that YouTube knows what they are doing and it's easy for a plugin to fix this using the embed_oembed_html filter.

comment:11 follow-up: aldolat4 years ago

I agree. But how about adding this option into WordPress?

comment:12 nacin4 years ago

  • Resolution set to wontfix
  • Status changed from assigned to closed

aldolat: A plugin can easily handle this by using a filter that currently exists in WordPress and hooking into and 'filtering' the HTML.

comment:13 nacin4 years ago

  • Milestone 3.0 deleted

comment:14 in reply to: ↑ 11 Viper007Bond4 years ago

Replying to aldolat:

I agree. But how about adding this option into WordPress?

Too many options is a bad thing. You get lost in them all. WordPress aims to only provide a UI for the really important things and leave the rest to filters which plugins can use to change things.

comment:15 SergeyBiryukov3 years ago

Admin Bar makes this more apparent: #16537

SergeyBiryukov3 years ago

comment:16 SergeyBiryukov3 years ago

Does it make sense to add wmode only if Admin Bar is displayed? Attaching the patch.

comment:17 Denis-de-Bernardy3 years ago

Should this get re-opened considering #16537?

comment:18 SergeyBiryukov3 years ago

  • Keywords has-patch added; needs-patch removed
  • Resolution wontfix deleted
  • Status changed from closed to reopened

Reopening for consideration, as per the comments above.

comment:19 ocean903 years ago

  • Milestone set to Awaiting Review

SergeyBiryukov3 years ago

Added check if wmode is already set.

hakre3 years ago

comment:20 hakre3 years ago

@SergeyBiryukov: My review of your last patch.

comment:21 JonezJeA3 years ago

not only the admin bar was having this problem but also the window of posting images and etc. I have this patch is going to be released in 3.2 :D

comment:22 nacin20 months ago

  • Milestone Awaiting Review deleted
  • Resolution set to wontfix
  • Status changed from reopened to closed

No traction, re-closing.

If I had to guess, movement towards HTML5 has made this less and less of a reported issue?

Note: See TracTickets for help on using tickets.