fix(onStart): Remove stop propagation call so events will bubble up

[andreeib]

Closes #611

[codecov-io]

Codecov Report

Merging #612 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master   #612   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files           1      1           
  Lines         950    949    -1     
=====================================
- Hits          950    949    -1

Impacted Files	Coverage Δ
src/rzslider.js	100% <100%> (ø)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 698f927...9a8be6c. Read the comment docs.

[ValentinH]

Thanks! :)

[stripathix]

In ionic, there is an event attached to swipe left/right to show/hide side menu. Removing event.stopPropagation() on start causes opening of side menu when the slider is dragged to right. There can be many different cases or framework which has an event attached to swipe especially for mobile application, I am not sure removing event.stopPropagation() from the library was a good idea because now every developer who uses this library for mobile apps will have to write onStart option to ensure event.stopPropagation(); to prevent side menu from opening.

To solve this I have added an option for slider configuration.

options: { onStart: function () { event.stopPropagation(); }, }

[stripathix]

Adding onStart as an option does not solve this issue of side menu opening. Rolling back to 6.4.3 :-(