#27085 closed Cleanup/optimization (fixed)
UI Discrepancy in "Writing your first Django app, part 2"
| Reported by: | pluralschool | Owned by: | Abhishek Bera |
|---|---|---|---|
| Component: | Documentation | Version: | 1.10 |
| Severity: | Normal | Keywords: | |
| Cc: | Triage Stage: | Accepted | |
| Has patch: | yes | Needs documentation: | no |
| Needs tests: | no | Patch needs improvement: | yes |
| Easy pickings: | yes | UI/UX: | no |
Description (last modified by )
Hi
Here's reporting UI Discrepancy in "Writing your first Django app, part 2" tutorial. Please refer to the screenshots (Tutorial.JPG and Actual.JPG).
As per https://docs.djangoproject.com/en/1.10/intro/tutorial02/, "Home › Polls › Questions" is shown displaying Check Box: "QUESTION TEXT"(vide Tutorial.JPG). However, the actual out from the code displays Check Box: "QUESTION" (vide Actual.JPG). The code was run on the following environments:
CentOS 7.1 64-Bit
Django 1.9.6
Python 2.7.5
Windows 8.1 64-Bit
Django 1.10
Python 2.7.11
Windows 7 64-Bit
Django 1.8.2
Python 3.4.3
Thanks
Attachments (2)
Change History (19)
by , 9 years ago
| Attachment: | Tutorial.JPG added |
|---|
by , 9 years ago
| Attachment: | Actual.JPG added |
|---|
comment:1 by , 9 years ago
| Description: | modified (diff) |
|---|
comment:2 by , 9 years ago
| Triage Stage: | Unreviewed → Accepted |
|---|
comment:3 by , 9 years ago
| Easy pickings: | set |
|---|
Regenerating the image will be tricky because we want to match the style of the others.
However, in this instance we should be able to use an image-editing software to crop out the word "TEXT" and replace it with a solid gray backround.
The image to edit is located in docs/intro/_images/admin04t.png.
comment:5 by , 9 years ago
| Owner: | changed from to |
|---|---|
| Status: | new → assigned |
comment:6 by , 9 years ago
New image added.
Bug fixed on branch berabhishek:ticket_27085
This is my first patch ... Can anybody review it and suggest what to do next?
comment:7 by , 9 years ago
| Resolution: | → fixed |
|---|---|
| Status: | assigned → closed |
follow-up: 10 comment:8 by , 9 years ago
| Resolution: | fixed |
|---|---|
| Status: | closed → new |
Please don't set the resolution to fixed before the patch is committed to Django master.
Also could you please make a pull request, then set the Has patch flag?
follow-up: 12 comment:9 by , 9 years ago
As Claude said, please create a pull request from your branch, it will make it easier for us to leave comments on it (there should be a big green button on https://github.com/django/django/compare/master...berabhishek:ticket_27085).
Regarding the image, I see two issues:
1) The size is different from the original. Please crop/resize it so that your image is the same size as the old one (930x279).
2) The "Action" select menu looks different from the old one and is now inconsistent with the other screenshots. This is the problem I described in my comment earlier.
Thanks.
comment:10 by , 9 years ago
Replying to claudep:
Please don't set the resolution to fixed before the patch is committed to Django master.
Also could you please make a pull request, then set the
Has patchflag?
Thank you very much... point taken...
comment:11 by , 9 years ago
| Status: | new → assigned |
|---|
comment:12 by , 9 years ago
Replying to bmispelon:
As Claude said, please create a pull request from your branch, it will make it easier for us to leave comments on it (there should be a big green button on https://github.com/django/django/compare/master...berabhishek:ticket_27085).
Regarding the image, I see two issues:
1) The size is different from the original. Please crop/resize it so that your image is the same size as the old one (930x279).
2) The "Action" select menu looks different from the old one and is now inconsistent with the other screenshots. This is the problem I described in my comment earlier.
Thanks.
Thank You very much for pointing them out...
comment:13 by , 9 years ago
| Has patch: | set |
|---|
comment:14 by , 9 years ago
The discrepancy has been resolved in the branch berabhishek:ticket_27085
comment:15 by , 9 years ago
| Patch needs improvement: | set |
|---|
Hi,
Thanks for opening the pull request and fixing the size issue.
However, you haven't addressed my issue number 2: the form elements in your new screenshot are now inconsistent with the other screenshots on the page (checbox, select).
Hi,
Indeed, the screenshot in the tutorial is incorrect.
Thanks for spotting that.