Opened 12 years ago
Closed 12 years ago
#8106 closed defect (fixed)
TextColor Editor Plugin Does Not Destroy Constituents Properly
Reported by: | ptwobrussell | Owned by: | bill |
---|---|---|---|
Priority: | high | Milestone: | 1.3 |
Component: | Editor | Version: | 1.2.1 |
Keywords: | Cc: | [email protected]… | |
Blocked By: | Blocking: |
Description
The TextColor? plugin creates a ColorPalette? as this.dropDown, but never disposes of it. Seems that an uninitialize method should be in place that takes care of that. Something like this patch:
uninitialize: function() { this.inherited(arguments); this.dropDown.destroyRecursive(); }
Change History (3)
comment:1 Changed 12 years ago by
Owner: | changed from liucougar to bill |
---|---|
Status: | new → assigned |
comment:2 Changed 12 years ago by
Milestone: | tbd → 1.3 |
---|
comment:3 Changed 12 years ago by
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
Note: See
TracTickets for help on using
tickets.
(In [15988]) Fix leak where drop down wasn't being destroyed. Fixes #8106, #8107 !strict.