Fit and Finish Changes for DataStreams and Rollups

Signed-off-by: Sandeep Kumawat <[email protected]>
opensearch-project · Aug 28, 2024 · 167797d · 167797d
1 parent 620551b
commit 167797d
Show file tree

Hide file tree

Showing 59 changed files with 949 additions and 672 deletions.
diff --git a/public/components/ContentPanel/ContentPanel.tsx b/public/components/ContentPanel/ContentPanel.tsx
@@ -64,7 +64,7 @@ const ContentPanel: React.SFC<ContentPanelProps> = ({
   const isGhost = color === "ghost";
   const titleContent = (
     <EuiFlexGroup
-      style={{ ...(noExtraPadding ? { marginTop: 0, marginBottom: 0 } : {}), padding: isGhost || accordion ? undefined : "0px 10px" }}
+      style={{ ...(noExtraPadding ? { marginTop: 0, marginBottom: 0 } : {}), padding: isGhost || accordion ? undefined : "0px 16px" }}
       justifyContent="spaceBetween"
       alignItems="flexStart"
     >
@@ -111,7 +111,7 @@ const ContentPanel: React.SFC<ContentPanelProps> = ({
       {children ? (
         <div
           style={{
-            padding: isGhost ? undefined : "0px 10px",
+            padding: isGhost ? undefined : "0px 16px",
             ...bodyStyles,
             display: accordion && isAccordionOpen === "closed" ? "none" : undefined,
           }}

diff --git a/public/components/ContentPanel/__snapshots__/ContentPanel.test.tsx.snap b/public/components/ContentPanel/__snapshots__/ContentPanel.test.tsx.snap
@@ -7,7 +7,7 @@ exports[`<ContentPanel /> spec renders the component 1`] = `
 >
   <div
     class="euiFlexGroup euiFlexGroup--gutterLarge euiFlexGroup--alignItemsFlexStart euiFlexGroup--justifyContentSpaceBetween euiFlexGroup--directionRow euiFlexGroup--responsive"
-    style="padding: 0px 10px;"
+    style="padding: 0px 16px;"
   >
     <div
       class="euiFlexItem"
@@ -51,7 +51,7 @@ exports[`<ContentPanel /> spec renders the component 1`] = `
     class="euiHorizontalRule euiHorizontalRule--full euiHorizontalRule--marginXSmall"
   />
   <div
-    style="padding: 0px 10px;"
+    style="padding: 0px 16px;"
   >
     <div>
       Testing ContentPanel

diff --git a/public/components/DescriptionListHoz/DescriptionListHoz.tsx b/public/components/DescriptionListHoz/DescriptionListHoz.tsx
@@ -13,6 +13,7 @@ const DisplayItem = (
   const { listItem, ...others } = props;
   return (
     <EuiDescriptionList
+      compressed={true}
       listItems={[
         {
           ...listItem,

diff --git a/public/components/DescriptionListHoz/__snapshots__/DescriptionListHoz.test.tsx.snap b/public/components/DescriptionListHoz/__snapshots__/DescriptionListHoz.test.tsx.snap
@@ -9,7 +9,7 @@ exports[`<DescriptionListHoz /> spec renders the component 1`] = `
       class="euiFlexItem"
     >
       <dl
-        class="euiDescriptionList euiDescriptionList--row"
+        class="euiDescriptionList euiDescriptionList--row euiDescriptionList--compressed"
       >
         <dt
           class="euiDescriptionList__title"

diff --git a/public/components/IndexDetail/__snapshots__/IndexDetail.test.tsx.snap b/public/components/IndexDetail/__snapshots__/IndexDetail.test.tsx.snap
@@ -7,7 +7,7 @@ exports[`<IndexDetail /> spec renders the component 1`] = `
 >
   <div
     class="euiFlexGroup euiFlexGroup--gutterLarge euiFlexGroup--alignItemsFlexStart euiFlexGroup--justifyContentSpaceBetween euiFlexGroup--directionRow euiFlexGroup--responsive"
-    style="padding: 0px 10px;"
+    style="padding: 0px 16px;"
   >
     <div
       class="euiFlexItem"
@@ -29,7 +29,7 @@ exports[`<IndexDetail /> spec renders the component 1`] = `
     class="euiHorizontalRule euiHorizontalRule--full euiHorizontalRule--marginXSmall"
   />
   <div
-    style="padding: 0px 10px;"
+    style="padding: 0px 16px;"
   >
     <div
       class="euiForm"

diff --git a/public/components/IndexMapping/IndexMapping.tsx b/public/components/IndexMapping/IndexMapping.tsx
@@ -14,6 +14,7 @@ import {
   EuiButtonGroup,
   EuiLink,
   EuiCallOut,
+  EuiText,
 } from "@elastic/eui";
 import { set, get, isEmpty } from "lodash";
 import MonacoJSONEditor, { IJSONEditorRef } from "../MonacoJSONEditor";
@@ -199,7 +200,9 @@ const IndexMapping = (
               items={transformValueToTreeItems(value)}
             />
           ) : (
-            <p>You have no field mappings.</p>
+            <EuiText size="s">
+              <p>You have no field mappings.</p>
+            </EuiText>
           )}
           {readonly ? null : (
             <>

diff --git a/public/containers/ErrorNotification/__snapshots__/ErrorNotification.test.tsx.snap b/public/containers/ErrorNotification/__snapshots__/ErrorNotification.test.tsx.snap
@@ -7,7 +7,7 @@ exports[`<ErrorNotification /> spec renders the component 1`] = `
 >
   <div
     class="euiFlexGroup euiFlexGroup--gutterLarge euiFlexGroup--alignItemsFlexStart euiFlexGroup--justifyContentSpaceBetween euiFlexGroup--directionRow euiFlexGroup--responsive"
-    style="padding: 0px 10px;"
+    style="padding: 0px 16px;"
   >
     <div
       class="euiFlexItem"

diff --git a/public/containers/IndexDetail/__snapshots__/IndexDetail.test.tsx.snap b/public/containers/IndexDetail/__snapshots__/IndexDetail.test.tsx.snap
@@ -8,7 +8,7 @@ exports[`<IndexDetail /> spec render the component 1`] = `
   >
     <div
       class="euiFlexGroup euiFlexGroup--gutterLarge euiFlexGroup--alignItemsFlexStart euiFlexGroup--justifyContentSpaceBetween euiFlexGroup--directionRow euiFlexGroup--responsive"
-      style="padding: 0px 10px;"
+      style="padding: 0px 16px;"
     >
       <div
         class="euiFlexItem"
@@ -30,7 +30,7 @@ exports[`<IndexDetail /> spec render the component 1`] = `
       class="euiHorizontalRule euiHorizontalRule--full euiHorizontalRule--marginXSmall"
     />
     <div
-      style="padding: 0px 10px;"
+      style="padding: 0px 16px;"
     >
       <div
         class="euiSpacer euiSpacer--l"

diff --git a/public/containers/IndexForm/__snapshots__/IndexForm.test.tsx.snap b/public/containers/IndexForm/__snapshots__/IndexForm.test.tsx.snap
@@ -8,7 +8,7 @@ exports[`<IndexForm /> spec render page 1`] = `
   >
     <div
       class="euiFlexGroup euiFlexGroup--gutterLarge euiFlexGroup--alignItemsFlexStart euiFlexGroup--justifyContentSpaceBetween euiFlexGroup--directionRow euiFlexGroup--responsive"
-      style="padding: 0px 10px;"
+      style="padding: 0px 16px;"
     >
       <div
         class="euiFlexItem"
@@ -30,7 +30,7 @@ exports[`<IndexForm /> spec render page 1`] = `
       class="euiHorizontalRule euiHorizontalRule--full euiHorizontalRule--marginXSmall"
     />
     <div
-      style="padding: 0px 10px;"
+      style="padding: 0px 16px;"
     >
       <div
         class="euiForm"
@@ -190,7 +190,7 @@ exports[`<IndexForm /> spec render page 1`] = `
   >
     <div
       class="euiFlexGroup euiFlexGroup--gutterLarge euiFlexGroup--alignItemsFlexStart euiFlexGroup--justifyContentSpaceBetween euiFlexGroup--directionRow euiFlexGroup--responsive"
-      style="padding: 0px 10px;"
+      style="padding: 0px 16px;"
     >
       <div
         class="euiFlexItem"
@@ -212,7 +212,7 @@ exports[`<IndexForm /> spec render page 1`] = `
       class="euiHorizontalRule euiHorizontalRule--full euiHorizontalRule--marginXSmall"
     />
     <div
-      style="padding: 0px 10px;"
+      style="padding: 0px 16px;"
     >
       <div
         class="euiForm"
@@ -598,7 +598,7 @@ exports[`<IndexForm /> spec render page 1`] = `
   >
     <div
       class="euiFlexGroup euiFlexGroup--gutterLarge euiFlexGroup--alignItemsFlexStart euiFlexGroup--justifyContentSpaceBetween euiFlexGroup--directionRow euiFlexGroup--responsive"
-      style="padding: 0px 10px;"
+      style="padding: 0px 16px;"
     >
       <div
         class="euiFlexItem"
@@ -652,7 +652,7 @@ exports[`<IndexForm /> spec render page 1`] = `
       class="euiHorizontalRule euiHorizontalRule--full euiHorizontalRule--marginXSmall"
     />
     <div
-      style="padding: 0px 10px;"
+      style="padding: 0px 16px;"
     >
       <div
         class="euiFormRow euiFormRow--fullWidth euiFormRow--compressed"
@@ -728,9 +728,13 @@ exports[`<IndexForm /> spec render page 1`] = `
           <div
             class="euiSpacer euiSpacer--l"
           />
-          <p>
-            You have no field mappings.
-          </p>
+          <div
+            class="euiText euiText--small"
+          >
+            <p>
+              You have no field mappings.
+            </p>
+          </div>
           <div
             class="euiSpacer euiSpacer--l"
           />

diff --git a/public/containers/NotificationConfig/NotificationConfig.tsx b/public/containers/NotificationConfig/NotificationConfig.tsx
@@ -4,7 +4,7 @@
  */
 
 import React, { useRef, forwardRef, useMemo, useImperativeHandle, useContext, useEffect, useState } from "react";
-import { EuiBadge, EuiSmallButton, EuiFlexGroup, EuiFlexItem, EuiSpacer, EuiTitle } from "@elastic/eui";
+import { EuiBadge, EuiSmallButton, EuiFlexGroup, EuiFlexItem, EuiSpacer, EuiText } from "@elastic/eui";
 import ChannelSelect, { useChannels } from "../ChannelSelect";
 import { AllBuiltInComponents } from "../../components/FormGenerator";
 import {
@@ -122,9 +122,9 @@ const NotificationConfig = (
 
   const content = (
     <div>
-      <EuiTitle size="s">
-        <h5>Notifications</h5>
-      </EuiTitle>
+      <EuiText size="s">
+        <h3>Notifications</h3>
+      </EuiText>
       <NotificationCallout
         actionType={actionType}
         operationType={operationType}

diff --git a/public/containers/NotificationConfig/__snapshots__/NotificationConfig.test.tsx.snap b/public/containers/NotificationConfig/__snapshots__/NotificationConfig.test.tsx.snap
@@ -6,11 +6,13 @@ exports[`<ChannelNotification /> spec renders with create permission and no defa
     class="euiSpacer euiSpacer--l"
   />
   <div>
-    <h5
-      class="euiTitle euiTitle--small"
+    <div
+      class="euiText euiText--small"
     >
-      Notifications
-    </h5>
+      <h3>
+        Notifications
+      </h3>
+    </div>
     <div
       class="euiSpacer euiSpacer--s"
     />
@@ -111,11 +113,13 @@ exports[`<ChannelNotification /> spec renders with full permission and default n
     class="euiSpacer euiSpacer--l"
   />
   <div>
-    <h5
-      class="euiTitle euiTitle--small"
+    <div
+      class="euiText euiText--small"
     >
-      Notifications
-    </h5>
+      <h3>
+        Notifications
+      </h3>
+    </div>
     <div
       class="euiSpacer euiSpacer--s"
     />
@@ -287,11 +291,13 @@ exports[`<ChannelNotification /> spec renders with full permission and no defaul
     class="euiSpacer euiSpacer--l"
   />
   <div>
-    <h5
-      class="euiTitle euiTitle--small"
+    <div
+      class="euiText euiText--small"
     >
-      Notifications
-    </h5>
+      <h3>
+        Notifications
+      </h3>
+    </div>
     <div
       class="euiSpacer euiSpacer--s"
     />
@@ -407,11 +413,13 @@ exports[`<ChannelNotification /> spec renders with view permission and default n
     class="euiSpacer euiSpacer--l"
   />
   <div>
-    <h5
-      class="euiTitle euiTitle--small"
+    <div
+      class="euiText euiText--small"
     >
-      Notifications
-    </h5>
+      <h3>
+        Notifications
+      </h3>
+    </div>
     <div
       class="euiSpacer euiSpacer--s"
     />

diff --git a/public/pages/Aliases/containers/Aliases/__snapshots__/Aliases.test.tsx.snap b/public/pages/Aliases/containers/Aliases/__snapshots__/Aliases.test.tsx.snap
@@ -7,7 +7,7 @@ exports[`<Aliases /> spec renders the component 1`] = `
 >
   <div
     class="euiFlexGroup euiFlexGroup--gutterLarge euiFlexGroup--alignItemsFlexStart euiFlexGroup--justifyContentSpaceBetween euiFlexGroup--directionRow euiFlexGroup--responsive"
-    style="padding: 0px 10px;"
+    style="padding: 0px 16px;"
   >
     <div
       class="euiFlexItem"

diff --git a/...gePolicy/components/ChangeManagedIndices/__snapshots__/ChangeManagedIndices.test.tsx.snap b/...gePolicy/components/ChangeManagedIndices/__snapshots__/ChangeManagedIndices.test.tsx.snap
@@ -7,7 +7,7 @@ exports[`<ChangeManagedIndices /> spec renders the component 1`] = `
 >
   <div
     class="euiFlexGroup euiFlexGroup--gutterLarge euiFlexGroup--alignItemsFlexStart euiFlexGroup--justifyContentSpaceBetween euiFlexGroup--directionRow euiFlexGroup--responsive"
-    style="padding: 0px 10px;"
+    style="padding: 0px 16px;"
   >
     <div
       class="euiFlexItem"

diff --git a/public/pages/ChangePolicy/components/NewPolicy/__snapshots__/NewPolicy.test.tsx.snap b/public/pages/ChangePolicy/components/NewPolicy/__snapshots__/NewPolicy.test.tsx.snap
@@ -7,7 +7,7 @@ exports[`<NewPolicy /> spec renders the component 1`] = `
 >
   <div
     class="euiFlexGroup euiFlexGroup--gutterLarge euiFlexGroup--alignItemsFlexStart euiFlexGroup--justifyContentSpaceBetween euiFlexGroup--directionRow euiFlexGroup--responsive"
-    style="padding: 0px 10px;"
+    style="padding: 0px 16px;"
   >
     <div
       class="euiFlexItem"

diff --git a/public/pages/ChangePolicy/containers/ChangePolicy/__snapshots__/ChangePolicy.test.tsx.snap b/public/pages/ChangePolicy/containers/ChangePolicy/__snapshots__/ChangePolicy.test.tsx.snap
@@ -18,7 +18,7 @@ exports[`<ChangePolicy /> spec renders the component 1`] = `
   >
     <div
       class="euiFlexGroup euiFlexGroup--gutterLarge euiFlexGroup--alignItemsFlexStart euiFlexGroup--justifyContentSpaceBetween euiFlexGroup--directionRow euiFlexGroup--responsive"
-      style="padding: 0px 10px;"
+      style="padding: 0px 16px;"
     >
       <div
         class="euiFlexItem"
@@ -218,7 +218,7 @@ exports[`<ChangePolicy /> spec renders the component 1`] = `
   >
     <div
       class="euiFlexGroup euiFlexGroup--gutterLarge euiFlexGroup--alignItemsFlexStart euiFlexGroup--justifyContentSpaceBetween euiFlexGroup--directionRow euiFlexGroup--responsive"
-      style="padding: 0px 10px;"
+      style="padding: 0px 16px;"
     >
       <div
         class="euiFlexItem"

diff --git a/...eTemplates/containers/ComposableTemplates/__snapshots__/ComposableTemplates.test.tsx.snap b/...eTemplates/containers/ComposableTemplates/__snapshots__/ComposableTemplates.test.tsx.snap
@@ -7,7 +7,7 @@ exports[`<ComposableTemplates /> spec renders the component 1`] = `
 >
   <div
     class="euiFlexGroup euiFlexGroup--gutterLarge euiFlexGroup--alignItemsFlexStart euiFlexGroup--justifyContentSpaceBetween euiFlexGroup--directionRow euiFlexGroup--responsive"
-    style="padding: 0px 10px;"
+    style="padding: 0px 16px;"
   >
     <div
       class="euiFlexItem"

diff --git a/.../containers/CreateComposableTemplate/__snapshots__/CreateComposableTemplate.test.tsx.snap b/.../containers/CreateComposableTemplate/__snapshots__/CreateComposableTemplate.test.tsx.snap
@@ -63,7 +63,7 @@ exports[`<CreateComposableTemplate /> spec it goes to templates page when click
     >
       <div
         class="euiFlexGroup euiFlexGroup--gutterLarge euiFlexGroup--alignItemsFlexStart euiFlexGroup--justifyContentSpaceBetween euiFlexGroup--directionRow euiFlexGroup--responsive"
-        style="padding: 0px 10px;"
+        style="padding: 0px 16px;"
       >
         <div
           class="euiFlexItem"
@@ -85,7 +85,7 @@ exports[`<CreateComposableTemplate /> spec it goes to templates page when click
         class="euiHorizontalRule euiHorizontalRule--full euiHorizontalRule--marginXSmall"
       />
       <div
-        style="padding: 0px 10px;"
+        style="padding: 0px 16px;"
       >
         <div
           class="euiSpacer euiSpacer--s"
@@ -166,7 +166,7 @@ exports[`<CreateComposableTemplate /> spec it goes to templates page when click
     >
       <div
         class="euiFlexGroup euiFlexGroup--gutterLarge euiFlexGroup--alignItemsFlexStart euiFlexGroup--justifyContentSpaceBetween euiFlexGroup--directionRow euiFlexGroup--responsive"
-        style="margin-top: 0px; margin-bottom: 0px; padding: 0px 10px;"
+        style="margin-top: 0px; margin-bottom: 0px; padding: 0px 16px;"
       >
         <div
           class="euiFlexItem"
@@ -264,7 +264,7 @@ exports[`<CreateComposableTemplate /> spec it goes to templates page when click
     >
       <div
         class="euiFlexGroup euiFlexGroup--gutterLarge euiFlexGroup--alignItemsFlexStart euiFlexGroup--justifyContentSpaceBetween euiFlexGroup--directionRow euiFlexGroup--responsive"
-        style="margin-top: 0px; margin-bottom: 0px; padding: 0px 10px;"
+        style="margin-top: 0px; margin-bottom: 0px; padding: 0px 16px;"
       >
         <div
           class="euiFlexItem"
@@ -344,7 +344,7 @@ exports[`<CreateComposableTemplate /> spec it goes to templates page when click
     >
       <div
         class="euiFlexGroup euiFlexGroup--gutterLarge euiFlexGroup--alignItemsFlexStart euiFlexGroup--justifyContentSpaceBetween euiFlexGroup--directionRow euiFlexGroup--responsive"
-        style="margin-top: 0px; margin-bottom: 0px; padding: 0px 10px;"
+        style="margin-top: 0px; margin-bottom: 0px; padding: 0px 16px;"
       >
         <div
           class="euiFlexItem"