Skip to content

Commit

Permalink
fix(exposeAsideWhen): disable with isEnabled=false
Browse files Browse the repository at this point in the history
Closes #2210
  • Loading branch information
adamdbradley committed Sep 15, 2014
1 parent 74de015 commit 6f79a5e
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 0 deletions.
2 changes: 2 additions & 0 deletions js/angular/controller/sideMenuController.js
Original file line number Diff line number Diff line change
Expand Up @@ -259,6 +259,8 @@ function($scope, $attrs, $ionicSideMenuDelegate, $ionicPlatform, $ionicBody) {
};

self.exposeAside = function(shouldExposeAside) {
if(!self.left || !self.left.isEnabled) return;

self.close();
isAsideExposed = shouldExposeAside;

Expand Down
10 changes: 10 additions & 0 deletions test/unit/angular/controller/sideMenuController.unit.js
Original file line number Diff line number Diff line change
Expand Up @@ -134,6 +134,16 @@ describe('$ionicSideMenus controller', function() {
expect(ctrl.getOpenPercentage()).toEqual(0);
});

it('should set enabled/disabled exposeAside', function() {
expect(ctrl.isAsideExposed()).toEqual(false);
ctrl.left.setIsEnabled(false);
ctrl.exposeAside(true);
expect(ctrl.isAsideExposed()).toEqual(false);
ctrl.left.setIsEnabled(true);
ctrl.exposeAside(true);
expect(ctrl.isAsideExposed()).toEqual(true);
});

it('should toggle right', function() {
ctrl.toggleRight();
expect(ctrl.getOpenPercentage()).toEqual(-100);
Expand Down

0 comments on commit 6f79a5e

Please sign in to comment.