#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 by , 7 years ago
comment:2 by , 7 years ago
| Resolution: | → needsinfo |
|---|---|
| Status: | new → closed |
comment:3 by , 7 years ago
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.
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.