Modal/Dialog memory leak

[beig]

Issue type

I'm submitting a ... (check one with "x")

• bug report
• feature request

Issue description

Current behavior:
Modal Components stay in memory

Expected behavior:
They should get destroyed

Steps to reproduce:
create new angular project

ng new test
ng add @nebular/theme
# add NbDialogModule.forRoot() in app.module
# create another component and open it with the NbDialogservice
this.dialogService.open(TestComponent);

take a memory snapshot with dev tools

Related code:

insert short code snippets here

Other information:

npm, node, OS, Browser

<!--
Node, npm: `node --version` and `npm --version`
OS: Windows 10
Browser: Chrome
-->

Angular, Nebular

    "@angular/common": "~8.1.0",
    "@nebular/theme": "^4.1.2",

[yggg]

Thanks for reporting @beig, I can confirm the issue.

Related code:

nebular/src/framework/theme/components/dialog/dialog.service.ts

Line 238 in 3d2e4df

protected registerCloseListeners<T>(config: NbDialogConfig, overlayRef: NbOverlayRef, dialogRef: NbDialogRef<T>) {

Adding takeUntil(dialogRef.onClose) should be enough. Also, we need to check window component for the same problem.

[beig]

Ich bin derzeit nicht im Büro. Ich kehre zurück am 05.08.2019. Ich werde Ihre Nachricht nach meiner Rückkehr beantworten. Die Rückmeldung bezieht sich auf ein Mail mit folgendem Thema: Re: [akveo/nebular] Modal/Dialog memory leak (#1847)

…

____________________________________________________________________________________________ Gesendet (c) GRZ/RACON Linz 2019 Agent 'Abwesenheit'

---------------------------------------- Raiffeisen Software GmbH Firmenbuchnummer: 86804d, Landesgericht Linz DVR: 0572934, UID: ATU 36961105 Der Austausch von Nachrichten mit dem oben angeführten Absender via E-Mail dient ausschließlich Informationszwecken. Rechtsgeschäftliche Erklärungen dürfen über dieses Medium nicht ausgetauscht werden. Correspondence with above mentioned sender via e-mail is only for information purposes. This medium is not to be used for the exchange of legally-binding communications. ----------------------------------------