Skip to content

fix(NgFor): allow default templates with ng-for-template#5161

Closed
pkozlowski-opensource wants to merge 1 commit into
angular:masterfrom
pkozlowski-opensource:ng-for-template-default
Closed

fix(NgFor): allow default templates with ng-for-template#5161
pkozlowski-opensource wants to merge 1 commit into
angular:masterfrom
pkozlowski-opensource:ng-for-template-default

Conversation

@pkozlowski-opensource
Copy link
Copy Markdown
Member

@vsavkin sth I've discovered while playing with ng-for-template. This small change makes it possible to have a "default" template inside ngFor - a template that can be overridden with a non-null binding to ng-for-template.

In practice it allows us to write components that got default markup which can be, optionally, overridden from outside.

@pkozlowski-opensource pkozlowski-opensource added the action: review The PR is still awaiting reviews from at least one requested reviewer label Nov 6, 2015
@vsavkin vsavkin added pr_state: LGTM and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels Nov 6, 2015
@pkozlowski-opensource pkozlowski-opensource added the action: merge The PR is ready for merge by the caretaker label Nov 6, 2015
@mary-poppins
Copy link
Copy Markdown

Merging PR #5161 on behalf of @pkozlowski-opensource to branch presubmit-pkozlowski-opensource-pr-5161.

@mary-poppins mary-poppins removed the action: merge The PR is ready for merge by the caretaker label Nov 6, 2015
@angular-automatic-lock-bot
Copy link
Copy Markdown

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot Bot locked and limited conversation to collaborators Sep 7, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants