From 4704934c9a009830a79086d178217f9f09a97753 Mon Sep 17 00:00:00 2001 From: Miki Date: Tue, 23 Jul 2024 10:45:38 -0700 Subject: [PATCH] Use smaller and compressed varients of buttons and form components (#1010) * Use EuiSmallButton Signed-off-by: Miki * Use EuiSmallButtonIcon Signed-off-by: Miki * Use EuiSmallButtonEmpty Signed-off-by: Miki * Use EuiCompressedFormRow Signed-off-by: Miki * Use EuiCompressedField* Signed-off-by: Miki * Use EuiCompressedSelect and EuiCompressedSuperSelect Signed-off-by: Miki * Use EuiCompressedCheckbox and EuiCompressedCheckboxGroup Signed-off-by: Miki * Use EuiCompressedRadio and EuiCompressedRadioGroup Signed-off-by: Miki * Use EuiCompressedSwitch Signed-off-by: Miki * Use EuiCompressedEuiTextArea Signed-off-by: Miki * Use EuiCompressedComboBox Signed-off-by: Miki * Undo unnecessary EuiSmallButton* uses Signed-off-by: Miki * Introduce `compressed` FrequencyPicker` and `DateRangePicker` Also: * Clean up some code Signed-off-by: Miki * Update snapshots and fix errors Also: * Clean up some code Signed-off-by: Miki --------- Signed-off-by: Miki (cherry picked from commit cc8b95964616f23642ca88fc0564cb2fa3840eaf) Signed-off-by: Miki --- .../Comments/AlertCommentsFlyout.tsx | 34 +- public/components/Comments/CommentEditor.tsx | 26 +- .../components/Comments/ShowAlertComments.tsx | 12 +- .../AddAlertingMonitor/AddAlertingMonitor.tsx | 10 +- .../AssociateExisting/AssociateExisting.js | 4 +- .../CreateNew/CreateNew.tsx | 4 +- .../__snapshots__/CreateNew.test.js.snap | 2 +- .../AssociatedMonitors/AssociatedMonitors.tsx | 6 +- .../AssociatedMonitors/ConfirmUnlinkModal.js | 12 +- .../AssociatedMonitors.test.js.snap | 4 +- .../EnhancedAccordion/EnhancedAccordion.js | 10 +- .../EnhancedAccordion.test.js.snap | 2 +- .../__snapshots__/flyouts.test.js.snap | 2 +- .../Flyout/flyouts/alertsDashboard.js | 4 +- .../AlertsDashboardFlyoutComponent.js | 14 +- ...lertsDashboardFlyoutComponent.test.js.snap | 4 +- .../components/Flyout/flyouts/dataSources.js | 10 +- .../FormikCheckbox/FormikCheckbox.js | 4 +- .../__snapshots__/FormikCheckbox.test.js.snap | 6 +- .../FormikCodeEditor.test.js.snap | 2 +- .../FormikComboBox/FormikComboBox.js | 4 +- .../FormikFieldNumber/FormikFieldNumber.js | 4 +- .../FormikFieldNumber.test.js.snap | 4 +- .../FormikFieldPassword.js | 4 +- .../FormikFieldPassword.test.js.snap | 4 +- .../FormikFieldRadio/FormikFieldRadio.js | 4 +- .../FormikFieldText/FormikFieldText.js | 4 +- .../FormikFieldText.test.js.snap | 4 +- .../FormikFormRow/FormikFormRow.js | 6 +- .../__snapshots__/FormikFormRow.test.js.snap | 2 +- .../FormControls/FormikSelect/FormikSelect.js | 4 +- .../__snapshots__/FormikSelect.test.js.snap | 4 +- .../FormControls/FormikSwitch/FormikSwitch.js | 4 +- .../__snapshots__/FormikSwitch.test.js.snap | 11 +- .../FormikTextArea/FormikTextArea.js | 4 +- .../__snapshots__/FormikTextArea.test.js.snap | 4 +- .../EmptyFeaturesMessage.js | 10 +- .../AssociateMonitors.test.js.snap | 2 +- .../components/MonitorsList.js | 12 +- .../__snapshots__/MonitorsEditor.test.js.snap | 2 +- .../__snapshots__/MonitorsList.test.js.snap | 16 +- .../ClusterMetricsMonitor.js | 17 +- .../ClusterMetricsMonitor.test.js.snap | 10 +- .../DocumentLevelQuery.js | 12 +- .../DocumentLevelQueryTag.js | 10 +- .../ExtractionQuery/ExtractionQuery.js | 6 +- .../ExtractionQuery.test.js.snap | 4 +- .../MonitorDefinition.test.js.snap | 6 +- .../MonitorExpressions.test.js.snap | 10 +- .../expressions/GroupByPopover.js | 10 +- .../expressions/MetricPopover.js | 16 +- .../__snapshots__/ForExpression.test.js.snap | 10 +- .../__snapshots__/MonitorState.test.js.snap | 4 +- .../MonitorTimeField.test.js.snap | 8 +- .../components/MonitorType/MonitorType.js | 10 +- .../QueryPerformance/QueryPerformance.js | 10 +- .../components/Schedule/Frequencies/Daily.js | 61 +- .../Schedule/Frequencies/FrequencyPicker.js | 2 +- .../Schedule/Frequencies/Monthly.js | 4 +- .../components/Schedule/Frequencies/Weekly.js | 10 +- .../__snapshots__/Frequencies.test.js.snap | 44 +- .../AnomalyDetector.test.js.snap | 30 +- .../containers/CreateMonitor/CreateMonitor.js | 10 +- .../containers/DefineMonitor/DefineMonitor.js | 6 +- .../__snapshots__/DefineMonitor.test.js.snap | 4 +- .../__snapshots__/MonitorIndex.test.js.snap | 30 +- .../WorkflowDetails.test.js.snap | 20 +- .../CreateTrigger/components/Action/Action.js | 22 +- .../Action/__snapshots__/Action.test.js.snap | 88 +- .../components/Action/actions/Message.js | 37 +- .../__snapshots__/Message.test.js.snap | 28 +- .../ActionEmptyPrompt/ActionEmptyPrompt.js | 6 +- .../ActionEmptyPrompt.test.js.snap | 4 +- .../AddActionButton/AddActionButton.js | 6 +- .../BucketLevelTriggerExpression.js | 36 +- .../BucketLevelTriggerExpression.test.js.snap | 4 +- .../BucketLevelTriggerQuery.js | 10 +- .../ExpressionBuilder.js | 6 +- .../CompositeTriggerCondition.test.js.snap | 4 +- .../ExpressionBuilder.test.js.snap | 2 +- .../ExpressionEditor.test.js.snap | 2 +- .../TriggerExpressions/TriggerExpressions.js | 27 +- .../TriggerExpressions.test.js.snap | 6 +- .../components/TriggerQuery/TriggerQuery.js | 14 +- .../__snapshots__/TriggerQuery.test.js.snap | 6 +- .../ConfigureTriggers/ConfigureTriggers.js | 4 +- .../CreateTrigger/CreateTrigger.js | 10 +- .../NotificationConfigDialog.js | 8 +- .../TriggerNotifications.js | 4 +- .../TriggerNotificationsContent.js | 10 +- .../DefineCompositeLevelTrigger.test.js.snap | 18 +- .../TriggerNotifications.test.js.snap | 2 +- .../TriggerNotificationsContent.test.js.snap | 20 +- .../DocumentLevelTriggerExpression.js | 10 +- .../AnomalyDetectorTrigger.test.js.snap | 2 +- .../AcknowledgeAlertsModal.js | 14 +- .../ChainedAlertDetailsFlyout.tsx | 10 +- .../DashboardControls/DashboardControls.js | 18 +- .../DashboardControls.test.js.snap | 12 +- .../DashboardEmptyPrompt.js | 8 +- .../DashboardEmptyPrompt.test.js.snap | 2 +- .../FindingsDashboard/FindingFlyout.js | 4 +- .../pages/Dashboard/containers/Dashboard.js | 18 +- .../Dashboard/containers/FindingsDashboard.js | 4 +- .../__snapshots__/Dashboard.test.js.snap | 1577 ++++++++++------- .../DestinationsActions.js | 6 +- .../DestinationsActions.test.js.snap | 2 +- .../DestinationsControls.js | 12 +- .../DestinationsControls.test.js.snap | 8 +- .../EmptyDestinations/EmptyDestinations.js | 6 +- .../EmptyDestinations.test.js.snap | 2 +- .../FullPageNotificationsInfoCallOut.js | 10 +- ...lPageNotificationsInfoCallOut.test.js.snap | 4 +- .../NotificationsInfoCallOut.js | 8 +- .../NotificationsInfoCallOut.test.js.snap | 4 +- .../createDestinations/Email/EmailGroup.js | 2 +- .../createDestinations/Email/Sender.js | 2 +- .../CreateDestination/CreateDestination.js | 6 +- .../EmailRecipients/EmailRecipients.js | 6 +- .../EmailSender/EmailSender.js | 6 +- .../ManageEmailGroups/ManageEmailGroups.js | 4 +- .../ManageSenders/ManageSenders.js | 6 +- .../DestinationsList.test.js.snap | 279 +-- .../EmptyHistory/EmptyHistory.js | 6 +- .../__snapshots__/EmptyHistory.test.js.snap | 2 +- .../containers/MonitorDetails.js | 18 +- .../MonitorHistory/DateRangePicker.js | 5 + .../MonitorHistory/MonitorHistory.js | 1 + .../MonitorActions/MonitorActions.js | 18 +- .../__snapshots__/MonitorActions.test.js.snap | 6 +- .../MonitorControls/MonitorControls.js | 12 +- .../MonitorEmptyPrompt/MonitorEmptyPrompt.js | 10 +- .../MonitorEmptyPrompt.test.js.snap | 2 +- 133 files changed, 1739 insertions(+), 1426 deletions(-) diff --git a/public/components/Comments/AlertCommentsFlyout.tsx b/public/components/Comments/AlertCommentsFlyout.tsx index f112c4c49..d7e677cf4 100644 --- a/public/components/Comments/AlertCommentsFlyout.tsx +++ b/public/components/Comments/AlertCommentsFlyout.tsx @@ -5,17 +5,17 @@ import React, { useCallback, useEffect, useState } from "react"; import { Comment } from "../../models/Comments"; -import { - EuiFlyout, - EuiFlyoutHeader, - EuiFlyoutBody, - EuiCommentList, - EuiText, - EuiButtonIcon, - EuiContextMenuItem, +import { + EuiFlyout, + EuiFlyoutHeader, + EuiFlyoutBody, + EuiCommentList, + EuiText, + EuiButtonIcon, + EuiContextMenuItem, EuiContextMenuPanel, - EuiPopover, - EuiTitle, + EuiPopover, + EuiTitle, EuiSpacer, EuiCallOut, EuiLink @@ -77,7 +77,7 @@ export const AlertCommentsFlyout: React.FC = ({ alertI const createComment = async () => { setCreatePending(true); - await httpClient.post(`../api/alerting/comments/${alertId}`, { body: JSON.stringify({ + await httpClient.post(`../api/alerting/comments/${alertId}`, { body: JSON.stringify({ content: draftCommentContent })}); @@ -150,7 +150,7 @@ export const AlertCommentsFlyout: React.FC = ({ alertI onCancel={() => onEditCancel(comment, idx)} /> ); - + const customActions = comment.state === 'readonly' && ( = ({ alertI /> ); - + return { username: comment.user || 'Unknown', event: `${comment.last_updated_time ? 'edited' : 'added'} comment on`, @@ -204,12 +204,12 @@ export const AlertCommentsFlyout: React.FC = ({ alertI - - The feature is experimental and should not be used in a production environment. + The feature is experimental and should not be used in a production environment. The posted comments will be impacted if the feature is deactivated. - For more information see Documentation. + For more information see Documentation. To leave feedback, visit github.com. @@ -236,4 +236,4 @@ export const AlertCommentsFlyout: React.FC = ({ alertI ) -} \ No newline at end of file +} diff --git a/public/components/Comments/CommentEditor.tsx b/public/components/Comments/CommentEditor.tsx index 1b6a00e05..0684f29ab 100644 --- a/public/components/Comments/CommentEditor.tsx +++ b/public/components/Comments/CommentEditor.tsx @@ -4,10 +4,10 @@ */ import React from "react"; -import { - EuiFlexGroup, - EuiFlexItem, - EuiButton +import { + EuiFlexGroup, + EuiFlexItem, + EuiSmallButton } from "@elastic/eui"; export interface CommentEditorProps { @@ -19,13 +19,13 @@ export interface CommentEditorProps { onContentChange: React.ChangeEventHandler; } -export const CommentEditor: React.FC = ({ - isLoading, +export const CommentEditor: React.FC = ({ + isLoading, draftCommentContent, saveDisabled, - onSave, + onSave, onCancel, - onContentChange, + onContentChange, }) => ( @@ -35,17 +35,17 @@ export const CommentEditor: React.FC = ({ {onCancel && ( - + Cancel - + )} - + Save - + -) \ No newline at end of file +) diff --git a/public/components/Comments/ShowAlertComments.tsx b/public/components/Comments/ShowAlertComments.tsx index 7b2b55dc1..340b6f471 100644 --- a/public/components/Comments/ShowAlertComments.tsx +++ b/public/components/Comments/ShowAlertComments.tsx @@ -5,7 +5,7 @@ import React, { useCallback, useEffect, useState } from "react"; import { Comment } from "../../models/Comments"; -import { EuiButtonIcon, EuiToolTip } from "@elastic/eui"; +import { EuiSmallButtonIcon, EuiToolTip } from "@elastic/eui"; import { AlertCommentsFlyout } from "./AlertCommentsFlyout"; export interface ShowAlertCommentsProps { @@ -15,19 +15,19 @@ export interface ShowAlertCommentsProps { export const ShowAlertComments: React.FC = ({ alert, httpClient }) => { const [commentsFlyout, setCommentsFlyout] = useState(null); - - const showCommentsFlyout = useCallback(() => { + + const showCommentsFlyout = useCallback(() => { setCommentsFlyout( setCommentsFlyout(null)} + closeFlyout={() => setCommentsFlyout(null)} />); }, [setCommentsFlyout]); return ( <> - = ({ alert, htt {commentsFlyout} ) -} \ No newline at end of file +} diff --git a/public/components/FeatureAnywhereContextMenu/AddAlertingMonitor/AddAlertingMonitor.tsx b/public/components/FeatureAnywhereContextMenu/AddAlertingMonitor/AddAlertingMonitor.tsx index d8e4f734f..8d3341b3f 100644 --- a/public/components/FeatureAnywhereContextMenu/AddAlertingMonitor/AddAlertingMonitor.tsx +++ b/public/components/FeatureAnywhereContextMenu/AddAlertingMonitor/AddAlertingMonitor.tsx @@ -12,8 +12,8 @@ import { EuiFlyoutFooter, EuiTitle, EuiSpacer, - EuiButton, - EuiButtonEmpty, + EuiSmallButton, + EuiSmallButtonEmpty, EuiFormFieldset, EuiCheckableCard, EuiText @@ -225,17 +225,17 @@ function AddAlertingMonitor({ - Cancel + Cancel - onSubmit({ handleSubmit, validateForm })} fill isLoading={isLoading} disabled={!isAssociateAllowed} > {flyoutMode === 'existing' ? 'Associate' : 'Create'} monitor - + diff --git a/public/components/FeatureAnywhereContextMenu/AddAlertingMonitor/AssociateExisting/AssociateExisting.js b/public/components/FeatureAnywhereContextMenu/AddAlertingMonitor/AssociateExisting/AssociateExisting.js index b05c2cf14..1fa3b2dcd 100644 --- a/public/components/FeatureAnywhereContextMenu/AddAlertingMonitor/AssociateExisting/AssociateExisting.js +++ b/public/components/FeatureAnywhereContextMenu/AddAlertingMonitor/AssociateExisting/AssociateExisting.js @@ -9,7 +9,7 @@ import { EuiSpacer, EuiIcon, EuiText, - EuiComboBox, + EuiCompressedComboBox, EuiLoadingSpinner, EuiLink, EuiFlexGroup, @@ -94,7 +94,7 @@ function AssociateExisting({ monitors, selectedMonitorId, setSelectedMonitorId } {!monitors && } {monitors && ( - - setIsShowVis(!isShowVis)} diff --git a/public/components/FeatureAnywhereContextMenu/AddAlertingMonitor/CreateNew/__snapshots__/CreateNew.test.js.snap b/public/components/FeatureAnywhereContextMenu/AddAlertingMonitor/CreateNew/__snapshots__/CreateNew.test.js.snap index 9488e4705..4e68c404a 100644 --- a/public/components/FeatureAnywhereContextMenu/AddAlertingMonitor/CreateNew/__snapshots__/CreateNew.test.js.snap +++ b/public/components/FeatureAnywhereContextMenu/AddAlertingMonitor/CreateNew/__snapshots__/CreateNew.test.js.snap @@ -36,7 +36,7 @@ exports[`CreateNew renders 1`] = ` /> - - setFlyoutMode('existing')}> + setFlyoutMode('existing')}> Associate a monitor - + diff --git a/public/components/FeatureAnywhereContextMenu/AssociatedMonitors/ConfirmUnlinkModal.js b/public/components/FeatureAnywhereContextMenu/AssociatedMonitors/ConfirmUnlinkModal.js index 63ed4af5f..7ecc36e77 100644 --- a/public/components/FeatureAnywhereContextMenu/AssociatedMonitors/ConfirmUnlinkModal.js +++ b/public/components/FeatureAnywhereContextMenu/AssociatedMonitors/ConfirmUnlinkModal.js @@ -5,8 +5,8 @@ import React, { useState } from 'react'; import { - EuiButton, - EuiButtonEmpty, + EuiSmallButton, + EuiSmallButtonEmpty, EuiModal, EuiModalBody, EuiModalFooter, @@ -35,11 +35,11 @@ export const ConfirmUnlinkDetectorModal = (props) => { {isLoading ? null : ( - + Cancel - + )} - { }} > {'Remove association'} - + diff --git a/public/components/FeatureAnywhereContextMenu/AssociatedMonitors/__snapshots__/AssociatedMonitors.test.js.snap b/public/components/FeatureAnywhereContextMenu/AssociatedMonitors/__snapshots__/AssociatedMonitors.test.js.snap index d32070857..4ce977175 100644 --- a/public/components/FeatureAnywhereContextMenu/AssociatedMonitors/__snapshots__/AssociatedMonitors.test.js.snap +++ b/public/components/FeatureAnywhereContextMenu/AssociatedMonitors/__snapshots__/AssociatedMonitors.test.js.snap @@ -33,14 +33,14 @@ exports[`AssociatedMonitors renders 1`] = ` - Associate a monitor - + (
- )} {isButton && ( - Add trigger - + )}
diff --git a/public/components/FeatureAnywhereContextMenu/EnhancedAccordion/__snapshots__/EnhancedAccordion.test.js.snap b/public/components/FeatureAnywhereContextMenu/EnhancedAccordion/__snapshots__/EnhancedAccordion.test.js.snap index 73335a69b..324631833 100644 --- a/public/components/FeatureAnywhereContextMenu/EnhancedAccordion/__snapshots__/EnhancedAccordion.test.js.snap +++ b/public/components/FeatureAnywhereContextMenu/EnhancedAccordion/__snapshots__/EnhancedAccordion.test.js.snap @@ -7,7 +7,7 @@ exports[`EnhancedAccordion renders 1`] = `
- - { @@ -29,7 +29,7 @@ const alertsDashboard = (payload) => { - ( - { const actions = [ - Acknowledge - , + , ]; if (!_.isEmpty(detectorId)) { actions.unshift( - View detector - + ); } return actions; diff --git a/public/components/Flyout/flyouts/components/__snapshots__/AlertsDashboardFlyoutComponent.test.js.snap b/public/components/Flyout/flyouts/components/__snapshots__/AlertsDashboardFlyoutComponent.test.js.snap index 95cfb576a..539f38a99 100644 --- a/public/components/Flyout/flyouts/components/__snapshots__/AlertsDashboardFlyoutComponent.test.js.snap +++ b/public/components/Flyout/flyouts/components/__snapshots__/AlertsDashboardFlyoutComponent.test.js.snap @@ -177,13 +177,13 @@ exports[`AlertsDashboardFlyoutComponent renders 1`] = ` Acknowledge - , + , ] } bodyStyles={ diff --git a/public/components/Flyout/flyouts/dataSources.js b/public/components/Flyout/flyouts/dataSources.js index 424687506..d3e97ae29 100644 --- a/public/components/Flyout/flyouts/dataSources.js +++ b/public/components/Flyout/flyouts/dataSources.js @@ -4,7 +4,13 @@ */ import React from 'react'; -import { EuiBasicTable, EuiFlexGroup, EuiButtonIcon, EuiTitle, EuiFlexItem } from '@elastic/eui'; +import { + EuiBasicTable, + EuiFlexGroup, + EuiSmallButtonIcon, + EuiTitle, + EuiFlexItem, +} from '@elastic/eui'; import { MONITOR_TYPE } from '../../../utils/constants'; export const DATA_SOURCES_FLYOUT_TYPE = 'dataSources'; @@ -76,7 +82,7 @@ const dataSources = ({ - ( - + ); FormikCheckbox.propTypes = { diff --git a/public/components/FormControls/FormikCheckbox/__snapshots__/FormikCheckbox.test.js.snap b/public/components/FormControls/FormikCheckbox/__snapshots__/FormikCheckbox.test.js.snap index a48e1f71d..656cf8241 100644 --- a/public/components/FormControls/FormikCheckbox/__snapshots__/FormikCheckbox.test.js.snap +++ b/public/components/FormControls/FormikCheckbox/__snapshots__/FormikCheckbox.test.js.snap @@ -2,7 +2,7 @@ exports[`FormikCheckbox render 1`] = `
( - ( - diff --git a/public/components/FormControls/FormikFieldNumber/__snapshots__/FormikFieldNumber.test.js.snap b/public/components/FormControls/FormikFieldNumber/__snapshots__/FormikFieldNumber.test.js.snap index 94e93e189..bdb723abe 100644 --- a/public/components/FormControls/FormikFieldNumber/__snapshots__/FormikFieldNumber.test.js.snap +++ b/public/components/FormControls/FormikFieldNumber/__snapshots__/FormikFieldNumber.test.js.snap @@ -2,13 +2,13 @@ exports[`FormikFieldNumber renders 1`] = `
diff --git a/public/components/FormControls/FormikFieldPassword/FormikFieldPassword.js b/public/components/FormControls/FormikFieldPassword/FormikFieldPassword.js index 9f25f2eb1..bd5f79f82 100644 --- a/public/components/FormControls/FormikFieldPassword/FormikFieldPassword.js +++ b/public/components/FormControls/FormikFieldPassword/FormikFieldPassword.js @@ -5,7 +5,7 @@ import React from 'react'; import PropTypes from 'prop-types'; -import { EuiFieldPassword } from '@elastic/eui'; +import { EuiCompressedFieldPassword } from '@elastic/eui'; import FormikInputWrapper from '../FormikInputWrapper'; import FormikFormRow from '../FormikFormRow'; @@ -38,7 +38,7 @@ const FieldPassword = ({ field, inputProps: { onChange, isInvalid, onFocus, ...rest }, }) => ( - diff --git a/public/components/FormControls/FormikFieldPassword/__snapshots__/FormikFieldPassword.test.js.snap b/public/components/FormControls/FormikFieldPassword/__snapshots__/FormikFieldPassword.test.js.snap index 1a14c7612..aaa527916 100644 --- a/public/components/FormControls/FormikFieldPassword/__snapshots__/FormikFieldPassword.test.js.snap +++ b/public/components/FormControls/FormikFieldPassword/__snapshots__/FormikFieldPassword.test.js.snap @@ -2,13 +2,13 @@ exports[`FormikFieldPassword renders 1`] = `
( - diff --git a/public/components/FormControls/FormikFieldText/FormikFieldText.js b/public/components/FormControls/FormikFieldText/FormikFieldText.js index 32a2fb5f4..0ef96ce97 100644 --- a/public/components/FormControls/FormikFieldText/FormikFieldText.js +++ b/public/components/FormControls/FormikFieldText/FormikFieldText.js @@ -5,7 +5,7 @@ import React from 'react'; import PropTypes from 'prop-types'; -import { EuiFieldText } from '@elastic/eui'; +import { EuiCompressedFieldText } from '@elastic/eui'; import FormikInputWrapper from '../FormikInputWrapper'; import FormikFormRow from '../FormikFormRow'; @@ -38,7 +38,7 @@ const FieldText = ({ field, inputProps: { onChange, isInvalid, onFocus, ...rest }, }) => ( - diff --git a/public/components/FormControls/FormikFieldText/__snapshots__/FormikFieldText.test.js.snap b/public/components/FormControls/FormikFieldText/__snapshots__/FormikFieldText.test.js.snap index 26362cffe..c9266c991 100644 --- a/public/components/FormControls/FormikFieldText/__snapshots__/FormikFieldText.test.js.snap +++ b/public/components/FormControls/FormikFieldText/__snapshots__/FormikFieldText.test.js.snap @@ -2,13 +2,13 @@ exports[`FormikFieldText renders 1`] = `
diff --git a/public/components/FormControls/FormikFormRow/FormikFormRow.js b/public/components/FormControls/FormikFormRow/FormikFormRow.js index 9de0b7a3a..df4a3fd54 100644 --- a/public/components/FormControls/FormikFormRow/FormikFormRow.js +++ b/public/components/FormControls/FormikFormRow/FormikFormRow.js @@ -5,17 +5,17 @@ import React from 'react'; import PropTypes from 'prop-types'; -import { EuiFormRow } from '@elastic/eui'; +import { EuiCompressedFormRow } from '@elastic/eui'; const FormikFormRow = ({ children, form, name, rowProps: { isInvalid, error, ...rest } }) => ( - {children} - + ); FormikFormRow.propTypes = { diff --git a/public/components/FormControls/FormikFormRow/__snapshots__/FormikFormRow.test.js.snap b/public/components/FormControls/FormikFormRow/__snapshots__/FormikFormRow.test.js.snap index eda3bc493..91b357a93 100644 --- a/public/components/FormControls/FormikFormRow/__snapshots__/FormikFormRow.test.js.snap +++ b/public/components/FormControls/FormikFormRow/__snapshots__/FormikFormRow.test.js.snap @@ -2,7 +2,7 @@ exports[`FormikFormRow renders 1`] = `
( -
diff --git a/public/pages/CreateMonitor/components/MonitorExpressions/__snapshots__/MonitorExpressions.test.js.snap b/public/pages/CreateMonitor/components/MonitorExpressions/__snapshots__/MonitorExpressions.test.js.snap index a7bdca539..a13ae43d9 100644 --- a/public/pages/CreateMonitor/components/MonitorExpressions/__snapshots__/MonitorExpressions.test.js.snap +++ b/public/pages/CreateMonitor/components/MonitorExpressions/__snapshots__/MonitorExpressions.test.js.snap @@ -110,20 +110,20 @@ exports[`MonitorExpressions renders 1`] = ` style="width:150px" >
@@ -23,19 +23,19 @@ exports[`MonitorTimeField renders 1`] = `