Opened 11 years ago

Closed 11 years ago

#9273 closed (invalid)

Breadcrumbs refactor in newforms-admin

Reported by: erny Owned by: nobody
Component: contrib.admin Version: 1.0
Severity: Keywords:
Cc: Triage Stage: Unreviewed
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no


The code to create breadcrumbs is duplicated in all templates.

In index.html and app_index.html, is used (with blocktrans which tries to translate "%(name)s", nonsense) , and application names can not be translated.

In change_form.html, change_list.html, delete_confirmation.html and object_list.html {{ app_label|capfirst|escape }} is used.

This is not DRY. Perhaps, breadcrumbs could be calculated in view functions and send to the templates within the context. There should be a way for the user to change the generated breadcrumbs.

Change History (1)

comment:1 Changed 11 years ago by Jacob

Resolution: invalid
Status: newclosed

There isn't a specific action plan here, nor really a bug.

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