Move inner pushs

Signed-off-by: Kristina Fefelova <kristin.fefelova@gmail.com>
hcengineering · Aug 21, 2024 · 713b89e · 713b89e
1 parent 84803fa
commit 713b89e
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 2 deletions.
diff --git a/packages/ui/src/components/notifications/Notification.svelte b/packages/ui/src/components/notifications/Notification.svelte
@@ -20,6 +20,7 @@
   }
 
   onDestroy(() => {
+    removeNotificationHandler()
     if (closeTimeout && timeout) {
       clearTimeout(timeout)
     }

diff --git a/packages/ui/src/components/notifications/Notifications.svelte b/packages/ui/src/components/notifications/Notifications.svelte
@@ -9,13 +9,15 @@
     'top-left': NotificationPosition.TopLeft,
     'top-right': NotificationPosition.TopRight
   }
+
+  const maxVisibleNotifications = 3
 </script>
 
 <slot />
 <div class="notifications">
   {#each Object.entries(positionByClassName) as [className, position]}
     <div class={className} style:z-index={9999}>
-      {#each $store as notification (notification.id)}
+      {#each $store.slice(0, maxVisibleNotifications) as notification (notification.id)}
         {#if notification.position === position}
           <Notification {notification} />
         {/if}

diff --git a/packages/ui/src/utils.ts b/packages/ui/src/utils.ts
@@ -110,7 +110,7 @@ export function addNotification (
     title,
     subTitle,
     severity,
-    position: NotificationPosition.BottomRight,
+    position: NotificationPosition.TopRight,
     component,
     closeTimeout,
     params