From 77274a0a60e19b19a571176d3035c1e99df09dde Mon Sep 17 00:00:00 2001 From: Miguel Andrade Date: Thu, 9 Mar 2017 01:34:16 +0000 Subject: [PATCH] set md-dragging class correctly. attach recognizers to the whole slider. big usability improvement. --- addon/components/paper-slider.js | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/addon/components/paper-slider.js b/addon/components/paper-slider.js index 457a28bc8..51adcea29 100644 --- a/addon/components/paper-slider.js +++ b/addon/components/paper-slider.js @@ -21,7 +21,7 @@ export default Component.extend(FocusableMixin, ColorMixin, { attributeBindings: ['min', 'max', 'step', 'discrete:md-discrete', 'tabindex'], classNames: ['md-default-theme'], - classNameBindings: ['isMinimum:md-min', 'active', 'dragging'], + classNameBindings: ['isMinimum:md-min', 'active', 'dragging:md-dragging'], constants: inject.service(), @@ -78,10 +78,8 @@ export default Component.extend(FocusableMixin, ColorMixin, { }, _setupHammer() { - let thumbContainer = this.$('.md-thumb-container').get(0); - // Enable dragging the slider - let containerManager = new Hammer.Manager(thumbContainer); + let containerManager = new Hammer.Manager(this.element); let pan = new Hammer.Pan({ direction: Hammer.DIRECTION_HORIZONTAL, threshold: 10 }); containerManager.add(pan);