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.

Sign up for GitHub

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

Jump to bottom

fix(ivy): allow inserting attached views to a different container #34156

Closed
wants to merge 3 commits into from
Closed

fix(ivy): allow inserting attached views to a different container #34156

wants to merge 3 commits into from

Commits on Dec 3, 2019

  1. fix(ivy): consistenly return -1 from ViewContainerRef.indexOf for non… 

    …-inserted view
    @pkozlowski-opensource
    pkozlowski-opensource committed Dec 3, 2019
    Configuration menu
    Copy the full SHA
    6929cc0 View commit details
    Browse the repository at this point in the history

  2. fix(ivy): allow insertion of views attached to a different container 

    Fixes angular#34152
    @pkozlowski-opensource
    pkozlowski-opensource committed Dec 3, 2019
    Configuration menu
    Copy the full SHA
    e792cfb View commit details
    Browse the repository at this point in the history

  3. test(ivy): ViewContainerRef.move where the old and new indexes are th… 

    …e same
    @pkozlowski-opensource
    pkozlowski-opensource committed Dec 3, 2019
    Configuration menu
    Copy the full SHA
    894ea73 View commit details
    Browse the repository at this point in the history