Re: [Django] #28372: Admin inline JS should fire signal on add/remove to other JS can react.

[Django]

#28372: Admin inline JS should fire signal on add/remove to other JS can react.
+--
 Reporter:  Curtis Maloney  |Owner:  nobody
 Type:  New feature |   Status:  closed
Component:  contrib.admin   |  Version:  1.11
 Severity:  Normal  |   Resolution:  needsinfo
 Keywords:  | Triage Stage:  Unreviewed
Has patch:  0   |  Needs documentation:  1
  Needs tests:  1   |  Patch needs improvement:  0
Easy pickings:  1   |UI/UX:  0
+--

Comment (by Curtis Maloney):

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

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/066.33c0c01e92dfd67adb56541c9aa31144%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #28372: Admin inline JS should fire signal on add/remove to other JS can react.

[Django]

#28372: Admin inline JS should fire signal on add/remove to other JS can react.
+--
 Reporter:  Curtis Maloney  |Owner:  nobody
 Type:  New feature |   Status:  closed
Component:  contrib.admin   |  Version:  1.11
 Severity:  Normal  |   Resolution:  needsinfo
 Keywords:  | Triage Stage:  Unreviewed
Has patch:  0   |  Needs documentation:  1
  Needs tests:  1   |  Patch needs improvement:  0
Easy pickings:  1   |UI/UX:  0
+--
Changes (by Tim Graham):

 * status:  new => closed
 * resolution:   => needsinfo


-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/066.e8322ab950871689ffae9c672ed800cb%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #28372: Admin inline JS should fire signal on add/remove to other JS can react.

[Django]

#28372: Admin inline JS should fire signal on add/remove to other JS can react.
+--
 Reporter:  Curtis Maloney  |Owner:  nobody
 Type:  New feature |   Status:  new
Component:  contrib.admin   |  Version:  1.11
 Severity:  Normal  |   Resolution:
 Keywords:  | Triage Stage:  Unreviewed
Has patch:  0   |  Needs documentation:  1
  Needs tests:  1   |  Patch needs improvement:  0
Easy pickings:  1   |UI/UX:  0
+--

Comment (by Tim Graham):

 Are you saying that the existing
 [https://docs.djangoproject.com/en/dev/ref/contrib/admin/javascript
 /#inline-form-events inline form events] are problematic? I'm not
 completely clear what the issue is and what adjustments are needed.

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/066.95a76e0b9ef5b6c3a637218533af39b1%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

[Django] #28372: Admin inline JS should fire signal on add/remove to other JS can react.

[Django]

#28372: Admin inline JS should fire signal on add/remove to other JS can react.
--+
   Reporter:  Curtis Maloney  |  Owner:  nobody
   Type:  New feature | Status:  new
  Component:  contrib.admin   |Version:  1.11
   Severity:  Normal  |   Keywords:
   Triage Stage:  Unreviewed  |  Has patch:  0
Needs documentation:  1   |Needs tests:  1
Patch needs improvement:  0   |  Easy pickings:  1
  UI/UX:  0   |
--+
 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.

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/051.69584db2c1244d1586118cf9e4719eff%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.