components icon indicating copy to clipboard operation
components copied to clipboard

feat(module:overlay): directive support disposeOnNavigation

Open wzhudev opened this issue 6 years ago • 8 comments

wzhudev avatar Feb 05 '20 02:02 wzhudev

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

:memo: Please visit https://cla.developers.google.com/ to sign.

Once you've signed (or fixed any issues), please reply here with @googlebot I signed it! and we'll verify it.


What to do if you already signed the CLA

Individual signers
Corporate signers

ℹ️ Googlers: Go here for more info.

googlebot avatar Feb 05 '20 02:02 googlebot

@googlebot I signed it!

wzhudev avatar Feb 05 '20 02:02 wzhudev

CLAs look good, thanks!

ℹ️ Googlers: Go here for more info.

googlebot avatar Feb 05 '20 02:02 googlebot

@crisbeto Please check this bug of ng-zorro https://github.com/NG-ZORRO/ng-zorro-antd/issues/1530 and this bug of components itself https://github.com/angular/components/issues/11478.

But I've not tested them with the latest release of CDK.

wzhudev avatar Feb 05 '20 08:02 wzhudev

I don't see where the overlay directive is being used in the linked example.

crisbeto avatar Feb 05 '20 11:02 crisbeto

@crisbeto Sorry for the misleading.

Material seems just missing a property here.

But in ng-zorro-antd we use Overlay in a declarative way.

wzhudev avatar Feb 05 '20 11:02 wzhudev

That was my point. Wouldn't that component that is used declaratively be destroyed when you navigate to a different route?

crisbeto avatar Feb 05 '20 11:02 crisbeto

That was my point. Wouldn't that component that is used declaratively be destroyed when you navigate to a different route?

I am afraid not, according to the reproductions provided in https://github.com/NG-ZORRO/ng-zorro-antd/issues/1530.

The user applied a router reuse strategy here so it seemed that the component is not destroyed?

wzhudev avatar Feb 05 '20 11:02 wzhudev

This was done in #27672. Closing.

crisbeto avatar Feb 28 '24 08:02 crisbeto

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.