search

paveltyavin / multirangeslider

Slider for timetable and more
http://paveltyavin.github.io/multirangeslider/
MIT License
20 stars 6 forks source link

Mouse down event required to place a new range. #16

Closed Khopa closed 6 years ago

Khopa commented 6 years ago

Hello,

I'm using multiple "multiple-range sliders" next to each other in one of my project, that i'm currently trying to migrate from Elessar to your implementation.

I noticed a small issue : when dragging the mouse to define a range on one bar, if the mouse up event ends up occuring on another bar, it'll add a new range on this other bar.

As you can see :

I modified it so an explicit mouse down event on the bar is required before being actually able to place a new range now :

paveltyavin commented 6 years ago

Khopa, wich browser did you use?

Khopa commented 6 years ago

Chrome latest (64), Firefox (don't know which version, i don't have access to this machine right now), and IE11. (Which is a requirement for my project and I had to include Babel to make it work with IE11)

Behaviour was the same on the three browsers.

paveltyavin commented 6 years ago

so do you experience problems still here https://jsfiddle.net/paveltyavin/4qw7onb0/1/ ? the thing is that I couldn't reproduce bugs

Khopa commented 6 years ago

No, i do not reproduce it anymore on the fiddle, with the new version.

I made a fiddle here (https://jsfiddle.net/Khopa/hjr29hw2/13/), using the current dist version that is available on the master branch (which i copy/pasted).

I believe you should be able to see the issue reproduced there.