Skip to content

rz-slider-on-change fires before value is changed #106

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
kritollm opened this issue Aug 20, 2015 · 8 comments
Closed

rz-slider-on-change fires before value is changed #106

kritollm opened this issue Aug 20, 2015 · 8 comments

Comments

@kritollm
Copy link

$scope.onRzSliderChange = function (value) {
console.log('changed', $scope.rzProfitSlider);
}

Is logging the old value

@ValentinH
Copy link
Member

Indeed. Let me fix this ;)

@sebastialonso
Copy link

Is anybody still having issues with this? After getting version 0.1.26?

EDIT: what version does bower has? the minimized file says v0.1.25 and the not minimized file says v0.1.21. bower.json says 0.1.26.

@ValentinH
Copy link
Member

Indeed the version number are messy... The source file should be OK for this issue and I think the minimized one too but let me check tomorrow (not at work anymore for today).

@sebastialonso
Copy link

Thanks for you quick answer and for the awesome module.

Just to check if what I'm trying to do is wrong, I started playing with the fiddle, which links to last version on Github.

fiddle example

Link to fiddle http://jsfiddle.net/pq7yr6d6/53/

Is this part of the design? Notice how otherVariable needs another input from the user to the slider, just to achieve the value it should had achieved on the previous input. I have been on this for hours so I apologize if I'm making a mistake here.

@ValentinH
Copy link
Member

Could you add the jsfiddle link please?

@sebastialonso
Copy link

Here you go http://jsfiddle.net/pq7yr6d6/53/

edit: it's 53. Sorry.

@ValentinH
Copy link
Member

This is good now there was indeed still an issue.

@sebastialonso
Copy link

Great. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants