Skip to content

Commit d7b579a

Browse files
committed
A workaround for mouseMove event triggering together with MouseDown
1 parent 1ea3d68 commit d7b579a

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

canvasUtils.js

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ var scrollerObj = new Scroller(function(left, top, zoom) {
99
maxZoom: 10
1010
});
1111

12-
var canvas, ctx, mousedown=false, mousemove=false;
12+
var canvas, ctx, mousedown=false, mousemove=0;
1313

1414
function prepareCanvas(){
1515
canvas=document.getElementById("canvas");
@@ -27,20 +27,20 @@ function prepareCanvas(){
2727

2828
canvas.addEventListener("mousedown", function(e) {
2929
scrollerObj.doTouchStart([{pageX: e.pageX,pageY: e.pageY}], e.timeStamp);
30-
mousemove=false;
30+
mousemove=0;
3131
mousedown = true;
3232
redraw();
3333
}, false);
3434

3535
canvas.addEventListener("mousemove", function(e) {
36-
mousemove=true;
36+
mousemove+=1;
3737
if (!mousedown) {return;}
3838
scrollerObj.doTouchMove([{pageX: e.pageX,pageY: e.pageY}], e.timeStamp);
3939
redraw();
4040
}, false);
4141

4242
canvas.addEventListener("mouseup", function(e) {
43-
if(!mousemove) { //it was a click
43+
if(mousemove<=1) { //ignore the first mouseMove as sometimes it's triggered together with mouseDown
4444
doClick(e);
4545
}
4646
if (!mousedown) {return;}

0 commit comments

Comments
 (0)