X-button Bug Fix (opensearch-project#1140)

* X-button Bug Fix Signed-off-by: Anand6Kumar <[email protected]> * moved constant to file Signed-off-by: Anand6Kumar <[email protected]> Signed-off-by: Anand6Kumar <[email protected]>
mengweieric · Oct 21, 2022 · 2e491f0 · 2e491f0
1 parent 5035712
commit 2e491f0
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/common/constants/explorer.ts b/common/constants/explorer.ts
@@ -273,3 +273,4 @@ export const SPAN = 'span';
 export const AGGREGATION_INFO = 'At least one metric is required to render a chart';
 export const DIMENSION_INFO = 'The timestamp type field can be selected as a first dimension only';
 export const TIME_FIELD = 'time_field';
+export const DISABLED_COLOUR = "#fafbfd";
diff --git a/.../visualizations/config_panel/config_panes/config_controls/logs_view_config_panel_item.tsx b/.../visualizations/config_panel/config_panes/config_controls/logs_view_config_panel_item.tsx
@@ -21,6 +21,7 @@ import {
   AVAILABLE_FIELDS,
   GROUPBY,
   SELECTED_FIELDS,
+  DISABLED_COLOUR
 } from '../../../../../../../../common/constants/explorer';
 import {
   ConfigList,
@@ -177,7 +178,7 @@ export const LogsViewConfigPanelItem = ({
           <EuiText size="xs">
             <EuiIcon
               type="cross"
-              color="danger"
+              color={queriedFields.length === 0 ? "danger" : DISABLED_COLOUR}
               onClick={() => handleServiceRemove(index, GROUPBY)}
             />
           </EuiText>