work out a default pit to compare to instead of always using the second-latest

Author: goose-life

indigo_app/static/javascript/indigo/views/document_editor.js

@@ -65,10 +65,10 @@
       this.editorReady = $.Deferred();
       this.setupRenderers();
 
-      // fake a click on the most recent point in time for comparison
-      let mostRecent = document.querySelector('.show-pit-comparison.most-recent');
-      if (mostRecent) {
-        mostRecent.click();
+      // fake a click on the previous point in time for comparison, if there is one
+      let defaultCompareTo = document.querySelector('.show-pit-comparison.default-compare-to');
+      if (defaultCompareTo) {
+        defaultCompareTo.click();
       }
     },
 

indigo_app/templates/indigo_api/document/_toolbar.html

@@ -11,7 +11,7 @@
         <div class="dropdown-menu">
           {% for exp in comparison_expressions %}
             <button
-                class="dropdown-item show-pit-comparison {% if forloop.counter0 == 1 %}most-recent{% endif %}"
+                class="dropdown-item show-pit-comparison {% if exp.id == default_comparison_id %}default-compare-to{% endif %}"
                 data-id="{{ exp.id }}"
             >
               {{ exp.expression_date|date:"Y-m-d" }}

indigo_app/views/documents.py

@@ -49,6 +49,8 @@ def get_context_data(self, **kwargs):
                 doc.work.expressions().all()
             ))
         context['comparison_expressions'] = doc.work.expressions().filter(language=doc.language).order_by('-expression_date')
+        previous_documents = context['comparison_expressions'].exclude(expression_date__gte=doc.expression_date)
+        context['default_comparison_id'] = previous_documents.first().pk if previous_documents else None
         context['place'] = doc.work.place
         context['country'] = doc.work.country
         context['locality'] = doc.work.locality