Notification page changes

Signed-off-by: Sandeep Kumawat <[email protected]>
skumawat2025 · Aug 28, 2024 · 27b739a · 27b739a
1 parent fba0e6e
commit 27b739a
Show file tree

Hide file tree

Showing 2 changed files with 44 additions and 18 deletions.
diff --git a/public/pages/Notifications/containers/Notifications/Notifications.tsx b/public/pages/Notifications/containers/Notifications/Notifications.tsx
@@ -5,7 +5,17 @@
 
 import React, { ReactChild, useContext, useEffect, useRef, useState } from "react";
 import { unstable_batchedUpdates } from "react-dom";
-import { EuiSmallButton, EuiCallOut, EuiEmptyPrompt, EuiFlexGroup, EuiFlexItem, EuiPanel, EuiSpacer, EuiText, EuiTitle } from "@elastic/eui";
+import {
+  EuiSmallButton,
+  EuiCallOut,
+  EuiEmptyPrompt,
+  EuiFlexGroup,
+  EuiFlexItem,
+  EuiPanel,
+  EuiSpacer,
+  EuiText,
+  EuiTitle,
+} from "@elastic/eui";
 import { get } from "lodash";
 import { CoreStart } from "opensearch-dashboards/public";
 import useField from "../../../../lib/field";
@@ -34,7 +44,7 @@ import { checkPermissionForSubmitLRONConfig } from "../../../../containers/Notif
 import "./index.scss";
 import { DataSourceMenuContext } from "../../../../services/DataSourceMenuContext";
 import { getApplication, getNavigationUI, getUISettings } from "../../../../services/Services";
-import { TopNavControlButtonData } from "../../../../../../../src/plugins/navigation/public";
+import { TopNavControlButtonData, TopNavControlDescriptionData } from "../../../../../../../src/plugins/navigation/public";
 
 export interface NotificationsProps {}
 
@@ -166,13 +176,9 @@ const Notifications = (props: NotificationsProps) => {
 
   const descriptionData = [
     {
-      renderComponent: (
-        <EuiText size="s" color="subdued">
-          Configure the default notification settings on index operation statuses, such as failed or completed. You can configure <br></br>
-          additional notification settings while performing an index operation.
-        </EuiText>
-      ),
-    },
+      description:
+        "Configure the default notification settings on index operation statuses, such as failed or completed. You can configure additional notification settings while performing an index operation.",
+    } as TopNavControlDescriptionData,
   ];
 
   const controlsData = [
@@ -287,7 +293,11 @@ const Notifications = (props: NotificationsProps) => {
             });
             return (
               <CustomFormRow
-                label={<div className="ISM-notifications-first-letter-uppercase">{record.title}</div>}
+                label={
+                  <EuiText size="s">
+                    <h3 className="ISM-notifications-first-letter-uppercase">{record.title}</h3>
+                  </EuiText>
+                }
                 helpText={ActionTypeMapDescription[getKeyByValue(ActionTypeMapTitle, record.title) as ActionType]}
                 direction="hoz"
                 key={record.action_name}

diff --git a/...ic/pages/Notifications/containers/Notifications/__snapshots__/Notifications.test.tsx.snap b/...ic/pages/Notifications/containers/Notifications/__snapshots__/Notifications.test.tsx.snap
@@ -170,9 +170,13 @@ exports[`<Notifications /> spec renders 1`] = `
             class="euiTitle euiTitle--xsmall euiDescribedFormGroup__title"
           >
             <div
-              class="ISM-notifications-first-letter-uppercase"
+              class="euiText euiText--small"
             >
-              reindex
+              <h3
+                class="ISM-notifications-first-letter-uppercase"
+              >
+                reindex
+              </h3>
             </div>
           </div>
           <div
@@ -292,9 +296,13 @@ exports[`<Notifications /> spec renders 1`] = `
             class="euiTitle euiTitle--xsmall euiDescribedFormGroup__title"
           >
             <div
-              class="ISM-notifications-first-letter-uppercase"
+              class="euiText euiText--small"
             >
-              shrink, split, clone
+              <h3
+                class="ISM-notifications-first-letter-uppercase"
+              >
+                shrink, split, clone
+              </h3>
             </div>
           </div>
           <div
@@ -414,9 +422,13 @@ exports[`<Notifications /> spec renders 1`] = `
             class="euiTitle euiTitle--xsmall euiDescribedFormGroup__title"
           >
             <div
-              class="ISM-notifications-first-letter-uppercase"
+              class="euiText euiText--small"
             >
-              force merge
+              <h3
+                class="ISM-notifications-first-letter-uppercase"
+              >
+                force merge
+              </h3>
             </div>
           </div>
           <div
@@ -536,9 +548,13 @@ exports[`<Notifications /> spec renders 1`] = `
             class="euiTitle euiTitle--xsmall euiDescribedFormGroup__title"
           >
             <div
-              class="ISM-notifications-first-letter-uppercase"
+              class="euiText euiText--small"
             >
-              open
+              <h3
+                class="ISM-notifications-first-letter-uppercase"
+              >
+                open
+              </h3>
             </div>
           </div>
           <div