Merge pull request #368 from OpenTechFund/feature/219-show-hide-reviewers

functionality to show/hide reviewers

opentech/apply/funds/templates/funds/applicationsubmission_admin_detail.html

@@ -27,7 +27,6 @@
     </div>
 {% endblock %}
 
-
 {% block extra_js %}
     {{ reviewer_form.media }}
 {% endblock %}

opentech/apply/funds/templates/funds/includes/review_table.html

 {% load review_tags %}
-<table class="reviews-sidebar">
+<table class="reviews-sidebar js-reviews-sidebar">
     {% if staff_reviews or object.staff_not_reviewed %}
         {% if staff_reviews %}
             <tr class="tr tr--subchild light-grey-bg">
@@ -20,8 +20,14 @@
     {% if object.stage.has_external_review %}
         <tr class="tr tr--subchild"><td colspan="4"><hr></td></tr>
 
-        {% include 'funds/includes/review_table_row.html' with reviews=reviewer_reviews %}
-        {# Add logic for showing missing reviews #}
-        {% include 'funds/includes/review_table_row.html' with reviews=object.reviewers_not_reviewed missing=True %}
+        {% if reviewer_reviews or object.reviewers_not_reviewed  %}
+            {% include 'funds/includes/review_table_row.html' with reviews=reviewer_reviews %}
+            {% if object.reviewers_not_reviewed %}
+                {% include 'funds/includes/review_table_row.html' with reviews=object.reviewers_not_reviewed missing=True class="hidden" %}
+                <tr><td colspan="4"><a class="link link--bold link--underlined js-toggle-reviewers" href="#">All Assigned Advisors</a></td></tr>
+            {% endif %}
+        {% else %}
+            <tr><td colspan="4" class="reviews-sidebar__no-reviews">No assignees or reviews yet</td></tr>
+        {% endif %}
     {% endif %}
 </table>

opentech/apply/funds/templates/funds/includes/review_table_row.html

 {% for review in reviews %}
-    <tr class="tr--subchild {% if missing %}no-response{% endif %}">
+    <tr class="tr--subchild {{ class }} {% if missing %}no-response{% endif %}">
         {% if missing %}
             <td colspan="2">{{ review }}</td>
             <td>-</td>

opentech/static_src/src/javascript/components/toggle-reviewers.js

+import $ from './../globals';
+
+export default () => {
+    // get all the reviewers that are missing
+    const reviewers = Array.prototype.slice.call($('.js-reviews-sidebar').find('tr.hidden.no-response'));
+
+    $('.js-toggle-reviewers').click(function(e) {
+        e.preventDefault();
+
+        // toggle class and update text
+        $(this).toggleClass('is-open');
+        $(this).hasClass('is-open') ? $(this).html('Hide All Assigned Advisors') : $(this).html('All Assigned Advisors');
+
+        // toggle the reviewers
+        toggleReviewers(reviewers);
+    });
+
+    // show/hide the reviewers
+    function toggleReviewers(reviewers) {
+        reviewers.forEach(((reviewer) => {
+            $(reviewer).toggleClass('hidden');
+        }));
+    }
+};

opentech/static_src/src/javascript/main.js

@@ -13,6 +13,7 @@ import submissionFilters from './components/submission-filters';
 import mobileFilterPadding from './components/mobile-filter-padding';
 import generateTooltips from './components/submission-tooltips';
 import DeterminationCopy from './components/determination-template';
+import toggleReviewers from './components/toggle-reviewers';
 
 (function ($) {
     $(document).ready(function(){
@@ -62,6 +63,9 @@ import DeterminationCopy from './components/determination-template';
 
         // Submission filters logic
         submissionFilters();
+
+        // Toggle all reviewers in the sidebar
+        toggleReviewers();
     });
 
     // Add active class to select2 checkboxes after page has been filtered

opentech/static_src/src/sass/apply/base/_base.scss

@@ -74,6 +74,7 @@ ol {
     transition: opacity, z-index, $transition;
 }
 
+.hidden,
 .is-hidden,
 %is-hidden {
     display: none;

opentech/static_src/src/sass/apply/components/_link.scss

@@ -187,4 +187,9 @@
             }
         }
     }
+
+    &--toggle-reviewers {
+        display: block;
+        margin: 0 10px 30px;
+    }
 }