feat(slider): customizable event selectors

[lubber-de]

Description

This PR makes the event selector target customizable via settings, so the initial idea of #1987 would be now easier doable based on the new codebase from #2327

@mhthies Please review, i am interested in your opinion

Testcase

https://jsfiddle.net/lubber/k3efcLuj/1/
https://fomantic-ui.com/jsfiddle/#!lubber/k3efcLuj/1/ (for mobile device testing)

Replaces

#1987

[mhthies]

I like the idea! Unfortunately, this doesn't work as expected. Here is an extended JSFiddle with the code from this PR showing the different options:

• https://jsfiddle.net/bcd23hgs/5/
• https://fomantic-ui.com/jsfiddle/#!/bcd23hgs/5/

The mouse setting does not have an effect due to the additional mousedown binding in line 249. I think, we can drop one of the two event bindings (but didn't test it yet).

Setting touchTarget to anything other than .thumb makes (very strange) problems due to my code, which assumes that the event target is always a thumb (see line 332). This section would need to be extended to use module.determine.closestThumb() just like the mouse down event handler does.

[lubber-de]

Thanks for taking a look!

The mouse setting does not have an effect due to the additional mousedown binding in line 249. I think, we can drop one of the two event bindings (but didn't test it yet).

oh, yes, the second one seems to override the first (which is taking care of event bubbling...)
So i guess the second one can be removed ? Yes, have to test this

Setting touchTarget to anything other than .thumb makes (very strange) problems due to my code, which assumes that the event target is always a thumb (see line 332). This section would need to be extended to use module.determine.closestThumb() just like the mouse down event handler does.

Makes sense 👍🏼 I'll test this. Thanks for the hint!