views: Don't show chevron if user can't change sort order

We prevent users changing the column that sorting is done on when
allowing users to change the order of patches in a bundle. However, we
still show a chevron and clicking the link will appear to do something.
This is confusing/misleading. Remove the chevron and the link in this
situation.

Signed-off-by: Stephen Finucane <[email protected]>

Author: stephenfin

patchwork/templates/patchwork/partials/patch-list.html

@@ -69,6 +69,7 @@
 {% endif %}
 
         <th>
+{% if not order.editable %}
 {% if order.name == "name" %}
           <a class="colactive" href="{% listurl order=order.reversed_name %}">
             <span class="glyphicon glyphicon-chevron-{{ order.updown }}"></span>
@@ -77,11 +78,10 @@
            Patch
           </a>
 {% else %}
-{% if not order.editable %}
           <a class="colinactive" href="{% listurl order="name" %}">Patch</a>
+{% endif %}
 {% else %}
           <span class="colinactive">Patch</span>
-{% endif %}
 {% endif %}
         </th>
 
@@ -98,6 +98,7 @@
         </th>
 
         <th>
+{% if not order.editable %}
 {% if order.name == "date" %}
           <a class="colactive" href="{% listurl order=order.reversed_name %}">
             <span class="glyphicon glyphicon-chevron-{{ order.updown }}"></span>
@@ -106,15 +107,15 @@
             Date
           </a>
 {% else %}
-{% if not order.editable %}
           <a class="colinactive" href="{% listurl order="date" %}">Date</a>
+{% endif %}
 {% else %}
           <span class="colinactive">Date</span>
-{% endif %}
 {% endif %}
         </th>
 
         <th>
+{% if not order.editable %}
 {% if order.name == "submitter" %}
           <a class="colactive" href="{% listurl order=order.reversed_name %}">
             <span class="glyphicon glyphicon-chevron-{{ order.updown }}"></span>
@@ -123,17 +124,17 @@
             Submitter
           </a>
 {% else %}
-{% if not order.editable %}
           <a class="colinactive" href="{% listurl order="submitter" %}">
             Submitter
           </a>
+{% endif %}
 {% else %}
           <span class="colinactive">Submitter</span>
-{% endif %}
 {% endif %}
         </th>
 
         <th>
+{% if not order.editable %}
 {% if order.name == "delegate" %}
           <a class="colactive" href="{% listurl order=order.reversed_name %}">
             <span class="glyphicon glyphicon-chevron-{{ order.updown }}"></span>
@@ -142,15 +143,15 @@
             Delegate
           </a>
 {% else %}
-{% if not order.editable %}
           <a class="colinactive" href="{% listurl order="delegate" %}">Delegate</a>
+{% endif %}
 {% else %}
           <span class="colinactive">Delegate</span>
-{% endif %}
 {% endif %}
         </th>
 
         <th>
+{% if not order.editable %}
 {% if order.name == "state" %}
           <a class="colactive" href="{% listurl order=order.reversed_name %}">
             <span class="glyphicon glyphicon-chevron-{{ order.updown }}"></span>
@@ -159,11 +160,10 @@
             State
           </a>
 {% else %}
-{% if not order.editable %}
           <a class="colinactive" href="{% listurl order="state" %}">State</a>
+{% endif %}
 {% else %}
           <span class="colinactive">State</span>
-{% endif %}
 {% endif %}
         </th>