github/jquery-ui
1
1
Fork 0
mirror of https://github.com/jquery/jquery-ui.git synced 2026-04-20 03:02:41 -04:00
Code Issues Packages Projects Releases Wiki Activity

slider: fixed disabled on init, methods enable and disable

Browse Source
This commit is contained in:
Richard Worth
2009-09-19 15:19:42 +00:00
parent 122a6cdd5f
commit f164edbdd9
2 changed files with 20 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

16
tests/unit/slider/slider_methods.js
View File

@@ -69,14 +69,26 @@ test("enable", function() {
var expected = $('<div></div>').slider(),
actual = expected.slider('enable');
equals(actual, expected, 'enable is chainable');
ok(false, "missing test - untested code is broken code.");
var el = $('<div></div>').slider({ disabled: true });
ok(el.hasClass('.ui-disabled'), 'slider has ui-disabled class before enable method call');
ok(el.hasClass('.ui-slider-disabled'), 'slider has ui-slider-disabled class before enable method call');
el.slider('enable');
ok(!el.hasClass('.ui-disabled'), 'slider does not have ui-disabled class after enable method call');
ok(!el.hasClass('.ui-slider-disabled'), 'slider does not have ui-slider-disabled class after enable method call');
});
test("disable", function() {
var expected = $('<div></div>').slider(),
actual = expected.slider('disable');
equals(actual, expected, 'disable is chainable');
ok(false, "missing test - untested code is broken code.");
var el = $('<div></div>').slider({ disabled: false });
ok(!el.hasClass('.ui-disabled'), 'slider does not have ui-disabled class before disabled method call');
ok(!el.hasClass('.ui-slider-disabled'), 'slider does not have ui-slider-disabled class before disable method call');
el.slider('disable');
ok(el.hasClass('.ui-disabled'), 'slider has ui-disabled class after disable method call');
ok(el.hasClass('.ui-slider-disabled'), 'slider has ui-slider-disabled class after disable method call');
});
test("value", function() {

6
ui/jquery.ui.slider.js vendored
View File

@@ -29,6 +29,10 @@ $.widget("ui.slider", $.extend({}, $.ui.mouse, {
+ " ui-widget"
+ " ui-widget-content"
+ " ui-corner-all");
if (o.disabled) {
this.element.addClass('ui-slider-disabled ui-disabled');
}
this.range = $([]);
@@ -439,8 +443,10 @@ $.widget("ui.slider", $.extend({}, $.ui.mouse, {
this.handles.filter(".ui-state-focus").blur();
this.handles.removeClass("ui-state-hover");
this.handles.attr("disabled", "disabled");
this.element.addClass("ui-disabled");
} else {
this.handles.removeAttr("disabled");
this.element.removeClass("ui-disabled");
}
case 'orientation':