From 37326364fca13db5c2e894f401693c9870b070e1 Mon Sep 17 00:00:00 2001
From: arkaprota <arkaprota.ghosh@gmail.com>
Date: Tue, 17 Sep 2024 15:52:23 +0530
Subject: [PATCH 1/3] fix(core): :bug: notificationpopover is design is
 completed properly

notificationpopover has been fixed

Ref: #373
---
 package/components/utils/NotificationPopOver.js | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/package/components/utils/NotificationPopOver.js b/package/components/utils/NotificationPopOver.js
index 34b26f48..02e2df32 100644
--- a/package/components/utils/NotificationPopOver.js
+++ b/package/components/utils/NotificationPopOver.js
@@ -13,18 +13,19 @@ export default function NotificationPopOver() {
     <>
       <CoreBox styleClasses={[CoreClasses.POPOVER.CONTENT]}>
         <CoreBox
-          styleClasses={[CoreClasses?.FLEX?.DIRECTION_ROW, CoreClasses.ALIGNMENT.JUSTIFY_CONTENT_SPACE_BETWEEN, CoreClasses.LAYOUT.VERTICAL_CENTER]}
+          styleClasses={[CoreClasses.DISPLAY.FLEX, CoreClasses.ALIGNMENT.JUSTIFY_CONTENT_SPACE_BETWEEN, CoreClasses.ALIGNMENT.ALIGN_ITEMS_CENTER]}
         >
-          <CoreTypographyBody1>Notifications</CoreTypographyBody1>
+          <CoreTypographyBody1 styleClasses={[CoreClasses.MARGIN.MB0]}>Notifications</CoreTypographyBody1>
 
           <CoreIconButton disabled>
-            <CoreIcon>clear_all</CoreIcon>
+            <CoreIcon>more_horiz</CoreIcon>
           </CoreIconButton>
         </CoreBox>
 
-        <CoreDivider />
       </CoreBox>
 
+      <CoreDivider styleClasses={[CoreClasses.MARGIN.M0]}/>
+
       <CoreBox styleClasses={[CoreClasses.POPOVER.CONTENT]}>
         <CoreTypographyBody1
           sx={{ minWidth: "350px" }}

From a4885b60d5ac0a40267c13fcad9da9662142e812 Mon Sep 17 00:00:00 2001
From: arkaprota <arkaprota.ghosh@gmail.com>
Date: Mon, 23 Sep 2024 18:44:14 +0530
Subject: [PATCH 2/3] fix(global): :bug: change the coreicon more_horiz to
 clear_all which was previouly given

the coreicon more_horiz to clear_all which was there previouly was been required is solved

Ref: #373
---
 package/components/utils/NotificationPopOver.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/package/components/utils/NotificationPopOver.js b/package/components/utils/NotificationPopOver.js
index 02e2df32..60b6ab7e 100644
--- a/package/components/utils/NotificationPopOver.js
+++ b/package/components/utils/NotificationPopOver.js
@@ -18,7 +18,7 @@ export default function NotificationPopOver() {
           <CoreTypographyBody1 styleClasses={[CoreClasses.MARGIN.MB0]}>Notifications</CoreTypographyBody1>
 
           <CoreIconButton disabled>
-            <CoreIcon>more_horiz</CoreIcon>
+            <CoreIcon>clear_all</CoreIcon>
           </CoreIconButton>
         </CoreBox>
 

From 45a8b8989219226cda0cc4726924d5b2ed370d93 Mon Sep 17 00:00:00 2001
From: arkaprota <arkaprota.ghosh@gmail.com>
Date: Tue, 22 Oct 2024 13:46:02 +0530
Subject: [PATCH 3/3] fix(core): :bug: remove the sx

remove the sx property

Ref: #373
---
 package/components/utils/NotificationPopOver.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/package/components/utils/NotificationPopOver.js b/package/components/utils/NotificationPopOver.js
index 60b6ab7e..f515e358 100644
--- a/package/components/utils/NotificationPopOver.js
+++ b/package/components/utils/NotificationPopOver.js
@@ -28,7 +28,7 @@ export default function NotificationPopOver() {
 
       <CoreBox styleClasses={[CoreClasses.POPOVER.CONTENT]}>
         <CoreTypographyBody1
-          sx={{ minWidth: "350px" }}
+          minWidth={350}
           styleClasses={[CoreClasses.MARGIN.MY4, CoreClasses.PADDING.PX5]}
         >
           No new notifications