diff --git a/src/ng/directive/ngIf.js b/src/ng/directive/ngIf.js index 35c122ad12d9..78547f471f24 100644 --- a/src/ng/directive/ngIf.js +++ b/src/ng/directive/ngIf.js @@ -87,23 +87,31 @@ var ngIfDirective = ['$animate', function($animate) { $$tlb: true, compile: function (element, attr, transclude) { return function ($scope, $element, $attr) { - var block = {}, childScope; + var block, childScope; $scope.$watch($attr.ngIf, function ngIfWatchAction(value) { - if (block.startNode) { - $animate.leave(getBlockElements(block)); - block = {}; - } - if (block.startNode) { - getBlockElements(block).$destroy(); - block = {}; - } + if (toBoolean(value)) { + childScope = $scope.$new(); transclude(childScope, function (clone) { - block.startNode = clone[0]; - block.endNode = clone[clone.length++] = document.createComment(' end ngIf: ' + $attr.ngIf + ' '); + block = { + startNode: clone[0], + endNode: clone[clone.length++] = document.createComment(' end ngIf: ' + $attr.ngIf + ' ') + }; $animate.enter(clone, $element.parent(), $element); }); + + } else { + + if (childScope) { + childScope.$destroy(); + childScope = null; + } + + if (block) { + $animate.leave(getBlockElements(block)); + block = null; + } } }); }; diff --git a/test/ng/directive/ngIfSpec.js b/test/ng/directive/ngIfSpec.js index 509cf26db371..ce4de98c5f87 100755 --- a/test/ng/directive/ngIfSpec.js +++ b/test/ng/directive/ngIfSpec.js @@ -36,7 +36,7 @@ describe('ngIf', function () { expect(element.children().length).toBe(0); }); - it('should create a new scope', function () { + it('should create a new scope every time the expression evaluates to true', function () { $scope.$apply('value = true'); element.append($compile( '