Description changes

Signed-off-by: Kshitij Tandon <tandonks@amazon.com>
opensearch-project · Aug 22, 2024 · 8acd6c7 · 8acd6c7
1 parent 780a936
commit 8acd6c7
Show file tree

Hide file tree

Showing 3 changed files with 35 additions and 59 deletions.
diff --git a/public/pages/Reindex/container/Reindex/Reindex.tsx b/public/pages/Reindex/container/Reindex/Reindex.tsx
@@ -43,6 +43,7 @@ import NotificationConfig, { NotificationConfigRef } from "../../../../container
 import { ActionType } from "../../../Notifications/constant";
 import { useUpdateUrlWithDataSourceProperties } from "../../../../components/MDSEnabledComponent";
 import { getApplication, getNavigationUI, getUISettings } from "../../../../services/Services";
+import { TopNavControlDescriptionData, TopNavControlLinkData } from "src/plugins/navigation/public/top_nav_menu/top_nav_control_data";
 
 interface ReindexProps extends RouteComponentProps {
   commonService: CommonService;
@@ -555,25 +556,17 @@ class Reindex extends Component<ReindexProps, ReindexState> {
       </CustomFormRow>
     );
 
-    const description = [
+    const descriptionData = [
       {
-        renderComponent: (
-          <CustomFormRow
-            fullWidth
-            label=""
-            helpText={
-              <div>
-                Use reindex to make extensive changes to your index. Reindex will copy data of the source index into another index.{" "}
-                <EuiLink href={this.context.docLinks.links.opensearch.reindexData.base} target="_blank" rel="noopener noreferrer">
-                  Learn more
-                </EuiLink>
-              </div>
-            }
-          >
-            <></>
-          </CustomFormRow>
-        ),
-      },
+        description: "Shrink an existing index into a new index with fewer primary shards.",
+        links: {
+          label: "Learn more",
+          href: this.context.docLinks.links.opensearch.reindexData.base,
+          iconType: "popout",
+          iconSide: "right",
+          controlType: "link",
+        } as TopNavControlLinkData,
+      } as TopNavControlDescriptionData,
     ];
 
     // expand data streams and aliases
@@ -737,7 +730,7 @@ class Reindex extends Component<ReindexProps, ReindexState> {
 
     return this.props.useUpdatedUX ? (
       <div>
-        <HeaderControl controls={description} setMountPoint={setAppDescriptionControls} />
+        <HeaderControl controls={descriptionData} setMountPoint={setAppDescriptionControls} />
         {Common()}
       </div>
     ) : (

diff --git a/public/pages/ShrinkIndex/container/ShrinkIndex/ShrinkIndex.tsx b/public/pages/ShrinkIndex/container/ShrinkIndex/ShrinkIndex.tsx
@@ -649,24 +649,17 @@ class ShrinkIndex extends Component<ShrinkIndexProps, ShrinkIndexState> {
       </EuiCompressedFormRow>
     );
 
-    const description = [
+    const descriptionData = [
       {
-        renderComponent: (
-          <EuiCompressedFormRow
-            fullWidth
-            helpText={
-              <div>
-                Shrink an existing index into a new index with fewer primary shards.&nbsp;
-                <EuiLink href={SHRINK_DOCUMENTATION_URL} target="_blank" rel="noopener noreferrer">
-                  Learn more
-                </EuiLink>
-              </div>
-            }
-          >
-            <></>
-          </EuiCompressedFormRow>
-        ),
-      },
+        description: "Shrink an existing index into a new index with fewer primary shards.",
+        links: {
+          label: "Learn more",
+          href: SHRINK_DOCUMENTATION_URL,
+          iconType: "popout",
+          iconSide: "right",
+          controlType: "link",
+        } as TopNavControlLinkData,
+      } as TopNavControlDescriptionData,
     ];
 
     const Common = () => {
@@ -714,7 +707,7 @@ class ShrinkIndex extends Component<ShrinkIndexProps, ShrinkIndexState> {
 
     return this.props.useUpdatedUX ? (
       <div style={{ padding: "0px 0px" }}>
-        <HeaderControl controls={description} setMountPoint={setAppDescriptionControls} />
+        <HeaderControl controls={descriptionData} setMountPoint={setAppDescriptionControls} />
 
         {Common()}
       </div>

diff --git a/public/pages/SplitIndex/container/SplitIndex/SplitIndex.tsx b/public/pages/SplitIndex/container/SplitIndex/SplitIndex.tsx
@@ -30,6 +30,7 @@ import { EVENT_MAP, destroyListener, listenEvent } from "../../../../JobHandler"
 import { ServerResponse } from "../../../../../server/models/types";
 import { useUpdateUrlWithDataSourceProperties } from "../../../../components/MDSEnabledComponent";
 import { getApplication, getNavigationUI, getUISettings } from "../../../../services/Services";
+import { TopNavControlDescriptionData, TopNavControlLinkData } from "src/plugins/navigation/public/top_nav_menu/top_nav_control_data";
 
 interface SplitIndexProps extends RouteComponentProps {
   commonService: CommonService;
@@ -228,28 +229,17 @@ export class SplitIndex extends Component<SplitIndexProps> {
     const { HeaderControl } = getNavigationUI();
     const { setAppDescriptionControls } = getApplication();
 
-    const description = [
+    const descriptionData = [
       {
-        renderComponent: (
-          <EuiCompressedFormRow
-            fullWidth
-            helpText={
-              <div style={{ width: "100%" }}>
-                Split an existing read-only index into a new index with more primary shards.&nbsp;
-                <EuiLink
-                  href={"https://opensearch.org/docs/latest/api-reference/index-apis/split/"}
-                  target="_blank"
-                  rel="noopener noreferrer"
-                >
-                  Learn more
-                </EuiLink>
-              </div>
-            }
-          >
-            <></>
-          </EuiCompressedFormRow>
-        ),
-      },
+        description: "Shrink an existing index into a new index with fewer primary shards.",
+        links: {
+          label: "Learn more",
+          href: "https://opensearch.org/docs/latest/api-reference/index-apis/split/",
+          iconType: "popout",
+          iconSide: "right",
+          controlType: "link",
+        } as TopNavControlLinkData,
+      } as TopNavControlDescriptionData,
     ];
 
     const Common = () => {
@@ -278,7 +268,7 @@ export class SplitIndex extends Component<SplitIndexProps> {
 
     return this.props.useUpdatedUX ? (
       <div style={{ padding: "0px" }}>
-        <HeaderControl controls={description} setMountPoint={setAppDescriptionControls} />
+        <HeaderControl controls={descriptionData} setMountPoint={setAppDescriptionControls} />
         {Common()}
       </div>
     ) : (