Twenty Thirteen: Make videos in video post formats 724px wide

[obenland]

Image and video post formats are supposed to display their image/video content at a width of 724px. All other content (if any) should stay at 604px. Image post formats are doing this already, we have to look at videos though.

Only problem with videos is that we need to change wp_embed_defaults() width value to 724px, so iframe can scale properly.

Mamaduka -- we can do it by filtering content_width right?

Lance we're already doing it on template_redirect, but this only would work on single post view, it isn't working on post archives.

Another option is to filter wp_embed_defaults and hard-code width for video post formats.

23542.diff​ Filters embed defaults, aslo add CSS styling to center video embeds, same way as we do with images.

Would this also work for legacy video posts from pre-activation?

Did you run in an instance where it is not called in a post context and where a if ( has_post_format( 'video' ) ) wouldn't suffice?

Replying to obenland:

Would this also work for legacy video posts from pre-activation?

Did you run in an instance where it is not called in a post context and where a if ( has_post_format( 'video' ) ) wouldn't suffice?

Yes it would work, tested by switching between default themes.

Can give more specific example for "not in post context" case?

Replying to Mamaduka:

Can give more specific example for "not in post context" case?

I don't think we need to check the post object, has_post_format() will do that for us.

Let's wait a bit on this ticket -- we'll be soon dropping in the 3.6 Post Formats "structured-post-formats" support, and it'll mean we can style the video separately from the rest of the post content.

Depends now on #23620.

Just a note: 'defaults' is misspelled in this patch for 'height'.

Replying to mindctrl:

Just a note: 'defaults' is misspelled in this patch for 'height'.

Thanks for the note. Chances are we'll avoid this patch altogether based on #23620.

Duplicate of #23620.