From b5d88a621fe62aea619974ace0e4a5d273632c70 Mon Sep 17 00:00:00 2001
From: sundasnoreen12 <sundasnoreen12@gmail.com>
Date: Mon, 24 Jun 2024 13:24:41 +0500
Subject: [PATCH] fix: fix loading and margin issues

---
 src/notification-preferences/NotificationPreferenceApp.jsx | 2 +-
 src/notification-preferences/NotificationPreferences.jsx   | 4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/notification-preferences/NotificationPreferenceApp.jsx b/src/notification-preferences/NotificationPreferenceApp.jsx
index 2d6b9a12a..2c42862dd 100644
--- a/src/notification-preferences/NotificationPreferenceApp.jsx
+++ b/src/notification-preferences/NotificationPreferenceApp.jsx
@@ -35,7 +35,7 @@ const NotificationPreferenceApp = ({ appId }) => {
     >
       <Collapsible.Trigger>
         <div className="d-flex align-items-center">
-          <span className="mr-auto preference-app font-weight-bold">
+          <span className={classNames("mr-auto preference-app font-weight-bold", { 'mb-2': !mobileView })}>
             {intl.formatMessage(messages.notificationAppTitle, { key: appId })}
           </span>
         </div>
diff --git a/src/notification-preferences/NotificationPreferences.jsx b/src/notification-preferences/NotificationPreferences.jsx
index da83f352e..b7b5018ad 100644
--- a/src/notification-preferences/NotificationPreferences.jsx
+++ b/src/notification-preferences/NotificationPreferences.jsx
@@ -82,7 +82,7 @@ const NotificationPreferences = () => {
             {course?.name}
           </span>
         </div>
-        {!mobileView && (
+        {!mobileView && !isLoading && (
         <div className="d-flex flex-row justify-content-between float-right">
           <div className="d-flex">
             {Object.values(NOTIFICATION_CHANNELS).map((channel) => (
@@ -90,7 +90,7 @@ const NotificationPreferences = () => {
                 <NavItem
                   id={channel}
                   key={channel}
-                  className={classNames('mb-3 header-label column-padding', {
+                  className={classNames('header-label column-padding', {
                     'pr-0': channel === NOTIFICATION_CHANNELS[NOTIFICATION_CHANNELS.length - 1],
                     'mr-2': channel === 'web',
                     'email-channel ': channel === 'email',