From f246c5aa2094ec74865ce149b3f1a1183dc4950a Mon Sep 17 00:00:00 2001 From: Adam Bradley Date: Wed, 6 Aug 2014 15:24:31 -0500 Subject: [PATCH] fix(sideMenu): remove .menu-open on destroy If a menu was opened when navigating to a different view, it is possible that the `menu-open` class was left on the body tag, which disables the elements on the next side menu view. On side menu destroy, ensure menu-open was removed from the body class. --- js/angular/directive/sideMenus.js | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/js/angular/directive/sideMenus.js b/js/angular/directive/sideMenus.js index 84579a7a99..de3eb6cf9a 100644 --- a/js/angular/directive/sideMenus.js +++ b/js/angular/directive/sideMenus.js @@ -57,12 +57,19 @@ IonicModule * with {@link ionic.service:$ionicSideMenuDelegate}. * */ -.directive('ionSideMenus', [function() { +.directive('ionSideMenus', ['$document', function($document) { return { restrict: 'ECA', controller: '$ionicSideMenus', compile: function(element, attr) { attr.$set('class', (attr['class'] || '') + ' view'); + + return function($scope) { + $scope.$on('$destroy', function(){ + $document[0].body.classList.remove('menu-open'); + }); + + } } }; }]);