r3170 committed - selectable refactor: fixed destroy to remove ui-widget class
Revision: 3170
Author: rdworth
Date: Fri Aug 28 05:45:47 2009
Log: selectable refactor: fixed destroy to remove ui-widget class
http://code.google.com/p/jquery-ui/source/detail?r=3170
Modified:
/branches/dev/selectable/tests/unit/selectable/selectable_methods.js
/branches/dev/selectable/ui/ui.selectable.js
=======================================
--- /branches/dev/selectable/tests/unit/selectable/selectable_methods.js
Sun Aug 23 03:01:34 2009
+++ /branches/dev/selectable/tests/unit/selectable/selectable_methods.js
Fri Aug 28 05:45:47 2009
@@ -30,7 +30,7 @@
});
test("destroy", function() {
- expect(7);
+ expect(8);
$("<div></div>").appendTo('body').selectable().selectable("destroy").remove();
ok(true, '.selectable("destroy") called on element');
@@ -51,6 +51,9 @@
$("<div></div>").selectable().selectable("destroy").data("foo.selectable", "bar").remove();
ok(true, 'arbitrary option setter after destroy');
+
+ el = $("#selectable1").selectable().selectable('destroy');
+ ok(!el.hasClass('ui-widget'), 'ui-widget class removed on destroy');
var expected = $('<div></div>').selectable(),
actual = expected.selectable('destroy');
=======================================
--- /branches/dev/selectable/ui/ui.selectable.js Thu Aug 27 07:31:09 2009
+++ /branches/dev/selectable/ui/ui.selectable.js Fri Aug 28 05:45:47 2009
@@ -189,7 +189,7 @@
destroy: function() {
this.items.removeClass("ui-selected ui-state-default ui-state-active");
this.element
- .removeClass("ui-selectable ui-selectable-disabled")
+ .removeClass("ui-selectable ui-selectable-disabled ui-widget")
.removeData("selectable")
.unbind(".selectable");
this._mouseDestroy();