perf(menu): avoid change detection when it isn't needed (#8986)

ionic-team · Nov 4, 2016 · d531ec2 · d531ec2
1 parent cf1196a
commit d531ec2
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/src/components/menu/menu-gestures.ts b/src/components/menu/menu-gestures.ts
@@ -21,6 +21,7 @@ export class MenuContentGesture extends SlideEdgeGesture {
       threshold: 0,
       maxEdgeStart: menu.maxEdgeStart || 50,
       maxAngle: 40,
+      zone: false,
       debouncer: new NativeRafDebouncer(),
       gesture: gestureCtrl.create('menu-swipe', {
         priority: GesturePriority.MenuSwipe,
@@ -58,7 +59,6 @@ export class MenuContentGesture extends SlideEdgeGesture {
       'z', z,
       'stepValue', stepValue);
 
-    ev.preventDefault();
     this.menu.swipeProgress(stepValue);
   }
 

diff --git a/src/components/menu/menu.ts b/src/components/menu/menu.ts
@@ -450,7 +450,11 @@ export class Menu {
       return;
     }
     this._getType().setProgessStep(stepValue);
-    this.ionDrag.emit(stepValue);
+
+    let ionDrag = this.ionDrag;
+    if (ionDrag.observers.length > 0) {
+      this._zone.run(ionDrag.emit.bind(ionDrag, stepValue));
+    }
   }
 
   /**