Skip to content

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