[svn r16087] Lp Improvements FS#2841

main/css/public_admin/scorm.css

@@ -21,7 +21,7 @@
 .inner_lp_toc{
 	overflow: auto;
 	background-color: white;
-	height: 600px;
+	height: 530px;
 	width: 90%;
 	border-right: 1px #999999 solid;
 	border-left: 15px #FFF solid;

main/newscorm/display_audiorecorder.php

@@ -38,13 +38,12 @@ $html='';
 
 if ($_GET['show_audio'])
 {	
-	$html='<div id="preview_image">'."\n";		
+	echo '<div id="preview_image">'."\n";		
 	if ($_SESSION['oLP']->get_preview_image()!='')
-		$html .='<img alt="'.$_SESSION['oLP']->get_author().'" src='.api_get_path(WEB_COURSE_PATH).api_get_course_path().'/upload/learning_path/images/'.$_SESSION['oLP']->get_preview_image().'>';
+		echo '<img alt="'.$_SESSION['oLP']->get_author().'" src='.api_get_path(WEB_COURSE_PATH).api_get_course_path().'/upload/learning_path/images/'.$_SESSION['oLP']->get_preview_image().'>';
-	else
+	else 
-		$html .=Display::display_icon('unknown.jpg',$_SESSION['oLP']->get_author());
+		 echo Display::display_icon('unknown.jpg',$_SESSION['oLP']->get_author());
-	$html .='</div>';
+	echo '</div>';		
-	echo $html;		
 	
 	echo '<div id="audiorecorder">	';
 	$audio_recorder_studentview = 'true';
@@ -57,13 +56,12 @@ if ($_GET['show_audio'])
 }
 else
 {
-	$html='<div id="preview_image">'."\n";		
+	echo '<div id="preview_image">'."\n";		
 	if ($_SESSION['oLP']->get_preview_image()!='')
-		$html .='<img alt="'.$_SESSION['oLP']->get_author().'" src='.api_get_path(WEB_COURSE_PATH).api_get_course_path().'/upload/learning_path/images/'.$_SESSION['oLP']->get_preview_image().'>';
+		echo '<img alt="'.$_SESSION['oLP']->get_author().'" src='.api_get_path(WEB_COURSE_PATH).api_get_course_path().'/upload/learning_path/images/'.$_SESSION['oLP']->get_preview_image().'>';
 	else
-		$html .=Display::display_icon('unknown.jpg',$_SESSION['oLP']->get_author());
+		echo Display::display_icon('unknown.jpg',$_SESSION['oLP']->get_author());
-	$html .='</div>'; 
+	echo '</div>';		
-	echo $html;	
 }
 
 echo '</body></html>';

main/newscorm/lp_header.php

@@ -57,7 +57,7 @@ if($show_link)
 // else we don't display get_lang(ucfirst(TOOL_LEARNPATH)) in the breadcrumb since the learner accessed it directly from the course homepage
 $interbreadcrumb[] = array("url"=>"./lp_controller.php?action=view&lp_id=".$path_id,'name'=>$path_name);
 $noPHP_SELF = true;
-echo $lp_theme_css=$_SESSION['oLP']->get_theme();
+$lp_theme_css=$_SESSION['oLP']->get_theme();
 include('../inc/reduced_header.inc.php');
 echo '<div style="padding-left: 17px;">';
 echo '<a class="link" href="./lp_controller.php?action=return_to_course_homepage" target="_top" onclick="window.parent.API.save_asset();">'.get_lang('CourseHomepageLink').'</a>';

main/newscorm/lp_view.php

@@ -174,10 +174,10 @@ else
 	//set flag to ensure lp_header.php is loaded by this script (flag is unset in lp_header.php)
 	$_SESSION['loaded_lp_view'] = true;	
 	$audio_record_width='';	
-	if ($displayAudioRecorder || $_SESSION['oLP']->get_preview_image()!='')
+	if (1 || $_SESSION['oLP']->get_preview_image()!='')
 	{		
-		if ($displayAudioRecorder)
+		if (1)
-			$audio_record_width='220,';
+			$audio_record_width='170,';
 		else
 			$audio_record_width='120,';
 	}
@@ -187,12 +187,12 @@ else
 	}	
 	?>
 		<frameset cols="270,*">
-			<frameset rows="30,<?php echo $audio_record_width; ?>50,20,*,20">
+			<frameset rows="20,<?php echo $audio_record_width; ?>50,20,*,20">
                 <frame id="header" src="lp_header.php"  border="0" frameborder="0" scrolling="no"/>
 				<?php 
-				if ($displayAudioRecorder || $_SESSION['oLP']->get_preview_image()!='')
+				if (1 || $_SESSION['oLP']->get_preview_image()!='')
 				{
-					if($displayAudioRecorder) //if audio recorder is required (set a few lines above)
+					if(1) //if audio recorder is required (set a few lines above)
 						echo '<frame id="audiorecorder_id" name="audiorecorder_name" src="display_audiorecorder.php?show_audio=1" border="0" frameborder="0" scrolling="no"/>';
 					else
 						echo '<frame id="audiorecorder_id" name="audiorecorder_name" src="display_audiorecorder.php?show_audio=0" border="0" frameborder="0" scrolling="no"/>';