From 5030c9d35b76d4e6da5b898b8ee4fef56b736a38 Mon Sep 17 00:00:00 2001
From: Diana Huang <dkh@edx.org>
Date: Fri, 7 Dec 2012 11:20:05 -0500
Subject: [PATCH] Cleanup of the meta information styles and make the feedback
 box less important.

---
 .../src/staff_grading/staff_grading.coffee    |  8 ++++----
 lms/static/sass/course/_staff_grading.scss    | 16 +++++++++++++--
 lms/templates/instructor/staff_grading.html   | 20 +++++++++----------
 3 files changed, 28 insertions(+), 16 deletions(-)

diff --git a/lms/static/coffee/src/staff_grading/staff_grading.coffee b/lms/static/coffee/src/staff_grading/staff_grading.coffee
index e5fba57710..abcf99e40b 100644
--- a/lms/static/coffee/src/staff_grading/staff_grading.coffee
+++ b/lms/static/coffee/src/staff_grading/staff_grading.coffee
@@ -98,7 +98,7 @@ The standard chunk of Lorem Ipsum used since the 1500s is reproduced below for t
           message: 'No more submissions'
 
 
-    if @mock_cnt % 3 == 0
+    if @mock_cnt % 7 == 0
       response =
         success: false
         error: 'An error for testing'
@@ -340,9 +340,9 @@ class StaffGrading
     else if @state == state_grading
       @ml_error_info_container.html(@ml_error_info)
       meta_list = $("<ul>")
-      meta_list.append("<li><span class='meta-info'>Pending:</span> #{@num_pending}</li>")
-      meta_list.append("<li><span class='meta-info'>Graded:</span> #{@num_graded}</li>")
-      meta_list.append("<li><span class='meta-info'>Needed for ML:</span> #{Math.max(@min_for_ml - @num_graded)}</li>")
+      meta_list.append("<li><span class='meta-info'>Pending - </span> #{@num_pending}</li>")
+      meta_list.append("<li><span class='meta-info'>Graded - </span> #{@num_graded}</li>")
+      meta_list.append("<li><span class='meta-info'>Needed for ML - </span> #{Math.max(@min_for_ml - @num_graded)}</li>")
       @problem_meta_info.html(meta_list)
 
       @prompt_container.html(@prompt)
diff --git a/lms/static/sass/course/_staff_grading.scss b/lms/static/sass/course/_staff_grading.scss
index 3de22d40bb..f1b6c5845d 100644
--- a/lms/static/sass/course/_staff_grading.scss
+++ b/lms/static/sass/course/_staff_grading.scss
@@ -1,6 +1,6 @@
 div.staff-grading {
     textarea.feedback-area {
-        height: 100px;
+        height: 75px;
         margin: 20px;
     }
 
@@ -53,7 +53,19 @@ div.staff-grading {
     {
         background-color: #eee;
         padding:15px;
-        margin-left:0px;
+        h3
+        {
+            font-size:1em;
+        }
+        ul
+        {
+            list-style-type: none;
+            font-size: .85em;
+            li
+            {
+                margin: 5px 0px;
+            }
+        }
     }
     .message-container
     {
diff --git a/lms/templates/instructor/staff_grading.html b/lms/templates/instructor/staff_grading.html
index dd5aa9ae08..f7c0d4d568 100644
--- a/lms/templates/instructor/staff_grading.html
+++ b/lms/templates/instructor/staff_grading.html
@@ -17,14 +17,6 @@
 <section class="container">
 
 <div class="staff-grading" data-ajax_url="${ajax_url}">
-<div class="meta-info-wrapper">
-    <h3>Problem Information</h3>
-    <div class="problem-meta-info-container">
-    </div>
-    <h3>Maching Learning Information</h3>
-    <div class="ml-error-info-container">
-    </div>
-</div>
 <h1>Staff grading</h1>
 <div class="breadcrumbs">
 </div>
@@ -45,6 +37,14 @@
 
 <section class="prompt-wrapper">
 <h2 class="prompt-name"></h2>
+<div class="meta-info-wrapper">
+    <h3>Problem Information</h3>
+    <div class="problem-meta-info-container">
+    </div>
+    <h3>Maching Learning Information</h3>
+    <div class="ml-error-info-container">
+    </div>
+</div>
 <div class="prompt-information-container">
 <h3>Question</h3>
 <div class="prompt-container">
@@ -72,10 +72,10 @@
 
 <div class="grading-container">
 <div class="evaluation">
-  <textarea name="feedback" placeholder="Feedback for student..."
-            class="feedback-area" cols="70" rows="10"></textarea>
   <p class="score-selection-container">
   </p>
+  <textarea name="feedback" placeholder="Feedback for student (optional)"
+            class="feedback-area" cols="70" ></textarea>
 </div>