-
Notifications
You must be signed in to change notification settings - Fork 27.4k
ngInclude breaks in ngSwitch #3650
Comments
Pulled current mainline and added reverts from PR #3640. Expected functionality across app is now as expected. Consider closing upon merge of PR. |
Will not fix. The issue is that both ng-include and ng-switch need transclusion. This used to work before animations but transclusion is required for animation. Work around place extra div between ng-include and ng-switch. |
I updated the example with the suggested workaround but it doesn't seem to work. Perhaps I am misunderstanding what the workaround should be: http://plnkr.co/edit/fCu5Pu?p=preview? |
@mhevery This worked on every other release of angular. How is that a will not fix? |
@patrickrs I think this is related to bug #3627 and they say it is fixed in the snapshot version. I think you are testing in a old version. Try using the snapshot version : http://code.angularjs.org/snapshot/angular.min.js and you will see that your example works. |
With 1.2.0rc1, ngIncludes inside a ngSwitch does not render. Possibly related to #3584.
An previous working example when changed from 1.1.5 to 1.2.0rc1 no longer works:
http://plnkr.co/edit/o29Ybe?p=preview
The text was updated successfully, but these errors were encountered: