Merged in [15358] from kivinen@iki.fi:

Fixed the tablesorter issue by adding display:none character in the
front of the date. Tablesorter looks first characters and picks the
parser based on that, meaning that if the text starts with number, it
will try to parse everything in column as numbers, and if there is
anything that is not numebr it will fail, and only sorts the column
once. This is bit ugly but I do not think we want to add yet another
jquery meta tool. Fixes #2089, and #2151.
 - Legacy-Id: 15403
Note: SVN reference [15358] has been migrated to Git commit 6301862b27
This commit is contained in:
Henrik Levkowetz 2018-07-23 22:53:10 +00:00
commit a948cd1c48

View file

@ -43,8 +43,8 @@
<tr>
<td>{% if r.pk != None %}<a href="{% url "ietf.doc.views_review.review_request" name=r.doc.name request_id=r.pk %}">{% endif %}{{ r.doc.name }}-{% if r.requested_rev %}{{ r.requested_rev }}{% else %}{{ r.doc.rev }}{% endif %}{% if r.pk != None %}</a>{% endif %}</td>
<td>{{ r.type.name }}</td>
<td>{% if r.pk %}{{ r.time|date:"Y-m-d" }} by {{r.requested_by.plain_name}}{% else %}<em>auto-suggested</em>{% endif %}</td>
<td>
<td><span style="display:none">X</span>{% if r.pk %}{{ r.time|date:"Y-m-d" }} by {{r.requested_by.plain_name}}{% else %}<em>auto-suggested</em>{% endif %}</td>
<td><span style="display:none">X</span>
{{ r.deadline|date:"Y-m-d" }}
{% if r.due %}<span class="label label-warning" title="{{ r.due }} day{{ r.due|pluralize }} past deadline">{{ r.due }} day{{ r.due|pluralize }}</span>{% endif %}
</td>
@ -101,7 +101,7 @@
<tr>
<td><a href="{% url "ietf.doc.views_review.review_request" name=r.doc.name request_id=r.pk %}">{{ r.doc.name }}{% if r.requested_rev %}-{{ r.requested_rev }}{% endif %}</a></td>
<td>{{ r.type }}</td>
<td>{{ r.time|date:"Y-m-d" }} by {{ r.requested_by.plain_name }}</td>
<td><span style="display:none">X</span>{{ r.time|date:"Y-m-d" }} by {{ r.requested_by.plain_name }}</td>
<td>{{ r.deadline|date:"Y-m-d" }}</td>
<td>{{ r.review_done_time|date:"Y-m-d" }}</td>
<td>