notifications: toaster container closes when empty (#12457)

FernandoAscencio · web-flow · commit 2f8b8b65d46f · 2023-05-17T11:48:32.000-04:00
The commit adds a condition so that the toaster container will close when empty.
The issue was found when testing 12446. When seemingly it needed two `esc`.

Signed-off-by: FernandoAscencio &lt;fernando.ascencio.cama@ericsson.com&gt;
diff --git a/packages/messages/src/browser/notifications-manager.ts b/packages/messages/src/browser/notifications-manager.ts
@@ -131,7 +131,7 @@ export class NotificationManager extends MessageClient {
             return;
         }
         this.deferredResults.delete(messageId);
-        if (this.centerVisible && this.notifications.size === 0) {
+        if ((this.centerVisible && !this.notifications.size) || (this.toastsVisible && !this.toasts.size)) {
             this.visibilityState = 'hidden';
         }
         result.resolve(action);

Original file line number	Diff line number	Diff line change
`@@ -131,7 +131,7 @@ export class NotificationManager extends MessageClient {`
`131`	`131`	`return;`
`132`	`132`	`}`
`133`	`133`	`this.deferredResults.delete(messageId);`
`134`		`- if (this.centerVisible && this.notifications.size === 0) {`
	`134`	`+ if ((this.centerVisible && !this.notifications.size) \|\| (this.toastsVisible && !this.toasts.size)) {`
`135`	`135`	`this.visibilityState = 'hidden';`
`136`	`136`	`}`
`137`	`137`	`result.resolve(action);`