HEAD contains this code in media_enable():

$view_mode_settings['image']['media_preview'] = 'styles_file_square_thumbnail';
$view_mode_settings['image']['media_large'] = 'styles_file_large';
...
$view_mode_settings['video']['media_large'] = 'styles_file_large';

There are two things wrong with this:
1) It makes sense for video to use the same formatter as images for 'media_preview' as well as for 'media_large'. The whole point of the Styles module is that at this level, images and video should be treated the same way, and whatever differences we need in how to render video differently from images should be encapsulated within Styles settings.
2) Whenever $view_mode_settings['video']['media_large'] = 'styles_file_large'; was added, it was done so without a corresponding update function.

This patch addresses these two problems.

CommentFileSizeAuthor
media-video-styles.patch1.51 KBeffulgentsia

Comments

JacobSingh’s picture

Status: Needs review » Reviewed & tested by the community
effulgentsia’s picture

Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD.

Status: Fixed » Closed (fixed)

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