Fix: Fill styling for sliders

[oliverfoster]

fixes #177

Fix

• Allow fill styling in slider track

EDIT:

Fix

• Added index to each model item
• Removed index to value calculations
• Set up handle sizing
• Corrected fill alignment
• Corrected number alignment
• Corrected slider bar alignment
• Corrected correct answer range alignment
• Nested input fill inside of input track

[swashbuck]

Do we still want to change the color of the fill based on correct vs. incorrect? If so, we can borrow the styles from the deprecated rangeslider.

Then, what should we do when showing the correct answer when it is a range?

Should the fill extend to where the handle is (below 70)?

Should the fill span only over the correct range (50 - 90)?

[swashbuck]

Also wondering if we should hide the handle after the question has been submitted.

[oliverfoster]

Incorrect red, yes.
Should the fill extend to where the handle is (below 70), yes.
Hide handle, yes.

The fill describes the user's selection (width) and correctness (color). But not the correct range. Otherwise the ui is switching its function.

[oliverfoster]

@swashbuck would you mind picking up the remaining work?

[swashbuck]

@swashbuck would you mind picking up the remaining work?

@oliverfoster Sure!

[kirsty-hames]

Looking good thanks @swashbuck. Tested on Mac Safari, Chrome and FF. iPhone and Android tablet.

[guywillis]

Made a few adjustments / optimisations in the plugin and moved the styling parts to the vanilla theme to pick up - adaptlearning/adapt-contrib-vanilla#408

[joe-allen-89]

Tested on Windows machine, Chrome, Edge and Firefox, all seems to be working nicely.

@guywillis @oliverfoster @swashbuck one query, if you answer incorrectly and click show correct answer you see the range of correct answers, if you get the question right you only see your answer, would it make sense to see the range if you answer correctly?

[kirsty-hames]

I've just given this another test on the same spec above to review the latest commits. Looks good thanks.

[oliverfoster]

Tested on Windows machine, Chrome, Edge and Firefox, all seems to be working nicely.

@guywillis @oliverfoster @swashbuck one query, if you answer incorrectly and click show correct answer you see the range of correct answers, if you get the question right you only see your answer, would it make sense to see the range if you answer correctly?

This pr is meant to fix the existing implementation and return it to functionality. Discussions about new behaviour or behaviour changes should be put into a separate issue. But, sure. Sounds like a good idea.