Code

Opened 3 years ago

Closed 2 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: master
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 3 years ago.
Current img used for ChoicesAdmin
admin_corrected.png (13.7 KB) - added by m0nonoke 3 years ago.
Corrected img for ChoicesAdmin

Download all attachments as: .zip

Change History (10)

Changed 3 years ago by m0nonoke

Current img used for ChoicesAdmin

Changed 3 years ago by m0nonoke

Corrected img for ChoicesAdmin

comment:1 Changed 3 years ago by m0nonoke

  • Needs documentation unset
  • Needs tests unset
  • Patch needs improvement unset

comment:2 Changed 3 years ago by m0nonoke

  • Summary changed from Incorrect breadcrumbs shown on Admin imgage for Choices model in tutorial part 2 to Incorrect breadcrumbs shown on Admin image for Choices model in tutorial part 2

comment:3 Changed 3 years ago by julien

  • Patch needs improvement set
  • Triage Stage changed from Unreviewed to Accepted

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

comment:4 Changed 3 years ago by aaugustin

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

comment:5 Changed 3 years ago by kmtracey

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 Changed 2 years ago by krzysiumed

  • Has patch unset
  • Patch needs improvement unset

comment:7 Changed 2 years ago by claudep

  • 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 Changed 2 years ago by julien

  • Resolution set to fixed
  • Status changed from new to closed

In [17543]:

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

Add Comment

Modify Ticket

Change Properties
<Author field>
Action
as closed
as The resolution will be set. Next status will be 'closed'
The resolution will be deleted. Next status will be 'new'
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.