From 93d29af2d5d8bcdbd181761c024809591e890297 Mon Sep 17 00:00:00 2001 From: Adam Bradley Date: Thu, 9 Oct 2014 20:35:08 -0500 Subject: [PATCH] chore(menuToggle): simplify --- js/angular/directive/menuToggle.js | 17 ++++++++--------- 1 file changed, 8 insertions(+), 9 deletions(-) diff --git a/js/angular/directive/menuToggle.js b/js/angular/directive/menuToggle.js index 3328e50be8..b7ac95a436 100644 --- a/js/angular/directive/menuToggle.js +++ b/js/angular/directive/menuToggle.js @@ -5,11 +5,11 @@ * @restrict AC * * @description - * Toggle a side menu on the given side + * Toggle a side menu on the given side. * * @usage - * Below is an example of a link within a nav bar. Tapping this link would - * automatically open the given side menu + * Below is an example of a link within a nav bar. Tapping this button + * would open the given side menu, and tapping it again would close it. * * ```html * @@ -21,20 +21,19 @@ * ``` */ IonicModule -.directive('menuToggle', ['$ionicViewService', function($ionicViewService) { +.directive('menuToggle', function() { return { restrict: 'AC', require: '^ionSideMenus', link: function($scope, $element, $attr, sideMenuCtrl) { - var side = $attr.menuToggle || 'left'; $element.bind('click', function(){ - if(side === 'left') { - sideMenuCtrl.toggleLeft(); - } else if(side === 'right') { + if ($attr.menuToggle === 'right') { sideMenuCtrl.toggleRight(); + } else { + sideMenuCtrl.toggleLeft(); } }); } }; -}]); +});