Skip to main content

Search and Top Navigation

#5730 closed enhancement (wontfix)

Opened June 15, 2010 07:47PM UTC

Closed June 15, 2010 08:07PM UTC

Last modified October 11, 2012 09:15PM UTC

Dialog destroy option to destroy self.element

Reported by: cmathis430 Owned by:
Priority: minor Milestone:
Component: ui.dialog Version: 1.8.2
Keywords: destroy Cc:
Blocked by: Blocking:
Description

For dialogs created with anonymous content, it would be nice to have an option to destroy the dialog along with the content.

Anonymous content dialog:

$('<div>Anonymous content</div>').dialog({
	close: function() {
		$(this).dialog('destroy');
	}
});

I want to add a parameter to specify whether to remove the original content as well when calling destroy:

$(this).dialog('destroy', true);

Below is a change I made to allow this:

In jquery.ui.dialog.js line 164 I made the following change:

destroy: function(destroySelf) {
	var self = this;
	
	(self.overlay && self.overlay.destroy());
	self.uiDialog.hide();
	self.element
		.unbind('.dialog')
		.removeData('dialog')
		.removeClass('ui-dialog-content ui-widget-content');
	if (!destroySelf) {
		self.element.hide().appendTo('body');
	}
	self.uiDialog.remove();

	(self.originalTitle && self.element.attr('title', self.originalTitle));

	return self;
},
Attachments (0)
Change History (2)

Changed June 15, 2010 08:07PM UTC by scottgonzalez comment:1

resolution: → wontfix
status: newclosed

Just use .remove() instead of .dialog( "destroy" ).

Changed October 11, 2012 09:15PM UTC by scottgonzalez comment:2

milestone: TBD

Milestone TBD deleted