added fix for loading spinner issue for other database (#189) (#192)

* added fix for loading spinner issue * fix oui scroll issue * added button next to the table for query to be added * icon changed for wuery loader --------- (cherry picked from commit 5a956ba) Signed-off-by: sumukhswamy <[email protected]> Signed-off-by: Shenoy Pratik <[email protected]> Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: Shenoy Pratik <[email protected]>
opensearch-project · Oct 30, 2023 · 94805c5 · 94805c5
1 parent 4187b6d
commit 94805c5
Show file tree

Hide file tree

Showing 6 changed files with 172 additions and 177 deletions.
diff --git a/public/components/Main/__snapshots__/main.test.tsx.snap b/public/components/Main/__snapshots__/main.test.tsx.snap
@@ -237,7 +237,7 @@ exports[`<Main /> spec click clear button 1`] = `
             class="euiFlexItem euiFlexItem--flexGrowZero"
           >
             <div
-              class="euiFlexItem euiFlexItem--flexGrowZero"
+              class="euiFlexItem euiFlexItem--flexGrowZero workbench-tree"
             >
               <div
                 class="euiText euiText--medium euiTreeView__wrapper"
@@ -2904,7 +2904,7 @@ exports[`<Main /> spec click run button, and response is not ok 1`] = `
             class="euiFlexItem euiFlexItem--flexGrowZero"
           >
             <div
-              class="euiFlexItem euiFlexItem--flexGrowZero"
+              class="euiFlexItem euiFlexItem--flexGrowZero workbench-tree"
             >
               <div
                 class="euiText euiText--medium euiTreeView__wrapper"
@@ -5008,7 +5008,7 @@ exports[`<Main /> spec click run button, and response is ok 1`] = `
             class="euiFlexItem euiFlexItem--flexGrowZero"
           >
             <div
-              class="euiFlexItem euiFlexItem--flexGrowZero"
+              class="euiFlexItem euiFlexItem--flexGrowZero workbench-tree"
             >
               <div
                 class="euiText euiText--medium euiTreeView__wrapper"
@@ -5642,7 +5642,7 @@ exports[`<Main /> spec click run button, response fills null and missing values
             class="euiFlexItem euiFlexItem--flexGrowZero"
           >
             <div
-              class="euiFlexItem euiFlexItem--flexGrowZero"
+              class="euiFlexItem euiFlexItem--flexGrowZero workbench-tree"
             >
               <div
                 class="euiText euiText--medium euiTreeView__wrapper"
@@ -6275,7 +6275,7 @@ exports[`<Main /> spec click translation button, and response is ok 1`] = `
             class="euiFlexItem euiFlexItem--flexGrowZero"
           >
             <div
-              class="euiFlexItem euiFlexItem--flexGrowZero"
+              class="euiFlexItem euiFlexItem--flexGrowZero workbench-tree"
             >
               <div
                 class="euiText euiText--medium euiTreeView__wrapper"
@@ -8366,7 +8366,7 @@ exports[`<Main /> spec renders the component 1`] = `
             class="euiFlexItem euiFlexItem--flexGrowZero"
           >
             <div
-              class="euiFlexItem euiFlexItem--flexGrowZero"
+              class="euiFlexItem euiFlexItem--flexGrowZero workbench-tree"
             >
               <div
                 class="euiText euiText--medium euiTreeView__wrapper"

diff --git a/public/components/SQLPage/__tests__/__snapshots__/table_view.test.tsx.snap b/public/components/SQLPage/__tests__/__snapshots__/table_view.test.tsx.snap
@@ -70,7 +70,7 @@ exports[`Render databases in tree fetches and displays database nodes when datas
 exports[`Render databases in tree fetches and displays indicies when datasource is OpenSearch 1`] = `
 <div>
   <div
-    class="euiFlexItem euiFlexItem--flexGrowZero"
+    class="euiFlexItem euiFlexItem--flexGrowZero workbench-tree"
   >
     <div
       class="euiText euiText--medium euiTreeView__wrapper"

diff --git a/public/components/SQLPage/table_view.scss b/public/components/SQLPage/table_view.scss
@@ -0,0 +1,10 @@
+/*
+ * Copyright OpenSearch Contributors
+ * SPDX-License-Identifier: Apache-2.0
+ */
+
+.workbench-tree {
+  .euiTreeView__node--expanded {
+    max-height: none !important; // Fixes OUI issue: https://github.com/opensearch-project/oui/issues/1118
+  }
+}