Opened 2 years ago

Closed 2 years ago

Last modified 2 years ago

#28372 closed New feature (needsinfo)

Admin inline JS should fire signal on add/remove to other JS can react.

Reported by: Curtis Maloney Owned by: nobody
Component: contrib.admin Version: 1.11
Severity: Normal Keywords:
Cc: Triage Stage: Unreviewed
Has patch: no Needs documentation: yes
Needs tests: yes Patch needs improvement: no
Easy pickings: yes UI/UX: no

Description

Whilst chasing a bug reported to django-array-tags, I realised I needed to init my tag widget every time a new row was added on an admin inline.

On perusal of the code, there is a callback each for added and removed, but they're pre-set.

It would be simpler and more extensible to have these functions emit an Event, and have widgets listen for these to do their own fixups.

This would also allow 3rd party widgets to partake.

Change History (3)

comment:1 Changed 2 years ago by Tim Graham

Are you saying that the existing inline form events are problematic? I'm not completely clear what the issue is and what adjustments are needed.

comment:2 Changed 2 years ago by Tim Graham

Resolution: needsinfo
Status: newclosed

comment:3 Changed 2 years ago by Curtis Maloney

Besides being jQuery custom events, no. I didn't see them in my initial perusal of the code.

Note: See TracTickets for help on using tickets.
Back to Top