X Tutup
Skip to content

feat(Compiler): Allow overriding the projection selector#7742

Closed
vicb wants to merge 1 commit intoangular:masterfrom
vicb:0323-projAs
Closed

feat(Compiler): Allow overriding the projection selector#7742
vicb wants to merge 1 commit intoangular:masterfrom
vicb:0323-projAs

Conversation

@vicb
Copy link
Copy Markdown
Contributor

@vicb vicb commented Mar 23, 2016

  • Please check if the PR fulfills these requirements
  • The commit message follows our guidelines:
  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)
BREAKING CHANGE:

For static content projection, elements with *-directives are now matched against the element itself vs the template before.

    <p *ngIf="condition" foo></p>

Before:

    // Use the implicit template for projection
    <ng-content select="template"></ng-content>

After:

    // Use the actual element for projection
    <ng-content select="p[foo]"></ng-content>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Move this out of the if right after the var projectionSelector =.

@tbosch tbosch added the action: cleanup The PR is in need of cleanup, either due to needing a rebase or in response to comments from reviews label Mar 23, 2016
@tbosch tbosch assigned vicb and unassigned tbosch Mar 23, 2016
@vicb vicb removed the action: cleanup The PR is in need of cleanup, either due to needing a rebase or in response to comments from reviews label Mar 23, 2016
fixes angular#6303

BREAKING CHANGE:

For static content projection, elements with *-directives are now matched against the element itself vs the template before.

    <p *ngIf="condition" foo></p>

Before:

    // Use the implicit template for projection
    <ng-content select="template"></ng-content>

After:

    // Use the actual element for projection
    <ng-content select="p[foo]"></ng-content>
@vicb vicb added the action: merge The PR is ready for merge by the caretaker label Mar 24, 2016
@mary-poppins
Copy link
Copy Markdown

Merging PR #7742 on behalf of @kara to branch presubmit-kara-pr-7742.

@mhevery mhevery closed this in aa966f5 Mar 24, 2016
@vicb vicb deleted the 0323-projAs branch June 23, 2016 17:43
@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 8, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: merge The PR is ready for merge by the caretaker cla: yes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants

X Tutup