Skip to content
Permalink
Browse files

Slider: Style updates

Ref #14246
Ref gh-1588
  • Loading branch information
arschmitz committed Aug 21, 2015
1 parent d86ac02 commit 1d572f051a52c8e4c589a1554e519e81d6c3d7a8
Showing with 11 additions and 11 deletions.
  1. +1 −1 tests/unit/slider/common.js
  2. +1 −1 tests/unit/slider/methods.js
  3. +6 −6 tests/unit/slider/options.js
  4. +3 −3 ui/widgets/slider.js
@@ -23,7 +23,7 @@ common.testWidget( "slider", {
value: 0,
values: null,

// callbacks
// Callbacks
create: null,
change: null,
slide: null,
@@ -88,7 +88,7 @@ test( "value", function() {
equal( element.slider( "value", 2 ), element, "value method is chainable" );
equal( element.slider( "option", "value" ), 1, "value method set respects max" );

// set max value with step 0.01
// Set max value with step 0.01
element.slider( "option", {
min: 2,
value: 2,
@@ -20,7 +20,7 @@ test( "disabled", function( assert ){
count++;
});

// enabled
// Enabled
assert.lacksClasses( element, "ui-slider-disabled" );
equal( element.slider( "option", "disabled" ), false , "is not disabled" );

@@ -30,7 +30,7 @@ test( "disabled", function( assert ){
handle().simulate( "keydown", { keyCode: $.ui.keyCode.RIGHT } );
equal( count, 2, "slider moved" );

// disabled
// Disabled
element.slider( "option", "disabled", true );
assert.hasClasses( element, "ui-slider-disabled" );
equal( element.slider( "option", "disabled" ), true, "is disabled" );
@@ -240,7 +240,7 @@ test( "step", function() {
test( "values", function() {
expect( 2 );

// testing multiple ranges on the same page, the object reference to the values
// Testing multiple ranges on the same page, the object reference to the values
// property is preserved via multiple range elements, so updating options.values
// of 1 slider updates options.values of all the others
var ranges = $([
@@ -270,7 +270,7 @@ test( "range", function( assert ) {
expect( 32 );
var range;

// min
// Min
element = $( "<div></div>" ).slider({
range: "min",
min: 1,
@@ -282,7 +282,7 @@ test( "range", function( assert ) {
equal( element.find( ".ui-slider-range-min" ).length, 1, "range min" );
element.slider( "destroy" );

// max
// Max
element = $( "<div></div>" ).slider({
range: "max",
min: 1,
@@ -294,7 +294,7 @@ test( "range", function( assert ) {
equal( element.find( ".ui-slider-range-max" ).length, 1, "range max" );
element.slider( "destroy" );

// true
// True
element = $( "<div></div>" ).slider({
range: true,
min: 1,
@@ -57,14 +57,14 @@ return $.widget( "ui.slider", $.ui.mouse, {
value: 0,
values: null,

// callbacks
// Callbacks
change: null,
slide: null,
start: null,
stop: null
},

// number of pages in a slider
// Number of pages in a slider
// (how many times can you page up/down to go through the whole range)
numPages: 5,

@@ -516,7 +516,7 @@ return $.widget( "ui.slider", $.ui.mouse, {
}
},

// returns the step-aligned value that val is closest to, between (inclusive) min and max
// Returns the step-aligned value that val is closest to, between (inclusive) min and max
_trimAlignValue: function( val ) {
if ( val <= this._valueMin() ) {
return this._valueMin();

0 comments on commit 1d572f0

Please sign in to comment.
You can’t perform that action at this time.