Use chevron control for row linking, rather than hyperlinking first item

Author: tomchristie

rest_framework/renderers.py

@@ -730,12 +730,10 @@ def get_context(self, data, accepted_media_type, renderer_context):
 
         columns = [key for key in header.keys() if key != 'url']
         details = [key for key in header.keys() if key != 'url']
-        linked = [columns[0]] if columns else []
 
         context['style'] = style
         context['columns'] = columns
         context['details'] = details
-        context['linked'] = linked
         context['results'] = results
         return context
 

rest_framework/templates/rest_framework/admin/list.html

@@ -1,20 +1,20 @@
 {% load rest_framework %}
 <table class="table table-striped">
     <thead>
-        <tr>{% for column in columns%}<th>{{ column|capfirst }}</th>{% endfor %}</tr>
+        <tr>{% for column in columns%}<th>{{ column|capfirst }}</th>{% endfor %}<th></th></tr>
     </thead>
     <tbody>
         {% for row in results %}
         <tr>
             {% for key, value in row.items %}
             {% if key in columns %}
             <td {{ value|add_nested_class }} >
-                {% if key in linked %}<a href="{{ row.url }}">{% endif %}
                 {{ value|format_value }}
-                {% if key in linked %}</a>{% endif %}
             </td>
             {% endif %}
             {% endfor %}
+            <td><a href="{{ row.url }}"><span class="glyphicon glyphicon-chevron-right" aria-hidden="true"></span>
+</a></td>
         </tr>
         {% endfor %}
     </tbody>

rest_framework/templatetags/rest_framework.py

@@ -134,7 +134,9 @@ def format_value(value):
 
 @register.filter
 def add_nested_class(value):
-    if isinstance(value, (list, dict)):
+    if isinstance(value, dict):
+        return 'class=nested'
+    if isinstance(value, list) and any([isinstance(item, (list, dict)) for item in value]):
         return 'class=nested'
     return ''