Update review pages

Show the same options for reviews on the dashboard and on review screens.
Add title to all pages within the review section.

pinaxcon/templates/dashboard.html

@@ -83,18 +83,23 @@
       </div>
       <div class="card-body">
         <ul class="list-unstyled">
-            <li><a href="{% url "review_section" section.section.slug %}">All</a></li>
+            <li><a href="{% url "review_section" section.section.slug %}">All Reviews</a></li>
+            {% comment %}
+            <li><a href="{% url "review_section_assignments" section.section.slug %}">Your Assignments</a></li>
+            {% endcomment %}
             <li><a href="{% url "user_reviewed" section.section.slug %}">Reviewed by you</a></li>
             <li><a href="{% url "user_not_reviewed" section.section.slug %}">Not Reviewed by you</a></li>
+            <li><a href="{% url "user_random" section.section.slug %}">Random unreviewed proposal</a></li>
           </ul>
-          {% if section in manage_sections %}
+
           <ul class="list-unstyled">
-            <li><a href="{% url "review_bulk_update" section.section.slug %}">Bulk Update</a></li>
-            <li><a href="{% url "result_notification" section.section.slug "accepted" %}">Send notifications</a></li>
             <li><a href="{% url "review_status" section.section.slug %}">Voting Status</a></li>
+            {% if section in manage_sections %}
+            <li><a href="{% url "review_bulk_update" section.section.slug %}">Bulk Update</a></li>
+            <li><a href="{% url "result_notification" section.section.slug "accepted" %}">Result notifications</a></li>
             <li><a href="{% url "review_admin" section.section.slug %}">Reviewer Stats</a></li>
-          </ul>
             {% endif %}
+          </ul>
       </div>
     </div>
     </div>

pinaxcon/templates/symposion/reviews/base.html

@@ -22,17 +22,15 @@
                         {% trans "All Reviews" %}
                     </a>
                     {% comment %}
-                    <li>
+                    <a class="list-group-item" href="{% url "review_section_assignments" section.section.slug %}">
-                        <a href="{% url "review_section_assignments" section.section.slug %}">
                         {% trans "Your Assignments" %}
                     </a>
-                    </li>
                     {% endcomment %}
                     <a class="list-group-item user-reviewed" href="{% url "user_reviewed" section.section.slug %}">
                         {% trans "Reviewed by you" %}
                     </a>
                     <a class="list-group-item user-not-reviewed" href="{% url "user_not_reviewed" section.section.slug %}">
-                        {% trans "Unreviewed by you" %}
+                        {% trans "Not Reviewed by you" %}
                     </a>
                     <a class="list-group-item user-random" href="{% url "user_random" section.section.slug %}">
                         {% trans "Random unreviewed proposal" %}
@@ -40,8 +38,16 @@
                     <a class="list-group-item voting-status" href="{% url "review_status" section.section.slug %}">
                         {% trans "Voting Status" %}
                     </a>
-                    {% if request.user.is_staff %}
+                    {% if section in manage_sections %}
-                    <a class="list-group-item review-results" href="{% url "result_notification" section.section.slug 'accepted' %}">Result Notification</a>
+                    <a class="list-group-item" href="{% url "review_bulk_update" section.section.slug %}">
+                        Bulk Update
+                    </a>
+                    <a class="list-group-item review-results" href="{% url "result_notification" section.section.slug 'accepted' %}">
+                        Result Notifications
+                    </a>
+                    <a class="list-group-item" href="{% url "review_admin" section.section.slug %}">
+                        Reviewer Stats
+                    </a>
                     {% endif %}
                 </div>
             </div>

pinaxcon/templates/symposion/reviews/result_notification.html

@@ -2,7 +2,7 @@
 
 {% load i18n %}
 
-{% block head_title %}{% trans 'Result Notification' %}{% endblock %}
+{% block head_title %}Reviews - {% trans 'Result Notification' %} ({{ section_slug }}){% endblock %}
 
 {% block body_class %}{{ block.super }} review-results{% endblock %}
 
@@ -17,7 +17,7 @@
 
 {% block body %}
 
-    <h1>Result Notification</h1>
+    <h1>Result Notification ({{ section_slug }})</h1>
 
     <ul class="nav nav-tabs">
         <li class="nav-item"><a class="nav-link {% if status == 'accepted' %}active{% endif %}" href="{% url "result_notification" section_slug 'accepted' %}">Accepted</a>

pinaxcon/templates/symposion/reviews/review_admin.html

@@ -1,7 +1,9 @@
 {% extends "symposion/reviews/base.html" %}
 
+{% block head_title %}Reviews - Reviewer Stats ({{ section_slug }}){% endblock %}
+
 {% block body %}
-    <h1>Reviewers – {{ section_slug }}</h1>
+    <h1>Reviewer Stats ({{ section_slug }})</h1>
 
     <table class="table table-striped">
         <tr>

pinaxcon/templates/symposion/reviews/review_bulk_update.html

@@ -2,9 +2,11 @@
 
 {% load bootstrap %}
 
+{% block head_title %}Reviews - Bulk Accept ({{ section_slug }}){% endblock %}
+
 {% block body %}
 
-    <h1>Bulk Accept</h1>
+    <h1>Bulk Accept ({{ section_slug }})</h1>
     <form class="form-horizontal" action="" method="POST" class="uniForm" accept-charset="utf-8">
         {% csrf_token %}
         {{ form|bootstrap }}

pinaxcon/templates/symposion/reviews/review_detail.html

@@ -3,6 +3,7 @@
 {% load bootstrap %}
 {% load i18n %}
 
+{% block head_title %}Review #{{ proposal.number }}: {{ proposal.title }} ({{ proposal.speaker }}){% endblock %}
 
 {% block body %}
 

pinaxcon/templates/symposion/reviews/review_stats.html

@@ -1,5 +1,7 @@
 {% extends "symposion/reviews/base.html" %}
 
+{% block head_title %}Reviews - Voting Status ({{ section_slug }}){% endblock %}
+
 {% block body_class %}{{ block.super }} voting-status {{ key }}{% endblock %}
 
 {% block body %}

vendor/symposion/reviews/views.py

@@ -569,6 +569,7 @@ def review_bulk_update(request, section_slug):
 
     return render(request, "symposion/reviews/review_bulk_update.html", {
         "form": form,
+        "section_slug": section_slug,
     })