Skip to content

Add a draggableRangeOnly options. #203

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

Merged
merged 1 commit into from
Dec 22, 2015
Merged

Add a draggableRangeOnly options. #203

merged 1 commit into from
Dec 22, 2015

Conversation

ValentinH
Copy link
Member

As asked in #196 and #199.
Demo: http://jsfiddle.net/opnrb0hL/

@FantasyLions @yelnar is it what you wanted?

@fantasylion
Copy link

yes! is awesome.Maby can add this feature:move the whole slider when you drag the handle

@yelnar
Copy link

yelnar commented Dec 19, 2015

Cool! And also I think it is would be better if two edges of slider was draggable too. Thank you.

@ValentinH
Copy link
Member Author

Indeed you are both right. I just added what you wanted and also keyboard support.

@yelnar
Copy link

yelnar commented Dec 20, 2015

@ValentinH great, thanks!

@codecov-io
Copy link

Current coverage is 39.76%

Merging #203 into master will decrease coverage by -1.07% as of 02fb4aa

@@            master    #203   diff @@
======================================
  Files            1       1       
  Stmts          578     601    +23
  Branches         0       0       
  Methods          0       0       
======================================
+ Hit            236     239     +3
  Partial          0       0       
- Missed         342     362    +20

Review entire Coverage Diff as of 02fb4aa

Powered by Codecov. Updated on successful CI builds.

ValentinH added a commit that referenced this pull request Dec 22, 2015
@ValentinH ValentinH merged commit b646617 into master Dec 22, 2015
@ValentinH ValentinH deleted the draggableRangeOnly branch December 23, 2015 08:08
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

Successfully merging this pull request may close these issues.

4 participants