[Django] #33930: Add id="deleted-objects" to template admin/delete_confirmation.html

35 views
Skip to first unread message

Django

unread,
Aug 17, 2022, 2:24:31 AM8/17/22
to django-...@googlegroups.com
#33930: Add id="deleted-objects" to template admin/delete_confirmation.html
-------------------------------------+-------------------------------------
Reporter: Jacob | Owner: Jacob Rief
Rief |
Type: New | Status: assigned
feature |
Component: | Version: dev
contrib.admin | Keywords:
Severity: Normal | admin/delete_confirmation.html
Triage Stage: | Has patch: 0
Unreviewed |
Needs documentation: 0 | Needs tests: 0
Patch needs improvement: 0 | Easy pickings: 1
UI/UX: 1 |
-------------------------------------+-------------------------------------
As discussed on the developers mailing list, see
https://groups.google.com/g/django-developers/c/VTRqW7ZmB6A ,
is would make sense to add id="deleted-objects" to the <ul> element
showing the objects to be deleted.

Reason:
Whenever a user attempts to delete an object inside the Django admin,
a delete confirmation page is shown. If the object to be deleted contains
many relations, the user is overwhelmed with a list of other objects to be
deleted together with the current object. Often those objects are just
internal relations the user never heard of, and this may be unsettling.

By overriding that template one can use CSS to hide this list and add a
short
JavaScript snippet to unhide if necessary.

--
Ticket URL: <https://code.djangoproject.com/ticket/33930>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.

Django

unread,
Aug 17, 2022, 2:29:04 AM8/17/22
to django-...@googlegroups.com
#33930: Add id="deleted-objects" to template admin/delete_confirmation.html
-------------------------------------+-------------------------------------
Reporter: Jacob Rief | Owner: Jacob
Type: | Rief
Cleanup/optimization | Status: assigned
Component: contrib.admin | Version: dev
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
admin/delete_confirmation.html |
Has patch: 0 | Needs documentation: 0

Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 1 | UI/UX: 1
-------------------------------------+-------------------------------------
Changes (by Mariusz Felisiak):

* type: New feature => Cleanup/optimization
* stage: Unreviewed => Accepted


--
Ticket URL: <https://code.djangoproject.com/ticket/33930#comment:1>

Django

unread,
Aug 17, 2022, 4:16:11 AM8/17/22
to django-...@googlegroups.com
#33930: Add id="deleted-objects" to template admin/delete_confirmation.html
-------------------------------------+-------------------------------------
Reporter: Jacob Rief | Owner: Jacob
Type: | Rief
Cleanup/optimization | Status: assigned
Component: contrib.admin | Version: dev
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
admin/delete_confirmation.html |
Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 1 | UI/UX: 1
-------------------------------------+-------------------------------------
Changes (by אורי):

* cc: אורי (added)


--
Ticket URL: <https://code.djangoproject.com/ticket/33930#comment:2>

Django

unread,
Aug 17, 2022, 7:04:15 PM8/17/22
to django-...@googlegroups.com
#33930: Add id="deleted-objects" to template admin/delete_confirmation.html
-------------------------------------+-------------------------------------
Reporter: Jacob Rief | Owner: Jacob
Type: | Rief
Cleanup/optimization | Status: assigned
Component: contrib.admin | Version: dev
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
admin/delete_confirmation.html |
Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 1 | UI/UX: 1
-------------------------------------+-------------------------------------

Comment (by Jacob Rief):

Pull request: https://github.com/django/django/pull/15967

--
Ticket URL: <https://code.djangoproject.com/ticket/33930#comment:3>

Django

unread,
Aug 18, 2022, 12:08:58 AM8/18/22
to django-...@googlegroups.com
#33930: Add id="deleted-objects" to template admin/delete_confirmation.html
-------------------------------------+-------------------------------------
Reporter: Jacob Rief | Owner: Jacob
Type: | Rief
Cleanup/optimization | Status: assigned
Component: contrib.admin | Version: dev
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
admin/delete_confirmation.html |
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 1

Easy pickings: 1 | UI/UX: 1
-------------------------------------+-------------------------------------
Changes (by Mariusz Felisiak):

* needs_better_patch: 0 => 1
* has_patch: 0 => 1


--
Ticket URL: <https://code.djangoproject.com/ticket/33930#comment:4>

Django

unread,
Aug 18, 2022, 4:02:17 AM8/18/22
to django-...@googlegroups.com
#33930: Add id="deleted-objects" to template admin/delete_confirmation.html
-------------------------------------+-------------------------------------
Reporter: Jacob Rief | Owner: Jacob
Type: | Rief
Cleanup/optimization | Status: assigned
Component: contrib.admin | Version: dev
Severity: Normal | Resolution:
Keywords: | Triage Stage: Ready for
admin/delete_confirmation.html | checkin

Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0

Easy pickings: 1 | UI/UX: 1
-------------------------------------+-------------------------------------
Changes (by Mariusz Felisiak):

* needs_better_patch: 1 => 0
* stage: Accepted => Ready for checkin


--
Ticket URL: <https://code.djangoproject.com/ticket/33930#comment:5>

Django

unread,
Aug 18, 2022, 5:33:33 AM8/18/22
to django-...@googlegroups.com
#33930: Add id="deleted-objects" to template admin/delete_confirmation.html
-------------------------------------+-------------------------------------
Reporter: Jacob Rief | Owner: Jacob
Type: | Rief
Cleanup/optimization | Status: closed
Component: contrib.admin | Version: dev
Severity: Normal | Resolution: fixed

Keywords: | Triage Stage: Ready for
admin/delete_confirmation.html | checkin
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 1 | UI/UX: 1
-------------------------------------+-------------------------------------
Changes (by Mariusz Felisiak <felisiak.mariusz@…>):

* status: assigned => closed
* resolution: => fixed


Comment:

In [changeset:"aed60aee38215e293d6ec2f3c96ec55bb9a62fc2" aed60ae]:
{{{
#!CommitTicketReference repository=""
revision="aed60aee38215e293d6ec2f3c96ec55bb9a62fc2"
Fixed #33930 -- Eased customization of delete_confirmation.html template
in the admin.
}}}

--
Ticket URL: <https://code.djangoproject.com/ticket/33930#comment:6>

Reply all
Reply to author
Forward
0 new messages