Opened 48 minutes ago

Last modified 37 minutes ago

#37378 assigned Bug

Extra empty breadcrumb in admindocs model_index.html

Reported by: Jacob Walls Owned by: Django Sprints
Component: contrib.admindocs Version: 6.1
Severity: Release blocker Keywords:
Cc: Antoliny Triage Stage: Unreviewed
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: yes

Description (last modified by Jacob Walls)

This HTML typo creates an extra <li> and very likely an extra breadcrumb. Visual regression in bc03f1064e10fa247a46d4e8a98ba9b26aa4790d.

  • django/contrib/admindocs/templates/admin_doc/model_index.html 

    diff --git a/django/contrib/admindocs/templates/admin_doc/model_index.html b/django/contrib/admindocs/templates/admin_doc/model_index.html
index 590cb34b95..82f0a9cb8a 100644
    a b  
    66{% block breadcrumbs %}
    77<ol class="breadcrumbs">
    88<li><a href="{% url 'admin:index' %}">{% translate 'Home' %}</a></li>
    9 <li><a href="{% url 'django-admindocs-docroot' %}">{% translate 'Documentation' %}</a><li>
     9<li><a href="{% url 'django-admindocs-docroot' %}">{% translate 'Documentation' %}</a></li>
    1010<li aria-current="page">{% translate 'Models' %}</li>
    1111</ol>
    1212{% endblock %}

Change History (2)

comment:1 by Jacob Walls, 45 minutes ago

Description: modified (diff)
Owner: Jacob Walls removed
Status: assigned → new

comment:2 by Jacob Walls, 37 minutes ago

Owner: set to Django Sprints
Status: new → assigned

While backportable, this isn't urgent. This could be a good ticket for a newcomer to practice writing Django tests, because I suppose we will want a regression test in TestModelIndexView.

Note: See TracTickets for help on using tickets.
Back to Top