Slider: Fix to restrict handle values when set programmatically.

[dekajp]

Fixes #3762 - Slider: handles not restricted properly when set programmatically

[scottgonzalez]

Uninitialized variables go at the top:

var values,
    i = 0;

[scottgonzalez]

There are some spacing issues and I didn't do a full review because it looks like this still needs some changes to how it works.

[jzaefferer]

@dekajp please comment on your PRs after you push an update, since the push itself has no notification.

@scottgonzalez there's a new commit since yesterday

[scottgonzalez]

Why are the values being cleaned during a get?

[dekajp]

I will change this and move _cleanValues to create

[scottgonzalez]

Maybe we should start with just a set of tests so that we can agree the correct behavior is being tested.

[dekajp]

do you want me to add/modify the unit test cases ?

[dekajp]

From http://bugs.jqueryui.com/ticket/3762#comment:31

1. Restricting the value of a handle based on other handles only applies with range: true and therefore also only applies when exactly two handles exist.
2. When setting both values at the same time, either via .values( array ), .option( "values", array ), or initialization,
   1. the first handle can be any value between min and max and
   2. the second handle must be between the first handle and max.
3. When setting a single handle, it is restricted by the min, max, and existing value of the other handle.

[dekajp]

@scottgonzalez , i will send a new PR on this.

[scottgonzalez]

This is effectively a no-op. Why is it here?

[scottgonzalez]

@dekajp What's the status of this?

[dekajp]

@scottgonzalez last 3 weeks i was busy with relocation. i think this weekend i will be able to get back on this

[scottgonzalez]

Thanks. Hope the move went well.

[dekajp]

@scottgonzalez yeah move went all , everyone happy in family :) with more space . I rebased commits .

[scottgonzalez]

I understand that you're trying to show that the handles are matching, but I'd prefer an explicit value for the expected value.

[jzaefferer]

@dekajp would you be available to rebase this and update the implementation?

[dekajp]

@jzaefferer i rebased by commit. I will look into update the implementation

[sameer05k21a0440]

I have dynamic multiple input and dynamic slider handle so when on change input text value need to set slider value but I can't set slider handle check below code .

 $(this).parents("tr").find('input[name="assesmentScore"]').change( function (event) {
	   					 var textValue = $(this).parents("tr").find('input[name="assesmentScore"]').val();
$("#bonusScore", $(this)).slider('value',textValue);    //here need to add dynamic slide how ? 
	   					handle.text(textValue); //Can't set corresponding text
});

Note :Please let me if needed full code.
Thanks

[dmethvin]

Closing as a stale branch.