Skip to content
This repository has been archived by the owner on May 30, 2024. It is now read-only.

Add CSS Transition to Slider Change #84

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from

Conversation

EnigmaSolved
Copy link

I've used the Frequency Decoder fd-slider for years and one of the things I always liked was that if you click/tap on a space on the slider (other than the slider handle) then the slider would animate/transition smoothly to that new position.

This Pull Request adds SCSS and JS to rangeslider.js to optionally add a CSS transition effect like described above. I set the defaults to enable the transition, but that is a judgment call. You may wish to reverse that (which will be super-simple with how I've coded it).

I like what I've see of rangeslider.js! I appreciate that it is very lightweight and efficient, and does what it does well. :) And since fd-slider is no longer being updated I'm hoping to switch over to rangeslider.js at some point in the future.

Thanks!
Sean

I think this is helpful particularly for desktop in that clicking on the range can be easier with a mouse that clicking and dragging the handle. Adding the Pointer style for the whole range thus clues the user into the fact that the entire range is clickable.

I also removed the Pointer from the handle class as it seems redundant with having added it to the whole range.
@EnigmaSolved
Copy link
Author

@andreruffert, Any chance of this getting merged in at some point? :)

Sean

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

Successfully merging this pull request may close these issues.

1 participant