Embedfield project - resolved XHTML strict for blip.tv provider

Kudos forcebook, nice post... I really think this or a variation on this should be swapped out in the next release of the actual module. After all, who wants invalid code on their video embed pages? I have tested this in IE 6 & 7 as well and it works like a charm.

thanks; committed to emfield v1. moving to media: bliptv for porting.

committed; sorry it took so long for me to get back to this... thanks @forcebook!

Now, after using this for a while, I realize that I never fully tested this. It works great for embedding and leaves you with nicely valid code, but there is an important parameter missing. The code above does not allow the full screen button to function, leaving a confusing experience for the user, so I've just added that line to the code for both non-IE and IE:

$output .= '<object classid="clsid:D27CDB6E-AE6D-11cf-96B8-444553540000" codebase="http://download.macromedia.com/pub/shockwave/cabs/flash/swflash.cab#version=6,0,40,0" width="' . $width .'" height="'. $height .'">
    <param name="movie" value="http://blip.tv/play/'. $embed_code . '" />
    <param name="quality" value="high" />
    <param name="bgcolor" value="#FFFFFF" />
	<param name="allowfullscreen" value="true" />
    <param name="wmode" value="transparent" />
    <!--[if !IE]> <-->
    <object data="http://blip.tv/play/'. $embed_code.'" width="'. $width .'" height="'. $height .'" type="application/x-shockwave-flash">
    <param name="quality" value="high" />
    <param name="bgcolor" value="#FFFFFF" />
	<param name="allowfullscreen" value="true" />
    <param name="pluginurl" value="http://www.adobe.com/go/getflashplayer" />
    FAIL (the browser should render some flash content, not this).
    </object>
    <!--> <![endif]-->
</object>';

This should be committed again, as this is going to be pretty important functionality for a lot of people.

awesome, thanks everyone! got it committed, will roll a new release shortly.