Giter Club home page Giter Club logo

Comments (5)

jnizet avatar jnizet commented on May 1, 2024 1

I've edited the issue to provide a minimal structural directive that reproduces the issue. All it takes is the following, which looks quite innocent to me:

export class FooDirective implements OnInit {
  private tpl = inject(TemplateRef);
  private vcr = inject(ViewContainerRef);

  ngOnInit() {
    Promise.resolve().then(() => {
      this.vcr.createEmbeddedView(this.tpl, {});
    });
  }
}

from angular.

alxhub avatar alxhub commented on May 1, 2024 1

Thank you for the excellent minimal reproduction.

Yep, this is indeed a bug. Digging in, the problem seems to be with the ChangeDetectorRef trick we're using in effect() to know if we're inside a view that has been initialized or not.

When BugDirective calls effect(), we use the LView to decide if the first update pass is already complete (see the code). The ChangeDetectorRef though gives us the LView of the parent component, which of course has already been initialized. Thus we don't defer the effect from scheduling immediately, causing the issue.

from angular.

JeanMeche avatar JeanMeche commented on May 1, 2024

Hi JB !
Is there any reason you opened the issue on the FW and not the transloco repo ? Structural directives provided by the framework like NgIf don't have this issue. It look like it's a problem the way transloco projects the content of the directive.

from angular.

jnizet avatar jnizet commented on May 1, 2024

Hi Matthieu

I assumed that whatever Transloco is doing in its directive shouldn't have an impact on how Angular schedules the execution of an effect: it should be executed after the input has been set, and not before.
Isn't that a reasonable assumption?

from angular.

JeanMeche avatar JeanMeche commented on May 1, 2024

The first effect execution is scheduled to run during the first CD. Transloco is probably triggering a CD before the input has been set.

from angular.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.