technabob

What’s the easiest way to upgrade to the development version? I’m used to using the built in WordPress upgrader. Or should I just wait for the update release to come out? Donncha, any idea when that might be?

Oh, and I second what has already been said – thanks for all your work on this plugin – you’ve made millions of sites much faster.

And…. Here ya go. Simply add this to your functions.php and it’ll automatically add 30px height to any oEmbed from YouTube. This is only a workaround until YouTube returns the proper values in its embedded HTML. Please let me know if you encounter any problems, but it seems to successfully handle everything I’ve tested thusfar.

//change_youtube_embed_height
function do_maths($matches) {
	$addheight= 30;//height to add
	$retstring= 'height="'.strval($matches[1]+$addheight).'"';
	return $retstring;
}

function change_ytembed_height($content) {
	$pattern = '(height=[\'\"](\d+)[\'\"])'; //pattern to look for
    if (false !== strpos($content,"youtube")) {
        $content = preg_replace_callback($pattern, do_maths, $content);
    }
    return $content;
}

add_filter('embed_oembed_html','change_ytembed_height');
//end change_youtube_embed_height

The amount of height to add is in the $addheight variable, and the pattern to match is in the $pattern variable, so it should be fairly easy to adapt this to do other math against values coming back in the HTML from oEmbed.

Nah, I’m not using Jetpack at all. I’m using the built-in wordpress embeds in WP 3.3, and get black bars on all 16×9 content from YouTube embeds.

I’d think a filter similar to this:
http://www.bsdeluxe.com/removing-the-height-from-wordpress-embeds/
or
http://wp-snippets.com/654/adjust-vimeo-auto-embed-size/

Would allow us to just modify the height attribute of all your YouTube embeds by adding the height of the bar.

Yes, works great – fixes some nagging issues with the still being all stretched in the old player too.

I only wish there were support for the IFRAME code instead of the Flash code, but maybe that’ll come at some point.

Viper should at least merge your patch into the main code if he has time (which I know he doesn’t 🙂

Yeah, that fixed it for me too. I upgraded using the automatic upgrade plugin and for some reason this file was missed. A manual upload of “swfupload.swf” fixed it right up.