From f297b44eb7b6c403dd6cf6f8330a148ecc535fc8 Mon Sep 17 00:00:00 2001
From: Dave Goodwin <gingerdog@gmail.com>
Date: Wed, 5 Oct 2011 20:44:54 +0000
Subject: [PATCH] move feedback message to where the user can see it (hiding
 off hte screen is bad)

git-svn-id: https://xerteonlinetoolkits.googlecode.com/svn/trunk@127 912cdd6b-5c7d-d5a7-a2ba-d0f0cdb91641
---
 modules/xerte/peer.php | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/modules/xerte/peer.php b/modules/xerte/peer.php
index a200b7ed13..331075ac64 100644
--- a/modules/xerte/peer.php
+++ b/modules/xerte/peer.php
@@ -25,7 +25,7 @@ function show_template($row_play){
 
 		$string_for_flash = $xerte_toolkits_site->users_file_area_short . $row_play['template_id'] . "-" . $row_play['username'] . "-" . $row_play['template_name'] . "/";
 
-		$dimension = split("~",get_template_screen_size($row_play['template_name'],$row_play['template_framework']));
+		list($x, $y) = explode("~",get_template_screen_size($row_play['template_name'],$row_play['template_framework']));
 
 		echo file_get_contents("modules/" . $row_play['template_framework'] . "/peer_" . $row_play['template_framework'] . "_top");
 
@@ -33,14 +33,15 @@ function show_template($row_play){
 		* slightly modified xerte preview code to allow for flash vars
 		*/
 
-		echo "myRLO = new rloObject('" . $dimension[0] . "','" . $dimension[1] . "','modules/" . $row_play['template_framework'] . "/parent_templates/" . $row_play['template_name'] . "/" . $row_play['template_name'] . ".rlt','$string_for_flash', '$string_for_flash_xml', '$xerte_toolkits_site->site_url')";
+		echo "myRLO = new rloObject('" . $x . "','" . $y . "','modules/" . $row_play['template_framework'] . "/parent_templates/" . $row_play['template_name'] . "/" . $row_play['template_name'] . ".rlt','$string_for_flash', '$string_for_flash_xml', '$xerte_toolkits_site->site_url')";
 
 		echo "</script>";
 
-		echo "<br><form name=\"peer\" action=\"javascript:send_review('" . $row_play['username'] . "','" . $row_play['template_id'] . "')\" method=\"post\" enctype=\"text/plain\"><textarea style=\"width:800px; height:300px;\" name=\"response\">You have been asked to provide some feedback on this learning object. Please enter your feedback and click save when you have finished. This feedback is anonymous.</textarea><br/><input type=\"image\" src=\"website_code/images/Bttn_SaveOff.gif\" onmouseover=\"this.src='website_code/images/Bttn_SaveOn.gif'\" onmousedown=\"this.src='website_code/images/Bttn_SaveClick.gif'\" onmouseout=\"this.src='website_code/images/Bttn_SaveOff.gif'\" /></form><a name=\"feedbackform\"><p style=\"width:250px;\"  id=\"feedback\"></p></a></div>";
+        echo "<a name=\"feedbackform\"><p style=\"width:250px; color:red;\"  id=\"feedback\"></p></a>";
+		echo "<br><form name=\"peer\" action=\"javascript:send_review('" . $row_play['username'] . "','" . $row_play['template_id'] . "')\" method=\"post\" enctype=\"text/plain\"><textarea style=\"width:800px; height:300px;\" name=\"response\">You have been asked to provide some feedback on this learning object. Please enter your feedback and click save when you have finished. This feedback is anonymous.</textarea><br/><input type=\"image\" src=\"website_code/images/Bttn_SaveOff.gif\" onmouseover=\"this.src='website_code/images/Bttn_SaveOn.gif'\" onmousedown=\"this.src='website_code/images/Bttn_SaveClick.gif'\" onmouseout=\"this.src='website_code/images/Bttn_SaveOff.gif'\" /></form></div>";
 
 		echo "</body></html>";
 
 }
 
-?>
\ No newline at end of file
+?>