Merge pull request #9 from mattip/deemphasize-old

comparison page: use the DEF_ENVIRONMENT when loading the page, uncheck old exes

Author: mattip

codespeed/static/js/comparison.js

@@ -394,10 +394,12 @@ function init(defaults) {
     $("#baseline").val(defaults.baseline);
     $("#direction").prop('checked', defaults.direction === "True");
 
+    /*
     var sel = $("input[name='executables']");
     $.each(defaults.executables, function(i, exe) {
         sel.filter("[value='" + exe + "']").prop('checked', true);
     });
+    */
 
     sel = $("input[name='benchmarks']");
     $.each(defaults.benchmarks, function(i, bench) {

codespeed/templates/codespeed/comparison.html

@@ -32,11 +32,19 @@
   <div class="boxbody">
     {% for proj, executable in executables.items %}
     <ul><a href="#" class="togglefold">{{ proj }}</a> <a href="#" class="checkall">(All</a>, <a href="#" class="uncheckall">None)</a>
+    {% if forloop.counter > 3 %}
     {% for exe in executable %}
       <li title="{{ exe.executable }} - {{ exe.revision }}">
         <input id="exe_{{ exe.key }}" type="checkbox" name="executables" value="{{ exe.key }}" />
         <label for="exe_{{ exe.key }}">{{ exe.name }}</label><div class="seriescolor"></div>
       </li>{% endfor %}
+    {% else %}
+    {% for exe in executable %}
+      <li title="{{ exe.executable }} - {{ exe.revision }}">
+        <input id="exe_{{ exe.key }}" type="checkbox" name="executables" value="{{ exe.key }}" checked />
+        <label for="exe_{{ exe.key }}">{{ exe.name }}</label><div class="seriescolor"></div>
+      </li>{% endfor %}
+    {% endif %}
     </ul>{% endfor %}
   </div>
 </div>
@@ -92,6 +100,10 @@
 <script type="text/javascript" src="{% static 'js/comparison.js' %}"></script>
 
 <script type="text/javascript">
+  var checkboxes = document.getElementsByTagName('environments');
+  for (c in checkboxes) {
+    c.checked = false;
+  }
   $(function() {
     Comparison.init({
       chart_type: "{{ selectedchart }}",

codespeed/views.py

@@ -199,7 +199,7 @@ def comparison(request):
     enviros = Environment.objects.all()
     if not enviros:
         return no_environment_error(request)
-    checkedenviros = get_default_environment(enviros, data, multi=True)
+    checkedenviros = get_default_environment(enviros, data)
 
     if not len(Project.objects.filter(track=True)):
         return no_default_project_error(request)

codespeed/views_data.py

@@ -46,7 +46,7 @@ def get_default_environment(enviros, data, multi=False):
     if multi:
         return defaultenviros
     else:
-        return defaultenviros[0]
+        return [defaultenviros[0],]
 
 
 def getbaselineexecutables():