diff --git a/js/ext/angular/src/directive/ionicContent.js b/js/ext/angular/src/directive/ionicContent.js index c64bce4885..2bbd72afe1 100644 --- a/js/ext/angular/src/directive/ionicContent.js +++ b/js/ext/angular/src/directive/ionicContent.js @@ -42,7 +42,7 @@ angular.module('ionic.ui.content', ['ionic.ui.service', 'ionic.ui.scroll']) * directive, and infinite scrolling with the {@link ionic.directive:ionInfiniteScroll} * directive. * - * Use the classes 'has-header', 'has-subheader', 'has-footer', and 'has-tabs' + * Use the classes 'has-header', 'has-subheader', 'has-footer', and 'has-tabs' * to modify the positioning of the ion-content relative to surrounding elements. * * @param {boolean=} padding Whether to add padding to the content. @@ -72,8 +72,6 @@ function($parse, $timeout, $controller, $ionicBind) { '
' + '', compile: function(element, attr, transclude) { - if(attr.padding == "true") { element.find('div').addClass('padding'); } - return { //Prelink so it can compile before other directives compile. //Then other directives can require ionicScrollCtrl @@ -100,7 +98,7 @@ function($parse, $timeout, $controller, $ionicBind) { }); $scope.$watch($attr.padding, function(newVal) { - $element.toggleClass('padding', !!newVal); + scrollContent.toggleClass('padding', !!newVal); }); if ($scope.scroll === "false") { diff --git a/js/ext/angular/test/directive/ionicContent.unit.js b/js/ext/angular/test/directive/ionicContent.unit.js index c191097d27..a43a314b66 100644 --- a/js/ext/angular/test/directive/ionicContent.unit.js +++ b/js/ext/angular/test/directive/ionicContent.unit.js @@ -22,11 +22,13 @@ describe('Ionic Content directive', function() { }); it('should add padding classname', function() { - var element = compile('')(scope); - expect(element.hasClass('scroll-content')).toEqual(true); - expect(element.hasClass('padding')).toEqual(false); + var element = compile('')(scope); var scrollElement = element.find('.scroll'); + expect(scrollElement.hasClass('padding')).toEqual(false); + element.scope().$apply('shouldPad = true'); expect(scrollElement.hasClass('padding')).toEqual(true); + element.scope().$apply('shouldPad = false'); + expect(scrollElement.hasClass('padding')).toEqual(false); }); it('Disables bouncing when has-bouncing = false', function() {