Closed (won't fix)
Project:
FlashVideo
Version:
5.x-2.1
Component:
User interface
Priority:
Normal
Category:
Feature request
Assigned:
Reporter:
Created:
19 Jun 2007 at 14:51 UTC
Updated:
25 Sep 2008 at 02:17 UTC
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');
| Comment | File | Size | Author |
|---|---|---|---|
| flashvideo_objects.inc_.patch | 403 bytes | papile |
Comments
Comment #1
travist commentedBeautiful! Consider it added to the next release.
Comment #2
travist commented