Opened 12 years ago

Closed 12 years ago

#3726 closed enhancement (fixed)

Editor has no way to be programmatically destroyed.

Reported by: guest Owned by: Adam Peller
Priority: high Milestone: 1.0
Component: Editor Version: 0.9
Keywords: Cc:
Blocked By: Blocking:

Description

I'd like to be able to close/destroy an instance of Editor and have it's toolbar be destroyed as well.

Currently, I've added the following as a close function in dijit.Editor:

close: function(save, force){
    this.toolbar.destroy();
    dijit.Editor.superclass.close.call(this, save, force);
}

This may not be the best way, but it works for now.

Change History (8)

comment:1 Changed 12 years ago by guest

Preview killed my email address: ben.schell@…

comment:2 Changed 12 years ago by Jared Jurkiewicz

Note that Ben Shell is covered under the same Corporate CLA that I am...

comment:3 Changed 12 years ago by Jared Jurkiewicz

Ben Schell, that is. :)

comment:4 Changed 12 years ago by Adam Peller

Component: GeneralEditor
Owner: changed from anonymous to alex

does this code go in dijit.Editor.destroy()?

comment:5 Changed 12 years ago by bill

Milestone: 1.0

comment:6 Changed 12 years ago by bill

(yup the code should go in Editor.destroy()

comment:7 Changed 12 years ago by Adam Peller

Owner: changed from alex to Adam Peller

comment:8 Changed 12 years ago by Adam Peller

Resolution: fixed
Status: newclosed

(In [10810]) Destroy toolbar when destroying this. Not sure about tying together close and destroy, but you can call destroy directly. Fixes #3726.

Note: See TracTickets for help on using tickets.