Opened 13 years ago

Closed 13 years ago

#16618 closed Cleanup/optimization (fixed)

Incorrect breadcrumbs shown on Admin image for Choices model in tutorial part 2

Reported by: m0nonoke Owned by: nobody
Component: Documentation Version: dev
Severity: Normal Keywords: admin tutorial
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: yes UI/UX: no

Description

The image shown in the section https://docs.djangoproject.com/en/dev/intro/tutorial02/#adding-related-objects does not show the correct breadcrumbs for the choices admin model. Given that Choices is part of the polls app, the breadcrumb should read Home > Polls > Choices > Add choice

Attachments (2)

admin10.png (11.7 KB ) - added by m0nonoke 13 years ago.
Current img used for ChoicesAdmin
admin_corrected.png (13.7 KB ) - added by m0nonoke 13 years ago.
Corrected img for ChoicesAdmin

Download all attachments as: .zip

Change History (10)

by m0nonoke, 13 years ago

Attachment: admin10.png added

Current img used for ChoicesAdmin

by m0nonoke, 13 years ago

Attachment: admin_corrected.png added

Corrected img for ChoicesAdmin

comment:1 by m0nonoke, 13 years ago

comment:2 by m0nonoke, 13 years ago

Summary: Incorrect breadcrumbs shown on Admin imgage for Choices model in tutorial part 2Incorrect breadcrumbs shown on Admin image for Choices model in tutorial part 2

comment:3 by Julien Phalip, 13 years ago

Patch needs improvement: set
Triage Stage: UnreviewedAccepted

Yes the breadcrumbs are incorrect. However, note that the same problem exists in *several* screenshots for that tutorial.

comment:4 by Aymeric Augustin, 13 years ago

The new image attached to this ticket has JPEG compression artifacts, which is weird for a PNG.

comment:5 by Karen Tracey, 13 years ago

We should not fix just one image. It looks like they are all from before r8474, which added the app index and thus the app name in the breadcrumbs. They should all be fixed together; fixing one would make things weirdly inconsistent. At least as it is now we can say they are from a previous version of the admin where the app index, and thus app names in the breadcrumbs, did not exist. I also thought there was already a ticket open for this, but cannot find it right now....so perhaps it has just been mentioned on the mailing list and no one bothered to create a ticket before.

comment:6 by Christopher Medrela, 13 years ago

Has patch: unset
Patch needs improvement: unset

comment:7 by Claude Paroz, 13 years ago

Has patch: set

I updated the images from admin04.png to admin14t.png. Here they are: http://www.2xlibre.net/files/_images.tar.gz

comment:8 by Julien Phalip, 13 years ago

Resolution: fixed
Status: newclosed

In [17543]:

Fixed #16618 -- Corrected the admin interface's breadcrumbs in the tutorials' screenshots. Many thanks to Claude Paroz.

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