From 4e80ca73a0815d143ae7fb72931145320ae769ee Mon Sep 17 00:00:00 2001
From: SundasNoreen <sundas.noreen@arbisoft.com>
Date: Tue, 18 Jul 2023 13:27:13 +0500
Subject: [PATCH] fix: fixed height issue

---
 src/Notifications/index.jsx | 8 ++++++++
 src/index.scss              | 2 --
 2 files changed, 8 insertions(+), 2 deletions(-)

diff --git a/src/Notifications/index.jsx b/src/Notifications/index.jsx
index 31b3aa2..91a995c 100644
--- a/src/Notifications/index.jsx
+++ b/src/Notifications/index.jsx
@@ -44,6 +44,13 @@ const Notifications = () => {
     };
   }, []);
 
+  const viewPortHeight = window.innerHeight;
+  const headerHeight = document.getElementsByClassName('learning-header');
+  let notificationBarHeight = 0;
+  if (headerHeight.length > 0) {
+    notificationBarHeight = viewPortHeight - headerHeight[0].clientHeight;
+  }
+
   return (
     <OverlayTrigger
       trigger="click"
@@ -54,6 +61,7 @@ const Notifications = () => {
       overlay={(
         <Popover
           id="notificationTray"
+          style={{ height: `${notificationBarHeight}px` }}
           data-testid="notification-tray"
           className={classNames('overflow-auto rounded-0 border-0', {
             'w-100': !isOnMediumScreen && !isOnLargeScreen,
diff --git a/src/index.scss b/src/index.scss
index 46ed12d..76f491d 100644
--- a/src/index.scss
+++ b/src/index.scss
@@ -190,8 +190,6 @@ $white: #fff;
 }
 
 .popover {
-  max-height: calc(100% - 68px);
-  min-height: 1220px;
   filter: none;
   box-shadow: 0px 2px 4px rgba(0, 0, 0, 0.15), 0px 2px 8px rgba(0, 0, 0, 0.15);