>+ window.removeEventListener("MozRotateGestureUpdate", this.onRotateGesture, true) >+ window.removeEventListener("MozRotateGestureStart", this.onRotateGesture, true) >+ window.removeEventListener("MozMagnifyGesture", this.stopEvent, true) >+ window.removeEventListener("MozMagnifyGestureUpdate", this.onMagnifyGestureUpdate, true) >+ window.removeEventListener("MozMagnifyGestureStart", this.onMagnifyGestureStart, true) >+ window.removeEventListener("MozSwipeGesture", this.onSwipeGesture, true) >+ window.addEventListener("MozRotateGesture", this.stopEvent, true) >+ window.addEventListener("MozRotateGestureUpdate", this.onRotateGesture, true) >+ window.addEventListener("MozRotateGestureStart", this.onRotateGesture, true) >+ window.addEventListener("MozMagnifyGesture", this.stopEvent, true) >+ window.addEventListener("MozMagnifyGestureUpdate", this.onMagnifyGestureUpdate, true) >+ window.addEventListener("MozMagnifyGestureStart", this.onMagnifyGestureStart, true) >+ window.addEventListener("MozSwipeGesture", this.onSwipeGesture, true) Otherwise you'll have to keep changing it when you contribute some more ) I don't think qualifying what you added is necessary. >diff -git a/browser/base/content/browser.js b/browser/base/content/browser.js
#Macbook multi touch gestures Patch#
:) I've attached a patch on top of yours that implements some of what I've commented on. I can't give an official review for this area, but here's some feedback if you want to use any of it.
0 kommentar(er)
