Skip to content
This repository was archived by the owner on Oct 8, 2021. It is now read-only.

Commit 31c0b2a

Browse files
cgackarschmitz
authored andcommitted
Slider: input does not get ui-state-disabled class
Fixes gh-7024 Fixes gh-6532 Closes gh-7617 (cherry picked from commit 61bdcd7)
1 parent 5e1a940 commit 31c0b2a

File tree

3 files changed

+15
-0
lines changed

3 files changed

+15
-0
lines changed

js/widgets/forms/slider.js

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -585,6 +585,8 @@ $.widget( "mobile.slider", $.extend( {
585585
this.slider
586586
.toggleClass( "ui-state-disabled", value )
587587
.attr( "aria-disabled", value );
588+
589+
this.element.toggleClass( "ui-state-disabled", value );
588590
}
589591

590592
}, $.mobile.behaviors.formReset ) );

tests/unit/slider/index.html

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -128,6 +128,10 @@
128128
<input type="range" name="tooltip-test-both" id="tooltip-test-both" value="17" min="0" max="100" data-nstest-popup-enabled="true" data-nstest-show-value="true"/>
129129
</label>
130130

131+
<label>Input slider:
132+
<input type="range" name="disable-input-test" id="disable-input-test" value="17" min="0" max="100" />
133+
</label>
134+
131135
</div>
132136

133137
<div id="enhancetest">

tests/unit/slider/slider_core.js

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -114,4 +114,13 @@
114114

115115
ok( slider.val() === sliderHandle.text(), "slider text should match handle text");
116116
});
117+
118+
test( "slider input is disabled correctly", function() {
119+
var slider = $( "#disable-input-test" );
120+
121+
slider.slider( "disable" );
122+
123+
ok( slider.hasClass( "ui-state-disabled" ), "disabling slider also disables the input" );
124+
});
125+
117126
})( jQuery );

0 commit comments

Comments
 (0)