Opened 4 weeks ago
Closed 11 days ago
#36032 closed New feature (fixed)
Apply a link to the URLField value on the admin app index page.
Reported by: | Antoliny | Owned by: | Antoliny |
---|---|---|---|
Component: | contrib.admin | Version: | dev |
Severity: | Normal | Keywords: | URLField, list_display |
Cc: | Triage Stage: | Ready for checkin | |
Has patch: | yes | Needs documentation: | no |
Needs tests: | no | Patch needs improvement: | no |
Easy pickings: | no | UI/UX: | no |
Description (last modified by )
When adding a URLField through ModelAdmin list_display
in the app index page's , the value is displayed as plain text instead of a link, which was not expected.
It would be better if the URLField value is displayed as a link when shown through list_display
in the ModelAdmin.
On the change page, the URLField value is displayed as a link.
Attachments (2)
Change History (13)
by , 4 weeks ago
Attachment: | Screenshot 2024-12-21 at 9.14.53 PM.png added |
---|
comment:1 by , 4 weeks ago
Owner: | set to |
---|---|
Status: | new → assigned |
comment:2 by , 4 weeks ago
Description: | modified (diff) |
---|
comment:3 by , 4 weeks ago
Description: | modified (diff) |
---|
by , 4 weeks ago
Attachment: | Screenshot 2024-12-21 at 9.21.12 PM.png added |
---|
comment:4 by , 4 weeks ago
comment:5 by , 4 weeks ago
Has patch: | set |
---|
comment:6 by , 4 weeks ago
Needs documentation: | set |
---|---|
Triage Stage: | Unreviewed → Accepted |
Version: | 5.1 → dev |
Thank you Antoliny for the ticket report. I think your proposal makes sense, I can't think of a case where this would not be desirable.
Accepting on that basis, could you please add a small note in the 5.2 release notes under `django.contrib.admin'? Once that's done, please unset the "patch needs documentation" in this ticket to get a full review.
Well done!
comment:7 by , 4 weeks ago
Needs documentation: | unset |
---|
comment:8 by , 3 weeks ago
Patch needs improvement: | set |
---|
comment:9 by , 12 days ago
Patch needs improvement: | unset |
---|
comment:10 by , 11 days ago
Triage Stage: | Accepted → Ready for checkin |
---|
PR