From e5dc6270f84c6941beb596d4562c39d516d3f53c Mon Sep 17 00:00:00 2001
From: Stratoula Kalafateli <efstratia.kalafateli@elastic.co>
Date: Mon, 29 Jul 2024 10:55:51 +0200
Subject: [PATCH] Address Andrea's comments

---
 src/plugins/esql_datagrid/public/data_grid.tsx              | 5 +++--
 .../shared/edit_on_the_fly/lens_configuration_flyout.tsx    | 1 +
 .../trigger_actions/open_lens_config/edit_action_helpers.ts | 6 ++++++
 3 files changed, 10 insertions(+), 2 deletions(-)

diff --git a/src/plugins/esql_datagrid/public/data_grid.tsx b/src/plugins/esql_datagrid/public/data_grid.tsx
index 872b742eaa0843..9454f1ddd0ba1a 100644
--- a/src/plugins/esql_datagrid/public/data_grid.tsx
+++ b/src/plugins/esql_datagrid/public/data_grid.tsx
@@ -159,11 +159,12 @@ const DataGrid: React.FC<ESQLDataGridProps> = (props) => {
             <EuiLink
               href={discoverLink}
               target="_blank"
-              color="text"
+              color="primary"
               css={css`
                 display: flex;
                 align-items: center;
               `}
+              external={false}
             >
               <EuiIcon
                 type="discoverApp"
@@ -174,7 +175,7 @@ const DataGrid: React.FC<ESQLDataGridProps> = (props) => {
               />
               <EuiText size="xs">
                 {i18n.translate('esqlDataGrid.openInDiscoverLabel', {
-                  defaultMessage: 'Open in Discover.',
+                  defaultMessage: 'Open in Discover',
                 })}
               </EuiText>
             </EuiLink>
diff --git a/x-pack/plugins/lens/public/app_plugin/shared/edit_on_the_fly/lens_configuration_flyout.tsx b/x-pack/plugins/lens/public/app_plugin/shared/edit_on_the_fly/lens_configuration_flyout.tsx
index 6ecc6474bc3047..65bef6038311ac 100644
--- a/x-pack/plugins/lens/public/app_plugin/shared/edit_on_the_fly/lens_configuration_flyout.tsx
+++ b/x-pack/plugins/lens/public/app_plugin/shared/edit_on_the_fly/lens_configuration_flyout.tsx
@@ -517,6 +517,7 @@ export function LensEditConfigurationFlyout({
                   flex: ${isESQLResultsAccordionOpen ? 1 : 'none'};
                 }
                 padding: 0 ${euiThemeVars.euiSize};
+                border-bottom: ${euiThemeVars.euiBorderThin};
               `}
             >
               <EuiAccordion
diff --git a/x-pack/plugins/lens/public/trigger_actions/open_lens_config/edit_action_helpers.ts b/x-pack/plugins/lens/public/trigger_actions/open_lens_config/edit_action_helpers.ts
index 7ec70e687efe5f..3834b4ba7590e5 100644
--- a/x-pack/plugins/lens/public/trigger_actions/open_lens_config/edit_action_helpers.ts
+++ b/x-pack/plugins/lens/public/trigger_actions/open_lens_config/edit_action_helpers.ts
@@ -6,6 +6,8 @@
  */
 import React from 'react';
 import './helpers.scss';
+import { css } from '@emotion/react';
+import { euiThemeVars } from '@kbn/ui-theme';
 import { toMountPoint } from '@kbn/react-kibana-mount';
 import { tracksOverlays } from '@kbn/presentation-containers';
 import type { IEmbeddable } from '@kbn/embeddable-plugin/public';
@@ -72,6 +74,10 @@ const openInlineLensConfigEditor = (
         overlayRef.close();
       },
       outsideClickCloses: true,
+      css: css`
+        max-width: calc(${euiThemeVars.euiSizeXXL} * 20);
+        min-width: calc(${euiThemeVars.euiSizeXXL} * 10);
+      `,
     }
   );