Opened 19 months ago

Closed 19 months ago

Last modified 18 months 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 19 months 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 19 months ago by Tim Graham

Resolution: needsinfo
Status: newclosed

comment:3 Changed 18 months 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