ashatora/openlayers
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Remove events causing errors on touch-screen devices.

Browse Source 
Fix for case #5226
This commit is contained in:
James Cheese
2018-09-24 09:23:49 +01:00
parent 49ee4bbd1c
commit 9c5227850f
2 changed files with 4 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/ol/control/ZoomSlider.js
-2
View File
@@ -243,10 +243,8 @@ class ZoomSlider extends Control {
const end = this.handleDraggerEnd_; const end = this.handleDraggerEnd_;
this.dragListenerKeys_.push( this.dragListenerKeys_.push(
listen(document, EventType.MOUSEMOVE, drag, this), listen(document, EventType.MOUSEMOVE, drag, this),
listen(document, EventType.TOUCHMOVE, drag, this),
listen(document, PointerEventType.POINTERMOVE, drag, this), listen(document, PointerEventType.POINTERMOVE, drag, this),
listen(document, EventType.MOUSEUP, end, this), listen(document, EventType.MOUSEUP, end, this),
listen(document, EventType.TOUCHEND, end, this),
listen(document, PointerEventType.POINTERUP, end, this) listen(document, PointerEventType.POINTERUP, end, this)
); );
} }
test/spec/ol/control/zoomslider.test.js
+4 -4
View File
@@ -119,7 +119,7 @@ describe('ol.control.ZoomSlider', function() {
dragger.dispatchEvent(event); dragger.dispatchEvent(event);
expect(control.currentResolution_).to.be(16); expect(control.currentResolution_).to.be(16);
expect(control.dragging_).to.be(true); expect(control.dragging_).to.be(true);
expect(control.dragListenerKeys_.length).to.be(6); expect(control.dragListenerKeys_.length).to.be(4);
event.type = 'pointermove'; event.type = 'pointermove';
event.clientX = 6 * control.widthLimit_ / 8; event.clientX = 6 * control.widthLimit_ / 8;
event.clientY = 0; event.clientY = 0;
@@ -153,7 +153,7 @@ describe('ol.control.ZoomSlider', function() {
dragger.dispatchEvent(event); dragger.dispatchEvent(event);
expect(control.currentResolution_).to.be(16); expect(control.currentResolution_).to.be(16);
expect(control.dragging_).to.be(true); expect(control.dragging_).to.be(true);
expect(control.dragListenerKeys_.length).to.be(6); expect(control.dragListenerKeys_.length).to.be(4);
event.type = 'pointermove'; event.type = 'pointermove';
event.clientX = 6 * control.widthLimit_ / 8; event.clientX = 6 * control.widthLimit_ / 8;
event.clientY = 0; event.clientY = 0;
@@ -188,7 +188,7 @@ describe('ol.control.ZoomSlider', function() {
dragger.dispatchEvent(event); dragger.dispatchEvent(event);
expect(control.currentResolution_).to.be(0.0625); expect(control.currentResolution_).to.be(0.0625);
expect(control.dragging_).to.be(true); expect(control.dragging_).to.be(true);
expect(control.dragListenerKeys_.length).to.be(6); expect(control.dragListenerKeys_.length).to.be(4);
event.type = 'pointermove'; event.type = 'pointermove';
event.clientX = 0; event.clientX = 0;
event.clientY = 2 * control.heightLimit_ / 8; event.clientY = 2 * control.heightLimit_ / 8;
@@ -222,7 +222,7 @@ describe('ol.control.ZoomSlider', function() {
dragger.dispatchEvent(event); dragger.dispatchEvent(event);
expect(control.currentResolution_).to.be(0.0625); expect(control.currentResolution_).to.be(0.0625);
expect(control.dragging_).to.be(true); expect(control.dragging_).to.be(true);
expect(control.dragListenerKeys_.length).to.be(6); expect(control.dragListenerKeys_.length).to.be(4);
event.type = 'pointermove'; event.type = 'pointermove';
event.clientX = 0; event.clientX = 0;
event.clientY = 2 * control.heightLimit_ / 8; event.clientY = 2 * control.heightLimit_ / 8;