From d75cdda064b00924a42abab77476e2967682edb6 Mon Sep 17 00:00:00 2001
From: Giulio Gratta <giulio@giuliogratta.com>
Date: Thu, 26 Sep 2013 11:22:16 -0700
Subject: [PATCH] An assortment of small OE changes, both for ICE and standard

- Changed save OE message text
- Fix ICE legend positioning and content
- Add space to "offensive" checkbox
- Change tracker variable in undo to correct one
---
 .../xmodule/xmodule/js/src/combinedopenended/display.coffee  | 2 +-
 .../xmodule/xmodule/js/src/peergrading/track_changes.coffee  | 2 +-
 lms/static/sass/course/_staff_grading.scss                   | 2 +-
 lms/templates/peer_grading/peer_grading_problem.html         | 5 +++--
 4 files changed, 6 insertions(+), 5 deletions(-)

diff --git a/common/lib/xmodule/xmodule/js/src/combinedopenended/display.coffee b/common/lib/xmodule/xmodule/js/src/combinedopenended/display.coffee
index d1006ae9a9..7103d2e841 100644
--- a/common/lib/xmodule/xmodule/js/src/combinedopenended/display.coffee
+++ b/common/lib/xmodule/xmodule/js/src/combinedopenended/display.coffee
@@ -353,7 +353,7 @@ class @CombinedOpenEnded
       @save_button.attr("disabled",true)
       $.postWithPrefix "#{@ajax_url}/store_answer", data, (response) =>
         if response.success
-          @gentle_alert("Answer saved.")
+          @gentle_alert("Answer saved, but not yet submitted.")
         else
           @errors_area.html(response.error)
         @save_button.attr("disabled",false)
diff --git a/common/lib/xmodule/xmodule/js/src/peergrading/track_changes.coffee b/common/lib/xmodule/xmodule/js/src/peergrading/track_changes.coffee
index ff0b4111f2..72df455b0b 100644
--- a/common/lib/xmodule/xmodule/js/src/peergrading/track_changes.coffee
+++ b/common/lib/xmodule/xmodule/js/src/peergrading/track_changes.coffee
@@ -56,7 +56,7 @@ class @TrackChanges
       key = parseInt(@attr('data-cid'))
       if key > keyOfLatestChange
         keyOfLatestChange = key
-    ICEtracker.rejectChange('[data-cid="'+ keyOfLatestChange + '"]')
+    @tracker.rejectChange('[data-cid="'+ keyOfLatestChange + '"]')
 
   stop_tracking_on_submit: () =>
     @tracker.stopTracking()
\ No newline at end of file
diff --git a/lms/static/sass/course/_staff_grading.scss b/lms/static/sass/course/_staff_grading.scss
index 6810fe1bff..fd48ac28dc 100644
--- a/lms/static/sass/course/_staff_grading.scss
+++ b/lms/static/sass/course/_staff_grading.scss
@@ -18,7 +18,7 @@ div.peer-grading{
         overflow: auto;
     }
 
-    div.feedback-area.track-changes, p.legend {
+    div.feedback-area.track-changes, p.ice-legend {
         .ice-controls {
             float: right;
         }
diff --git a/lms/templates/peer_grading/peer_grading_problem.html b/lms/templates/peer_grading/peer_grading_problem.html
index be357bf054..bc0bbdfa34 100644
--- a/lms/templates/peer_grading/peer_grading_problem.html
+++ b/lms/templates/peer_grading/peer_grading_problem.html
@@ -55,8 +55,8 @@
                                 <span class="del">${_("This is a deletion.")}</span>&nbsp;
                                 <span class="ins">${_("[This is a comment.]")}</span>&nbsp;
                                 <span class="ice-controls">
+                                    <a href="#" class="undo-change"><i class="icon-undo"></i> Undo Change</a>&nbsp;&nbsp;
                                     <a href="#" class="reset-changes"><i class="icon-refresh"></i> Reset Changes</a>
-                                    <a href="#" class="undo-change"><i class="icon-undo"></i> Undo Change</a>
                                 </span>
                             </p>
                             <div name="feedback" class="feedback-area track-changes" contenteditable="true"></div>
@@ -65,8 +65,9 @@
                             <textarea name="feedback" placeholder="Feedback for student" class="feedback-area" cols="70" ></textarea>
                         % endif
                         <div class="flag-student-container">
+                            <br />
                             <input type="checkbox" class="flag-checkbox" value="student_is_flagged"> 
-                            ${_("This submission has explicit, offensive, or (I suspect) plagiarized content: ")}
+                            ${_("This submission has explicit, offensive, or (I suspect) plagiarized content. ")}
                         </div>
                     </div>
                     <hr />