--- video.module.4.7 2006-10-18 16:30:26.000000000 -0700 +++ video.module 2006-10-19 10:43:00.000000000 -0700 @@ -839,7 +839,7 @@ $file = basename($url); $base_url = substr($url, 0, strrpos($url, '/')); - // this will be executed by not Internet Explorer browsers + // this will be executed by browsers other than Internet Explorer $output = ' @@ -852,7 +852,7 @@ codebase="http://download.macromedia.com/pub/shockwave/cabs/flash/swflash.cab#version=6,0,40,0"> ' . "\n"; - // params will be passed to both IE or not IE browsers + // params passed to all browsers $output .= ' @@ -956,29 +956,30 @@ */ function theme_video_play_quicktime($node) { //Increase the height to accommodate the player controls on the bottom. - $height = $node->videoy + 16; + $height = $node->videoy + 56; + $width = $node->videox + 120; $url = _video_get_fileurl($node->vidfile); - // this will be executed by not Internet Explorer browsers + // this will be NOT be executed by Internet Explorer $output = ' - - ' . "\n"; + ' . "\n"; // this will be executed by Internet Explorer $output .= '' . "\n"; + + ' . "\n"; - // params will be passed to both IE or not IE browsers + // params will be passed to all browsers $output .= ' ' . "\n" . _video_get_parameters($node) . '

'. t('Your browser is not able to display this multimedia content.') .'

-
'; // only one
needed becouse only one opening tag has been parsed by browsers +
'; // only one needed because only one opening tag has been parsed by browsers /* @@ -990,7 +991,7 @@ $output = _theme_video_format_play($output, t('http://www.apple.com/quicktime/download'), t('Link to QuickTime Download Page'), - t('Download latest Quicktime Player')); + t('Download latest Quicktime player')); return $output; } @@ -1008,41 +1009,41 @@ // in the height $node->videoy += 40; - // this will be executed by not Internet Explorer browsers - $output = ' - - ' . "\n"; - - // this will be executed by Internet Explorer - $output .= '' . "\n"; - - // params will be passed to both IE or not IE browsers - $output .= ' - - - - - - - - - - - - - ' - . _video_get_parameters($node) . - '

'. t('Your browser is not able to display this multimedia content.') .'

-
'; // only one needed becouse only one opening tag has been parsed by browsers - - + $url = _video_get_fileurl($node->vidfile); + + $output = ' + + + + + + + + + +
+ + + + + + + + + + + + + +
Launch in external player + '; $output = _theme_video_format_play($output, t('http://www.real.com/'), t('Link to Real'), - t('Download latest Realmedia Player')); + t('Download latest Realmedia player')); return $output; } @@ -1072,7 +1073,7 @@ classid="clsid:6BF52A52-394A-11d3-B153-00C04F79FAA6" > ' . "\n"; - // params will be passed to both IE or not IE browsers + // params will be passed to all browsers $output .= ' @@ -1082,7 +1083,7 @@ ' . _video_get_parameters($node) . '

'. t('Your browser is not able to display this multimedia content.') .'

-'; // only one needed becouse only one opening tag has been parsed by browsers +'; // only one needed because only one opening tag has been parsed by browsers $output = _theme_video_format_play($output, t('http://windowsupdate.microsoft.com/'),