Hello,
It would be useful if the end user can set the &overstretch settings for the flv player. Right now it is set to the default ('fit') which distorts some videos. I have a patch here that puts the setting for this into the control panel. The description below the control panel setting is from the flv player website verbatim.

Patch for flashvideo_objects.inc 2.1

27d26
<   $playback_vars = '&repeat=false&autostart='.$video['autostart'] . '&overstretch='.$video['overstretch'];
29c28
<   $flashvars = 'file='. $video['file'] . $img_vars . $img_intro . $time_vars . $logo_vars . $logo_link . $playback_vars;
---
>   $flashvars = 'file='. $video['file'] . $img_vars . $img_intro . $time_vars . $logo_vars . $logo_link . '&repeat=false&autostart='.$video['autostart'];

Patch for flashvideo.module

380,389d379
<    $form['player']['flashvideo_' . $node_type . '_overstretch'] = array(
<        '#title' => t('Overstretch'),
<        '#type' => 'select',
<        '#options' => array('true' => 'true', 'false' => 'false', 'fit' => 'fit', 'none' => 'none'),
<        '#default_value' => variable_get('flashvideo_' . $node_type . '_overstretch', 'fit'),
<        '#maxlength' =>  128,
<        '#description' => t('Defines how to stretch images/movies to make them fit the display. "true" will stretch them proportionally to fill the display, "false" will stretch them to fit. "fit" will stretch them disproportionally to fit both height and width. "none" will show all items in their original dimensions.'),
<        '#required' => TRUE
<        );
< 
915d904
<    $video['overstretch'] = (isset($params['overstretch'])) ? $params['overstretch'] :variable_get('flashvideo_' . $video['nodetype'] . '_overstretch', 'fit');
CommentFileSizeAuthor
flashvideo_objects.inc_.patch403 bytespapile

Comments

travist’s picture

Beautiful! Consider it added to the next release.

travist’s picture

Status: Needs review » Closed (won't fix)