hey there, on sites with non-white backgrounds it would be nice if youtube and google videos had a wmode=transparent added in.
also it keeps them from interfering with nice-menus that drop over them

  // this will be executed by not Internet Explorer browsers
  $output = '<!--[if !IE]> <-->
<object type="application/x-shockwave-flash" wmode="transparent" width="'. $width .'" height="'. $height .'"
data="http://www.youtube.com/v/' . check_plain($node->vidfile) . '">
<!--> <![endif]-->' . "\n";

  // this will be executed by Internet Explorer
  $output .= '<!--[if IE]>
<object type="application/x-shockwave-flash" wmode="transparent" width="'. $width .'" height="'. $height .'"
classid="clsid:D27CDB6E-AE6D-11cf-96B8-444553540000"
codebase="http://download.macromedia.com/pub/shockwave/cabs/flash/swflash.cab#version=6,0,40,0">
<![endif]-->' . "\n";

Comments

ikke’s picture

Status: Active » Closed (won't fix)
ikke’s picture

Assigned: Unassigned » ikke
Status: Closed (won't fix) » Active
Veggieryan’s picture

why won't fix?

will this break something?
otherwise movies have ugly white borders on non-white backgrounds and drop down menus dont work.. LAME.

ikke’s picture

Set it to "won't fix" was a bad click to early in the morning. I change it back to "active" a minute later. Sorry for confusion.

Ikke

Veggieryan’s picture

are there any side effects that would keep this from being committed?

fax8’s picture

is the wmode attribute a valid W3C one?
is the code proposed valid W3C code?

If yes please roll out a patch and I will commit asap.

Fabio

EvanDonovan’s picture

Has this ever been committed to the 5.x version?

fax8’s picture

No. Because it turned out that this option is not W3 compatible. I'm trying to let video module to generate strict XHTML code.

Anyone knows any alternative?

hypertext200’s picture

Status: Active » Fixed

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.