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)
Change History (10)
by , 13 years ago
Attachment: | admin10.png added |
---|
comment:1 by , 13 years ago
comment:2 by , 13 years ago
Summary: | Incorrect breadcrumbs shown on Admin imgage for Choices model in tutorial part 2 → Incorrect breadcrumbs shown on Admin image for Choices model in tutorial part 2 |
---|
comment:3 by , 13 years ago
Patch needs improvement: | set |
---|---|
Triage Stage: | Unreviewed → Accepted |
Yes the breadcrumbs are incorrect. However, note that the same problem exists in *several* screenshots for that tutorial.
comment:4 by , 13 years ago
The new image attached to this ticket has JPEG compression artifacts, which is weird for a PNG.
comment:5 by , 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 , 13 years ago
Has patch: | unset |
---|---|
Patch needs improvement: | unset |
comment:7 by , 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
Current img used for ChoicesAdmin