-
Notifications
You must be signed in to change notification settings - Fork 2.4k
Conversation
…ween slider-start and current handle-position
+1 |
Looks super nice (especially the the test :D), any chance you can resubmit this without the extra merge commits? Ie, new branch and cherry pick the single relevant commit. Is this something we want to support from a usability perspective? |
I was just saying that I wanted to add this as an option at some point so this is fantastic. Just confirming - this feature is activated by adding the In terms of docs, I'd like to leave the first slider alone because it is meant to show the default slider. We can add this as another demo below. This new data- attr needs to be documented here: |
@johnbender @toddparker |
Alright, I'm going to close this pull request in expectation of your update. |
I added a third theme to the ui.slider to highlight the part between slider-start and the current handle, like the jquery-ui slider does.