conditional remove empty div, change callout size to stop modal chang…

…ing size, anon function callback for enableStore, translations use correct type
elastic · Nov 11, 2024 · 6f3aafc · 6f3aafc
1 parent a3b9d2f
commit 6f3aafc
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 5 deletions.
diff --git a/..._solution/public/entity_analytics/components/entity_store/components/enablement_modal.tsx b/..._solution/public/entity_analytics/components/entity_store/components/enablement_modal.tsx
@@ -74,6 +74,7 @@ export const EntityStoreEnablementModal: React.FC<EntityStoreEnablementModalProp
   }
   const proceedWarning = (
     <EuiCallOut
+      size="s"
       color="danger"
       css={css`
         border-radius: ${euiTheme.border.radius.medium};
@@ -147,12 +148,12 @@ export const EntityStoreEnablementModal: React.FC<EntityStoreEnablementModalProp
       </EuiModalBody>
 
       <EuiModalFooter>
-        <EuiFlexGroup justifyContent="spaceBetween" alignItems="center">
-          <EuiFlexItem>{!enablementOptions && proceedWarning}</EuiFlexItem>
+        <EuiFlexGroup justifyContent="flexEnd" alignItems="center">
+          {!enablementOptions ? <EuiFlexItem>{proceedWarning}</EuiFlexItem> : null}
           <EuiFlexItem grow={false}>
-            <EuiFlexGroup direction="row">
+            <EuiFlexGroup direction="row" justifyContent="flexEnd">
               <EuiButtonEmpty onClick={() => toggle(false)}>{'Cancel'}</EuiButtonEmpty>
-              <EuiButton onClick={enableStore(enablements)} fill>
+              <EuiButton onClick={() => enableStore(enablements)} fill>
                 <FormattedMessage
                   id="xpack.securitySolution.entityAnalytics.enablements.modal.enable"
                   defaultMessage="Enable"

diff --git a/...plugins/security_solution/public/entity_analytics/components/entity_store/translations.ts b/...plugins/security_solution/public/entity_analytics/components/entity_store/translations.ts
@@ -64,7 +64,7 @@ export const ENABLEMENT_DESCRIPTION_BOTH = i18n.translate(
 );
 
 export const ENABLEMENT_WARNING_SELECT_TO_PROCEED = i18n.translate(
-  'xpack.securitySolution.entityAnalytics.entityStore.enablement.description.both',
+  'xpack.securitySolution.entityAnalytics.entityStore.enablement.description.enablementWarningMessage',
   {
     defaultMessage: 'Please enable at least one option to proceed.',
   }