IonDen / ion.rangeSlider

jQuery only range slider
http://ionden.com/a/plugins/ion.rangeSlider/en.html
MIT License
2.55k stars 506 forks source link

onFinish is not working in ionRangeSlider #383

Closed Raghava525 closed 8 years ago

Raghava525 commented 8 years ago

Can you please provide the what is the bug in the following code

$("#my_slide").ionRangeSlider({
    type: "double",
    min: 1000,
    max: 12000,
    step: 500,
    force_edges: true,
    onFinish:function () {
        alert('slider drag finished');
        }     
});

There is no alert after finishing the click and drag on slider.

IonDen commented 8 years ago

http://jsfiddle.net/278ktdnz/ It is working.

You may have some local JS-issues, open web developer console and look for errors. Remember, if you are using some other code, alert function could be changed in it and will do nothing, like this: http://jsfiddle.net/278ktdnz/1/

Raghava525 commented 8 years ago

Still its not alerting the result after completion of slider dragging from the link you have provided too ..... Please provide one example to alert a statement after finishing the slide drag.

IonDen commented 8 years ago

This link will have alert after drag: http://jsfiddle.net/278ktdnz/ If you don't see alert, then you have some problem with your browser, or maybe some browser extension, which blocks it.

Raghava525 commented 8 years ago

Ok Thankyou. Its working fine. You can close the discussion. :)

valtido commented 7 years ago

Why does this onFinish event triggers when the page first loads up? or when it is initialized ?

this is really puzzling me. (as from @IonDen jsfiddle below)

http://jsfiddle.net/278ktdnz/

IonDen commented 7 years ago

@valtido, it was the recent change to slider core, which i will probably revert soon, because it is really confusing.

valtido commented 7 years ago

Ok, cool, could you keep us posted please, I'd like to update to that version :)

mobilemeneer commented 7 years ago

@IonDen Did you revert the change to slider core for the onFinish event which is auto triggered with a page load? This is causing a heavy load on my page because i use three sliders which all post to the db.

Many thanks in advance.

IonDen commented 7 years ago

@mobilemeneer, revert will be ready next week

mobilemeneer commented 7 years ago

Thanks man

I could also install an older version but now I will wait for next week

Op do 9 mrt. 2017 om 21:43 schreef Denis Ineshin notifications@github.com

@mobilemeneer https://github.com/mobilemeneer, revert will be ready next week

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/IonDen/ion.rangeSlider/issues/383#issuecomment-285477146, or mute the thread https://github.com/notifications/unsubscribe-auth/AJvGxyMYPy4MDHl_hx-SUygcp3409D8Eks5rkGRzgaJpZM4IsPyv .