Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Nest ng-include inside header. #58

Closed
wants to merge 1 commit into from
Closed

Nest ng-include inside header. #58

wants to merge 1 commit into from

Conversation

bcronje
Copy link

@bcronje bcronje commented Oct 11, 2013

AngularJS 1.2 does not support ng-repeat and ng-include on the same
element. The official solution is to place the ng-include element
directly inside of ng-repeat. See comment from @matsko at
angular/angular.js#3584 (comment)
for more information.

AngularJS 1.2 does not support ng-repeat and ng-include on the same
element. The official solution is to place the ng-include element
directly inside of ng-repeat. See comment from @matsko at
angular/angular.js#3584 (comment)
for more information.
@lorenzofox3
Copy link
Owner

Thanks again :D. I'll check this later but for the moment I am not working on making it compatible with 1.2.* versions as they are not stable may change quickly.

Laurent

@matsko
Copy link

matsko commented Oct 17, 2013

@lorenzofox3 as an FYI, 1.2rc3 should be 1.2 and then 1.3 will become the new unstable API. This means that 1.2 will stay stable upon next release so you should be safe relying on 1.2.

@matsko
Copy link

matsko commented Oct 17, 2013

Also @bcronje, ng-repeat + ng-include should work together now: angular/angular.js@9efa46a

@VasimHayat
Copy link

ok

This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants