From a7f2d9677577a2d5a8e7bd7c9a9dc526452a7eec Mon Sep 17 00:00:00 2001
From: Bridger Maxwell <bridger@mit.edu>
Date: Fri, 10 Feb 2012 12:17:49 -0500
Subject: [PATCH 1/7] Added bar on progress graph to show overall grade.

--HG--
branch : bridger-dev
---
 profile_graphs.js | 40 +++++++++++++++++++++++++++++++++++-----
 1 file changed, 35 insertions(+), 5 deletions(-)

diff --git a/profile_graphs.js b/profile_graphs.js
index 80eb9dc3b0..43e15937d9 100644
--- a/profile_graphs.js
+++ b/profile_graphs.js
@@ -16,7 +16,7 @@ $(function () {
     }).appendTo("body").fadeIn(200);
   }
       
-  /* -------------------------------- Grade detail graph -------------------------------- */
+  /* -------------------------------- Grade detail bars -------------------------------- */
   var colors = [$.color.parse("#E8B800"), $.color.parse("#A0CEFA"), $.color.parse("#BD3738"), $.color.parse("#429A2E")];
   //var colors = [$.color.parse("#1B2045"), $.color.parse("#557a00"), $.color.parse("#F5600"), $.color.parse("#FEBA2C")];
   //var colors = [$.color.parse("#E7C856"), $.color.parse("#CD462E"), $.color.parse("#B01732"), $.color.parse("#41192A")];
@@ -82,9 +82,39 @@ $(function () {
   //Alwasy be sure that one series has the xaxis set to 2, or the second xaxis labels won't show up
   series.push( {label: 'Dropped Scores', data: droppedScores, points: {symbol: "cross", show: true, radius: 3}, bars: {show: false}, color: "red", xaxis: 2} );
       
+      
+  /* ----------------------------- Grade overviewew bar -------------------------*/
+  <%
+  totalWeight = 0.0
+  sectionIndex = 0
+  totalScore = 0.0
+  %>
+  %for section in grade_summary:
+    <%
+    weighted_score = section['totalscore']['score'] * section['weight']
+    summary_text = "{0} - {1:.0%} of possible {2:.0%}".format(section['category'], weighted_score, section['weight'])
+    %>
+    %if section['totalscore']['score'] > 0:
+      series.push({label: "${section['category']} - Weighted",
+        data: [[${tickIndex}, ${weighted_score}]],
+        color: colors[${sectionIndex}].toString(),
+      }); 
+    %endif
+     
+    detail_tooltips["${section['category']} - Weighted"] = [ "${summary_text}" ];
+    <%
+    sectionIndex += 1
+    totalWeight += section['weight']
+    totalScore += section['totalscore']['score'] * section['weight']
+    %>
+  %endfor
+  <% tickIndex += 1 + sectionSpacer %>
+  
+  
   var options = {
-    series: {lines: {show: false, steps: false },
-    bars: {show: true, barWidth: 0.6, align: 'center', lineWidth: 1},},
+    series: {stack: true,
+              lines: {show: false, steps: false },
+              bars: {show: true, barWidth: 0.6, align: 'center', lineWidth: 1},},
     xaxis: {tickLength: 0, min: 0.0, max: ${tickIndex - sectionSpacer}},
     xaxes: [{ticks: ticks}, {ticks: bottomTicks}],
     yaxis: {ticks: [[1, "100%"], [0.87, "A 87%"], [0.7, "B 70%"], [0.6, "C 60%"], [0, "0%"]], min: 0.0, max: 1.0, labelWidth: 50},
@@ -103,8 +133,8 @@ $(function () {
     $("#x").text(pos.x.toFixed(2));
     $("#y").text(pos.y.toFixed(2));
     if (item) {
-      if (previousPoint != item.dataIndex) {
-        previousPoint = item.dataIndex;
+      if (previousPoint != (item.dataIndex, item.seriesIndex)) {
+        previousPoint = (item.dataIndex, item.seriesIndex);
             
         $("#tooltip").remove();
             

From 59ff6ce7df96a9a2d3032ddb42839ba12fa26fc8 Mon Sep 17 00:00:00 2001
From: Bridger Maxwell <bridger@mit.edu>
Date: Fri, 10 Feb 2012 12:35:42 -0500
Subject: [PATCH 2/7] Added percentage label to course overview progress

--HG--
branch : bridger-dev
---
 profile_graphs.js | 16 ++++++++++++----
 1 file changed, 12 insertions(+), 4 deletions(-)

diff --git a/profile_graphs.js b/profile_graphs.js
index 43e15937d9..171cacc7f3 100644
--- a/profile_graphs.js
+++ b/profile_graphs.js
@@ -88,15 +88,16 @@ $(function () {
   totalWeight = 0.0
   sectionIndex = 0
   totalScore = 0.0
+  overviewBarX = tickIndex
   %>
   %for section in grade_summary:
     <%
     weighted_score = section['totalscore']['score'] * section['weight']
-    summary_text = "{0} - {1:.0%} of possible {2:.0%}".format(section['category'], weighted_score, section['weight'])
+    summary_text = "{0} - {1:.0%} of a possible {2:.0%}".format(section['category'], weighted_score, section['weight'])
     %>
     %if section['totalscore']['score'] > 0:
       series.push({label: "${section['category']} - Weighted",
-        data: [[${tickIndex}, ${weighted_score}]],
+        data: [[${overviewBarX}, ${weighted_score}]],
         color: colors[${sectionIndex}].toString(),
       }); 
     %endif
@@ -124,9 +125,16 @@ $(function () {
     legend: {show: false},
   };
   
-  if ($("#grade-detail-graph").length > 0) {
-    $.plot($("#grade-detail-graph"), series, options);
+  var $grade_detail_graph = $("#grade-detail-graph");
+  if ($grade_detail_graph.length > 0) {
+    var plot = $.plot($grade_detail_graph, series, options);
+    
+    var o = plot.pointOffset({x: ${overviewBarX} , y: ${totalScore}});
+    $grade_detail_graph.append('<div style="position:absolute;left:' + (o.left - 12) + 'px;top:' + (o.top - 20) + 'px">${"{:.0%}".format(totalScore)}</div>');
   }
+  
+
+  
       
   var previousPoint = null;
   $("#grade-detail-graph").bind("plothover", function (event, pos, item) {

From 69343aaedd81111711bea2d45f2243fe01ccb91a Mon Sep 17 00:00:00 2001
From: Bridger Maxwell <bridger@mit.edu>
Date: Fri, 10 Feb 2012 15:57:39 -0500
Subject: [PATCH 3/7] Rotated the profile graph labels, so the all fit

--HG--
branch : bridger-dev
---
 profile.html      |  4 ++--
 profile_graphs.js | 41 ++++++++++++++++++++++++++++-------------
 2 files changed, 30 insertions(+), 15 deletions(-)

diff --git a/profile.html b/profile.html
index 422d21f2b8..6e627005de 100644
--- a/profile.html
+++ b/profile.html
@@ -5,9 +5,9 @@
 %>
 
 <%block name="headextra">
-<script type="text/javascript" src="${ settings.LIB_URL }flot/jquery.flot.min.js"></script>
+<script type="text/javascript" src="${ settings.LIB_URL }flot/jquery.flot.js"></script>
 <script type="text/javascript" src="${ settings.LIB_URL }flot/jquery.flot.stack.js"></script>
-<script language="javascript" type="text/javascript" src="${ settings.LIB_URL }flot/jquery.flot.symbol.js"></script>
+<script type="text/javascript" src="${ settings.LIB_URL }flot/jquery.flot.symbol.js"></script>
 <script>
 <%include file="profile_graphs.js"/>
 </script>
diff --git a/profile_graphs.js b/profile_graphs.js
index 171cacc7f3..a24bdfb7c4 100644
--- a/profile_graphs.js
+++ b/profile_graphs.js
@@ -42,7 +42,7 @@ $(function () {
         data: ${ json.dumps( [[i + tickIndex, score['percentage']] for i,score in enumerate(section['subscores'])] ) },
         color: colors[${sectionIndex}].toString(),
       });
-      ticks = ticks.concat( ${ json.dumps( [[i + tickIndex, str(i + 1)] for i,score in enumerate(section['subscores'])] ) } );
+      ticks = ticks.concat( ${ json.dumps( [[i + tickIndex, score['label'] ] for i,score in enumerate(section['subscores'])] ) } );
       bottomTicks.push( [ ${tickIndex + len(section['subscores'])/2}, "${section['category']}" ] );
       detail_tooltips["${section['category']}"] = ${ json.dumps([score['summary'] for score in section['subscores']]  ) };
           
@@ -57,8 +57,8 @@ $(function () {
         data: [[${tickIndex}, ${section['totalscore']['score']}]],
         color: colors[${sectionIndex}].toString(),
       });
-      ticks = ticks.concat( [ [${tickIndex}, "Total"] ] );
-      detail_tooltips["${section['category']} Total"] = [ "${section['totalscore']['summary']}" ];
+      ticks = ticks.concat( [ [${tickIndex}, "${section['totallabel']}"] ] );
+      detail_tooltips["${section['category']} Average"] = [ "${section['totalscore']['summary']}" ];
       <% tickIndex += 1 + sectionSpacer %>
           
     %else: ##This is for sections like midterm or final, which have no smaller components
@@ -66,12 +66,8 @@ $(function () {
         data: [[${tickIndex}, ${section['totalscore']['score']}]],
         color: colors[${sectionIndex}].toString(),
       });
-      %if section['category'] == "Midterm":
-      bottomTicks = bottomTicks.concat( [ [${tickIndex}, "${section['category']}"] ] );
-      %else:
-      ticks = ticks.concat( [ [${tickIndex}, "${section['category']}"] ] );
-      %endif
-          
+      ticks = ticks.concat( [ [${tickIndex}," ${section['totallabel']}"] ] );
+      
       detail_tooltips["${section['category']}"] = [ "${section['totalscore']['summary']}" ];
 
       <% tickIndex += 1 + sectionSpacer %>
@@ -80,7 +76,7 @@ $(function () {
   %endfor
       
   //Alwasy be sure that one series has the xaxis set to 2, or the second xaxis labels won't show up
-  series.push( {label: 'Dropped Scores', data: droppedScores, points: {symbol: "cross", show: true, radius: 3}, bars: {show: false}, color: "red", xaxis: 2} );
+  series.push( {label: 'Dropped Scores', data: droppedScores, points: {symbol: "cross", show: true, radius: 3}, bars: {show: false}, color: "red"} );
       
       
   /* ----------------------------- Grade overviewew bar -------------------------*/
@@ -93,7 +89,7 @@ $(function () {
   %for section in grade_summary:
     <%
     weighted_score = section['totalscore']['score'] * section['weight']
-    summary_text = "{0} - {1:.0%} of a possible {2:.0%}".format(section['category'], weighted_score, section['weight'])
+    summary_text = "{0} - {1:.1%} of a possible {2:.0%}".format(section['category'], weighted_score, section['weight'])
     %>
     %if section['totalscore']['score'] > 0:
       series.push({label: "${section['category']} - Weighted",
@@ -109,6 +105,8 @@ $(function () {
     totalScore += section['totalscore']['score'] * section['weight']
     %>
   %endfor
+  ticks = ticks.concat( [ [${overviewBarX}, "Totals"] ] );
+  
   <% tickIndex += 1 + sectionSpacer %>
   
   
@@ -116,8 +114,7 @@ $(function () {
     series: {stack: true,
               lines: {show: false, steps: false },
               bars: {show: true, barWidth: 0.6, align: 'center', lineWidth: 1},},
-    xaxis: {tickLength: 0, min: 0.0, max: ${tickIndex - sectionSpacer}},
-    xaxes: [{ticks: ticks}, {ticks: bottomTicks}],
+    xaxis: {tickLength: 0, min: 0.0, max: ${tickIndex - sectionSpacer}, ticks: ticks, labelAngle: 90},
     yaxis: {ticks: [[1, "100%"], [0.87, "A 87%"], [0.7, "B 70%"], [0.6, "C 60%"], [0, "0%"]], min: 0.0, max: 1.0, labelWidth: 50},
     grid: { hoverable: true, clickable: true, borderWidth: 1,
       markings: [ {yaxis: {from: 0.87, to: 1 }, color: "#EBFFD5"}, {yaxis: {from: 0.7, to: 0.87 }, color: "#E6FFFF"}, 
@@ -131,6 +128,24 @@ $(function () {
     
     var o = plot.pointOffset({x: ${overviewBarX} , y: ${totalScore}});
     $grade_detail_graph.append('<div style="position:absolute;left:' + (o.left - 12) + 'px;top:' + (o.top - 20) + 'px">${"{:.0%}".format(totalScore)}</div>');
+    
+    // //Rotate the x-axis labels
+    // var rotateValue = "rotate(-60deg)";
+    // var rotateOrigin = "bottom left";
+    // $("#grade-detail-graph .x1Axis .tickLabel").css( {
+    //   '-webkit-transform': rotateValue,
+    //   '-moz-transform': rotateValue,
+    //   '-ms-transform': rotateValue,
+    //   '-o-transform': rotateValue,
+    //   'transform': rotateValue,
+    //   
+    //   '-webkit-transform-origin': rotateOrigin,
+    //   '-moz-transform-origin': rotateOrigin,
+    //   '-ms-transform-origin': rotateOrigin,
+    //   '-o-transform-origin': rotateOrigin,
+    //   
+    //   'text-align' : 'left',
+    // });
   }
   
 

From 885c3ac1a608dbb48b2022708e715a4607d37a15 Mon Sep 17 00:00:00 2001
From: Bridger Maxwell <bridger@mit.edu>
Date: Fri, 10 Feb 2012 16:57:00 -0500
Subject: [PATCH 4/7] The organization of simplewiki_base was a little messed
 up. Most content was being put in the header

--HG--
branch : bridger-dev
---
 simplewiki_base.html | 26 ++++++++++----------------
 1 file changed, 10 insertions(+), 16 deletions(-)

diff --git a/simplewiki_base.html b/simplewiki_base.html
index 17f110bf1a..0f4e70cb2c 100644
--- a/simplewiki_base.html
+++ b/simplewiki_base.html
@@ -65,35 +65,29 @@
 
   });
   </script>
-
-
-<%block name="bodyextra">
+  
   <%block name="wiki_head"/>
+  
 </%block>
 
+<%block name="bodyextra">
 <%include file="navigation.html" args="active_page='wiki'" />
 
 <section class="main-content">
   <div class="wiki-wrapper">
-      <%block name="wiki_panel">
-
+    <%block name="wiki_panel">
     <div id="wiki_panel">
-    <%
-        if (wiki_article is not UNDEFINED):
-        	baseURL = reverse("wiki_view", args=[wiki_article.get_url()])
-        else:
-        	baseURL = reverse("wiki_view", args=["/"])
-    %>
-
+      <%
+          if (wiki_article is not UNDEFINED):
+          	baseURL = reverse("wiki_view", args=[wiki_article.get_url()])
+          else:
+          	baseURL = reverse("wiki_view", args=["/"])
+      %>
       <ul class="action">
-        <!-- <li> -->
-        <!--   <input type="button" onclick="javascript:location.href='${reverse("wiki_random", args=[])}'" value="Random article" class="button" /> -->
-        <!-- </li> -->
         <li>
           <input type="button" onclick="javascript:location.href='${reverse("wiki_list_articles", args=[])}'" value="All articles" class="button" />
         </li>
         
-
         <li class="create-article">
           <h3>
             <a href="#" id="create-article"/>Create Article</a>

From 1921bd01baa28b24c9dd218cfb810311658547db Mon Sep 17 00:00:00 2001
From: Bridger Maxwell <bridger@mit.edu>
Date: Wed, 15 Feb 2012 14:20:26 -0500
Subject: [PATCH 5/7] Started implementing exit confirmation on wiki edit page.
 NOT WORKING YET

--HG--
branch : bridger-dev
---
 simplewiki_edit.html | 17 +++++++++++++++++
 1 file changed, 17 insertions(+)

diff --git a/simplewiki_edit.html b/simplewiki_edit.html
index 7d7a9c6bb8..0fb28f8d6e 100644
--- a/simplewiki_edit.html
+++ b/simplewiki_edit.html
@@ -8,6 +8,23 @@
 ${ wiki_article.title }
 </%block>
 
+<%block name="wiki_head">
+<script>
+  $(function() {
+    $("#id_contents").data('initial_contents') = $("#id_contents");
+    
+    
+    window.
+    
+  }
+
+  //On load, we save
+  
+  var blah;
+</script>
+</%block>
+
+
 <%block name="wiki_body">
 <form method="POST" id="wiki_revision">
   <div style="display:none">

From fdc4bd71beec10ebab1595c82001fa853542c88d Mon Sep 17 00:00:00 2001
From: Bridger Maxwell <bridger@mit.edu>
Date: Thu, 16 Feb 2012 18:16:01 -0500
Subject: [PATCH 6/7] Added check for unsaved changes before navigating away
 from wiki edit page.

--HG--
branch : bridger-dev
---
 simplewiki_edit.html | 17 ++++++++---------
 1 file changed, 8 insertions(+), 9 deletions(-)

diff --git a/simplewiki_edit.html b/simplewiki_edit.html
index 0fb28f8d6e..e0bffd5f31 100644
--- a/simplewiki_edit.html
+++ b/simplewiki_edit.html
@@ -11,16 +11,15 @@ ${ wiki_article.title }
 <%block name="wiki_head">
 <script>
   $(function() {
-    $("#id_contents").data('initial_contents') = $("#id_contents");
+    //Store the initial contents of #id_contents (the wiki body field) so we can compare for unsaved changes
+    $("#id_contents").data('initial_contents', $("#id_contents").val());
     
-    
-    window.
-    
-  }
-
-  //On load, we save
-  
-  var blah;
+    window.onbeforeunload = function askConfirm() { //Warn the user before they navigate away
+      if ( $("#id_contents").val() != $("#id_contents").data('initial_contents') ) {
+        return "You have made changes to the article that have not been saved yet.";
+      }
+    };
+  });
 </script>
 </%block>
 

From 67047cd1516df0f0aef9b3081a8c12881d23c5bd Mon Sep 17 00:00:00 2001
From: Bridger Maxwell <bridger@mit.edu>
Date: Fri, 17 Feb 2012 00:37:45 -0500
Subject: [PATCH 7/7] Some changes for code review

--HG--
branch : bridger-dev
---
 profile.html      | 6 +++---
 profile_graphs.js | 2 +-
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/profile.html b/profile.html
index 6e627005de..8925715dbf 100644
--- a/profile.html
+++ b/profile.html
@@ -5,9 +5,9 @@
 %>
 
 <%block name="headextra">
-<script type="text/javascript" src="${ settings.LIB_URL }flot/jquery.flot.js"></script>
-<script type="text/javascript" src="${ settings.LIB_URL }flot/jquery.flot.stack.js"></script>
-<script type="text/javascript" src="${ settings.LIB_URL }flot/jquery.flot.symbol.js"></script>
+<script type="text/javascript" src="/static/js/flot/jquery.flot.js"></script>
+<script type="text/javascript" src="/static/js/flot/jquery.flot.stack.js"></script>
+<script type="text/javascript" src="/static/js/flot/jquery.flot.symbol.js"></script>
 <script>
 <%include file="profile_graphs.js"/>
 </script>
diff --git a/profile_graphs.js b/profile_graphs.js
index a24bdfb7c4..073f3cda2e 100644
--- a/profile_graphs.js
+++ b/profile_graphs.js
@@ -105,7 +105,7 @@ $(function () {
     totalScore += section['totalscore']['score'] * section['weight']
     %>
   %endfor
-  ticks = ticks.concat( [ [${overviewBarX}, "Totals"] ] );
+  ticks = ticks.concat( [ [${overviewBarX}, "Total"] ] );
   
   <% tickIndex += 1 + sectionSpacer %>