Opened 4 months ago

Last modified 36 hours ago

#35650 assigned New feature

Multi select checkboxes with shift key is not working in TabularInlineAdmin and StackedInlineAdmin

Reported by: tanaydin sirin Owned by: tanaydin sirin
Component: contrib.admin Version: dev
Severity: Normal Keywords:
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: yes

Description (last modified by tanaydin sirin)

On any object list, we can select multiple objects by pressing the shift key while clicking the second checkbox, but it's not possible with TabularInline and StackedInlineAdmin ModelAdmin.

Change History (16)

comment:1 by tanaydin sirin, 4 months ago

Has patch: set
Last edited 4 months ago by tanaydin sirin (previous) (diff)

comment:2 by tanaydin sirin, 4 months ago

Version: 5.05.1
Last edited 4 months ago by tanaydin sirin (previous) (diff)

comment:3 by Natalia Bidart, 4 months ago

Patch needs improvement: set
Triage Stage: UnreviewedAccepted
Version: 5.1dev

Hello tanaydin sirin, thank you for your interest in making Django better. I have reproduced the described issue, and I agree it would be good to allow for shift-selecting in tabular admins just like in the changelist page.

I took a quite look at your PR, and I was thinking that instead of providing new javascript, we should try to re-use/generalize what the changelist page is already using. Could you look into that?

comment:4 by Sarah Boyce, 4 months ago

Owner: set to tanaydin sirin
Status: newassigned

in reply to:  3 comment:5 by tanaydin sirin, 3 months ago

Hello Natalia Bidart, I've made the changes. Shall I do anything else about ticket?

Replying to Natalia Bidart:

Hello tanaydin sirin, thank you for your interest in making Django better. I have reproduced the described issue, and I agree it would be good to allow for shift-selecting in tabular admins just like in the changelist page.

I took a quite look at your PR, and I was thinking that instead of providing new javascript, we should try to re-use/generalize what the changelist page is already using. Could you look into that?

comment:6 by tanaydin sirin, 3 months ago

Description: modified (diff)
Summary: Multi select checkboxes with shift key is not working in TabularInlineAdminMulti select checkboxes with shift key is not working in TabularInlineAdmin and StackedInlineAdmin

comment:7 by Natalia Bidart, 3 months ago

Thank you tanaydin sirin! The expected ticket update process is described in these docs. Basically you need to uncheck the "patch needs improvement" flag, and then this ticket will show up in the "needs review queue" in the Django Development Dashboard.

comment:8 by tanaydin sirin, 3 months ago

Patch needs improvement: unset

comment:9 by Sarah Boyce, 3 months ago

Needs tests: set
Patch needs improvement: set

comment:10 by tanaydin sirin, 3 days ago

Needs tests: unset
Patch needs improvement: unset
UI/UX: set

comment:11 by tanaydin sirin, 45 hours ago

Description: modified (diff)
Summary: Multi select checkboxes with shift key is not working in TabularInlineAdmin and StackedInlineAdminMulti select checkboxes with shift key is not working in TabularInlineAdmin

comment:12 by tanaydin sirin, 45 hours ago

Triage Stage: AcceptedReady for checkin

comment:13 by Sarah Boyce, 45 hours ago

Triage Stage: Ready for checkinAccepted

You cannot mark your own contributions as "Ready for checkin".
This should be done by a community member after a review and approval. See https://docs.djangoproject.com/en/5.1/internals/contributing/triaging-tickets/#ready-for-checkin

comment:14 by tanaydin sirin, 38 hours ago

Description: modified (diff)
Summary: Multi select checkboxes with shift key is not working in TabularInlineAdminMulti select checkboxes with shift key is not working in TabularInlineAdmin and StackedInlineAdmin

comment:15 by tanaydin sirin, 38 hours ago

Patch needs improvement: set

comment:16 by tanaydin sirin, 36 hours ago

Patch needs improvement: unset
Note: See TracTickets for help on using tickets.
Back to Top