oldValue in changed event is wrong

[pnachum]

I'm using the slider with immediateResponse set to true. The arguments passed to the changed event are the new value, and an object containing oldValue, but oldValue is also the new value, instead of the previous value.

[yankeeinlondon]

I'm heads down on other things at the moment but feel free to send me a PR if you need this more urgently. The problem is that this code works fine with only slideStart and slideStop events are modifying the value:

https://github.com/lifegadget/ui-slider/blob/master/addon/components/ui-slider.js#L283-L284

but with immediateResponse you also set the value on the slide event:

https://github.com/lifegadget/ui-slider/blob/master/addon/components/ui-slider.js#L67

Ok, need coffee. I will get to this at some point but just super busy at moment.

[pnachum]

No worries, it's definitely not urgent. Just something I noticed and figured I should bring to your attention.

I can take a look at trying to fix it

[yankeeinlondon]

thanks, will get to it but not this week