From fb03fa8277b9caa2a4c82c660b58e846cbda504d Mon Sep 17 00:00:00 2001 From: Joanne Wang Date: Tue, 24 Sep 2024 15:19:23 -0400 Subject: [PATCH] Revert "[Fit & Finish] Updated Fit and Finish guidelines (#256) (#259)" This reverts commit fb48dbcd099c1526c46ece90f640e65f32c3234f. --- .../components/ContentPanel/ContentPanel.tsx | 14 +-- .../__snapshots__/ContentPanel.test.tsx.snap | 20 ++-- public/pages/Channels/Channels.tsx | 9 +- .../ChannelDetails.test.tsx.snap | 70 +++++------- .../__snapshots__/Channels.test.tsx.snap | 36 +++--- .../DetailsListModal.test.tsx.snap | 52 +-------- .../DetailsTableModal.test.tsx.snap | 104 ++---------------- .../Channels/components/ChannelActions.tsx | 1 - .../components/details/ChannelDetails.tsx | 5 +- .../details/ChannelDetailsActions.tsx | 1 - .../components/modals/DeleteChannelModal.tsx | 11 +- .../components/modals/DetailsListModal.tsx | 7 +- .../components/modals/DetailsTableModal.tsx | 7 +- .../components/modals/MuteChannelModal.tsx | 8 +- public/pages/CreateChannel/CreateChannel.tsx | 5 +- .../ChannelNamePanel.test.tsx.snap | 20 ++-- .../__snapshots__/CreateChannel.test.tsx.snap | 10 +- .../modals/CreateRecipientGroupModal.tsx | 7 +- .../modals/CreateSESSenderModal.tsx | 7 +- .../components/modals/CreateSenderModal.tsx | 7 +- public/pages/Emails/CreateRecipientGroup.tsx | 6 +- public/pages/Emails/CreateSESSender.tsx | 6 +- public/pages/Emails/CreateSender.tsx | 6 +- public/pages/Emails/EmailGroups.tsx | 6 +- public/pages/Emails/EmailSenders.tsx | 8 +- .../CreateRecipientGroup.test.tsx.snap | 10 +- .../CreateSESSender.test.tsx.snap | 30 ++--- .../__snapshots__/CreateSender.test.tsx.snap | 30 ++--- .../__snapshots__/EmailGroups.test.tsx.snap | 10 +- .../__snapshots__/EmailSenders.test.tsx.snap | 20 ++-- .../RecipientGroupsTable.test.tsx.snap | 36 +++--- .../__snapshots__/SendersTable.test.tsx.snap | 36 +++--- .../modals/DeleteRecipientGroupModal.tsx | 10 +- .../components/modals/DeleteSenderModal.tsx | 11 +- .../tables/RecipientGroupsTable.tsx | 12 +- .../components/tables/SESSendersTable.tsx | 10 +- .../Emails/components/tables/SendersTable.tsx | 14 +-- .../__snapshots__/Main.test.tsx.snap | 36 +++--- 38 files changed, 222 insertions(+), 476 deletions(-) diff --git a/public/components/ContentPanel/ContentPanel.tsx b/public/components/ContentPanel/ContentPanel.tsx index 2f6acc7c..a6660e2b 100644 --- a/public/components/ContentPanel/ContentPanel.tsx +++ b/public/components/ContentPanel/ContentPanel.tsx @@ -8,13 +8,13 @@ import { EuiFlexItem, EuiHorizontalRule, EuiPanel, - EuiText, + EuiTitle, } from '@elastic/eui'; import React from 'react'; interface ContentPanelProps { title?: string; - titleSize?: 'xs' | 's' | 'm'; + titleSize?: 'xxxs' | 'xxs' | 'xs' | 's' | 'm' | 'l'; total?: number; bodyStyles?: object; panelStyles?: object; @@ -25,7 +25,7 @@ interface ContentPanelProps { const ContentPanel: React.SFC = ({ title = '', - titleSize = 's', + titleSize = 'l', total = undefined, bodyStyles = {}, panelStyles = {}, @@ -40,16 +40,16 @@ const ContentPanel: React.SFC = ({ alignItems="center" > - -

+ +

{title} {total !== undefined ? ( {` (${total})`} ) : null} -

- +

+ {actions ? ( diff --git a/public/components/__tests__/__snapshots__/ContentPanel.test.tsx.snap b/public/components/__tests__/__snapshots__/ContentPanel.test.tsx.snap index 13d35e14..4a277201 100644 --- a/public/components/__tests__/__snapshots__/ContentPanel.test.tsx.snap +++ b/public/components/__tests__/__snapshots__/ContentPanel.test.tsx.snap @@ -11,13 +11,11 @@ exports[` spec renders the component 1`] = `
-
-

- Testing -

-
+ Testing +
spec renders with empty actions 1`] = `
-
-

- Testing -

-
+ Testing +
isSelectable={true} selection={selection} noItemsMessage={

No channels to display

} - body={"To send or receive notifications, you will need to create a notification channel."} + title={

No channels to display

} + body="To send or receive notifications, you will need to create a notification channel." actions={ Create channel } />} @@ -310,7 +309,7 @@ export class Channels extends MDSEnabledComponent } bodyStyles={{ padding: 'initial' }} title="Channels" - titleSize="s" + titleSize="m" total={this.state.total} > {channelControlsComponent} @@ -319,7 +318,7 @@ export class Channels extends MDSEnabledComponent )} - + ); } }; diff --git a/public/pages/Channels/__tests__/__snapshots__/ChannelDetails.test.tsx.snap b/public/pages/Channels/__tests__/__snapshots__/ChannelDetails.test.tsx.snap index b03d0e62..f62687c9 100644 --- a/public/pages/Channels/__tests__/__snapshots__/ChannelDetails.test.tsx.snap +++ b/public/pages/Channels/__tests__/__snapshots__/ChannelDetails.test.tsx.snap @@ -15,13 +15,11 @@ exports[` spec handles a non-existing channel 1`] = `
-
-

- - -

-
+ - +
spec handles a non-existing channel 1`] = `
-
-

- Name and description -

-
+ Name and description +
spec handles a non-existing channel 1`] = `
-
-

- Configurations -

-
+ Configurations +
spec renders a specific channel 1`] = `
-
-

- - -

-
+ - +
spec renders a specific channel 1`] = `
-
-

- Name and description -

-
+ Name and description +
spec renders a specific channel 1`] = `
-
-

- Configurations -

-
+ Configurations +
spec renders the component 1`] = `
-
-

- - -

-
+ - +
spec renders the empty component 1`] = `
-
-

- Channels - - (0) - -

-
+ Channels + + (0) + +
spec renders the empty component 1`] = `
-
-

- No channels to display -

-
+ No channels to display + @@ -508,11 +504,7 @@ exports[` spec renders the empty component 1`] = `
-
- "To send or receive notifications, you will need to create a notification channel." -
+ To send or receive notifications, you will need to create a notification channel.
spec renders the component 1`] = `
-
-

- test header -

-
+ test header
spec renders the component 1`] = `
-
-

- test header -

-
+ test header
spec renders the component 1`] = `
-
-

- test header -

-
+ test header
spec renders the component 1`] = `
-
-

- test header -

-
+ test header
spec renders the component 1`] = `
-
-

- test header -

-
+ test header
spec renders the component 1`] = `
- -
-

- test header -

-
- + test header
diff --git a/public/pages/Channels/__tests__/__snapshots__/DetailsTableModal.test.tsx.snap b/public/pages/Channels/__tests__/__snapshots__/DetailsTableModal.test.tsx.snap index 5ff99ef8..1df1f38b 100644 --- a/public/pages/Channels/__tests__/__snapshots__/DetailsTableModal.test.tsx.snap +++ b/public/pages/Channels/__tests__/__snapshots__/DetailsTableModal.test.tsx.snap @@ -82,13 +82,7 @@ exports[` spec renders headers 1`] = `
-
-

- test header -

-
+ test header
spec renders headers 1`] = `
-
-

- test header -

-
+ test header
spec renders headers 1`] = `
-
-

- test header -

-
+ test header
spec renders headers 1`] = `
-
-

- test header -

-
+ test header
spec renders headers 1`] = `
-
-

- test header -

-
+ test header
spec renders headers 1`] = `
- -
-

- test header -

-
- + test header
@@ -1975,13 +1935,7 @@ exports[` spec renders parameters 1`] = `
-
-

- test header -

-
+ test header
spec renders parameters 1`] = `
-
-

- test header -

-
+ test header
spec renders parameters 1`] = `
-
-

- test header -

-
+ test header
spec renders parameters 1`] = `
-
-

- test header -

-
+ test header
spec renders parameters 1`] = `
-
-

- test header -

-
+ test header
spec renders parameters 1`] = `
- -
-

- test header -

-
- + test header
diff --git a/public/pages/Channels/components/ChannelActions.tsx b/public/pages/Channels/components/ChannelActions.tsx index 62b49eb3..b809e537 100644 --- a/public/pages/Channels/components/ChannelActions.tsx +++ b/public/pages/Channels/components/ChannelActions.tsx @@ -99,7 +99,6 @@ export function ChannelActions(props: ChannelActionsProps) { { setIsPopoverOpen(false); if (params.modal) { diff --git a/public/pages/Channels/components/details/ChannelDetails.tsx b/public/pages/Channels/components/details/ChannelDetails.tsx index 26a98565..96b71f4f 100644 --- a/public/pages/Channels/components/details/ChannelDetails.tsx +++ b/public/pages/Channels/components/details/ChannelDetails.tsx @@ -11,6 +11,7 @@ import { EuiSmallButton, EuiSpacer, EuiText, + EuiTitle, } from '@elastic/eui'; import { Toast } from '@elastic/eui/src/components/toast/global_toast_list'; import React, { useContext, useEffect, useState } from 'react'; @@ -236,9 +237,9 @@ export function ChannelDetails(props: ChannelDetailsProps) { - +

{channel?.name ?? '-'}

- + {badgeComponent} diff --git a/public/pages/Channels/components/details/ChannelDetailsActions.tsx b/public/pages/Channels/components/details/ChannelDetailsActions.tsx index e01b1716..7d01d3df 100644 --- a/public/pages/Channels/components/details/ChannelDetailsActions.tsx +++ b/public/pages/Channels/components/details/ChannelDetailsActions.tsx @@ -101,7 +101,6 @@ export function ChannelDetailsActions(props: ChannelDetailsActionsProps) { { setIsPopoverOpen(false); if (params.modal) { diff --git a/public/pages/Channels/components/modals/DeleteChannelModal.tsx b/public/pages/Channels/components/modals/DeleteChannelModal.tsx index 1002077c..636ed1f4 100644 --- a/public/pages/Channels/components/modals/DeleteChannelModal.tsx +++ b/public/pages/Channels/components/modals/DeleteChannelModal.tsx @@ -48,14 +48,10 @@ export const DeleteChannelModal = (props: DeleteChannelModalProps) => { - - -

{`Delete ${name}?`}

- - + {`Delete ${name}?`} - {message} + {message} {num >= 2 && ( <> @@ -63,7 +59,6 @@ export const DeleteChannelModal = (props: DeleteChannelModalProps) => {
  • {channel.name}
    •  @@ -71,7 +66,7 @@ export const DeleteChannelModal = (props: DeleteChannelModalProps) => { )} - + To confirm delete, type delete in the field. - - -

    {props.header}

    -     -     + {props.header}     diff --git a/public/pages/Channels/components/modals/DetailsTableModal.tsx b/public/pages/Channels/components/modals/DetailsTableModal.tsx index 462b4ebc..773fe1f3 100644 --- a/public/pages/Channels/components/modals/DetailsTableModal.tsx +++ b/public/pages/Channels/components/modals/DetailsTableModal.tsx @@ -13,7 +13,6 @@ import { EuiModalHeaderTitle, EuiOverlayMask, EuiTableFieldDataColumnType, - EuiText, } from '@elastic/eui'; import React from 'react'; import { ModalRootProps } from '../../../../components/Modal/ModalRoot'; @@ -50,11 +49,7 @@ export function DetailsTableModal(props: DetailsTableModalProps) { - - -

    {props.header}

    -     -     + {props.header}     diff --git a/public/pages/Channels/components/modals/MuteChannelModal.tsx b/public/pages/Channels/components/modals/MuteChannelModal.tsx index 1634f632..b1b807f6 100644 --- a/public/pages/Channels/components/modals/MuteChannelModal.tsx +++ b/public/pages/Channels/components/modals/MuteChannelModal.tsx @@ -37,14 +37,10 @@ export const MuteChannelModal = (props: MuteChannelModalProps) => { - - -

    {`Mute ${props.selected[0].name}?`}

    -     -     + {`Mute ${props.selected[0].name}?`}     - + This channel will stop sending notifications to its recipients. However, the channel will remain available for selection. diff --git a/public/pages/CreateChannel/CreateChannel.tsx b/public/pages/CreateChannel/CreateChannel.tsx index 3751f45c..d82d91a8 100644 --- a/public/pages/CreateChannel/CreateChannel.tsx +++ b/public/pages/CreateChannel/CreateChannel.tsx @@ -14,6 +14,7 @@ import { EuiCompressedSuperSelect, EuiSuperSelectOption, EuiText, + EuiTitle, } from '@elastic/eui'; import queryString from 'query-string'; import React, { createContext, useContext, useEffect, useState } from 'react'; @@ -374,9 +375,9 @@ export function CreateChannel(props: CreateChannelsProps) { value={{ edit: props.edit, inputErrors, setInputErrors }} > {!getUseUpdatedUx() && ( - +

    {`${props.edit ? 'Edit' : 'Create'} channel`}

    -     +     )} diff --git a/public/pages/CreateChannel/__tests__/__snapshots__/ChannelNamePanel.test.tsx.snap b/public/pages/CreateChannel/__tests__/__snapshots__/ChannelNamePanel.test.tsx.snap index 303884b0..0e4b7f41 100644 --- a/public/pages/CreateChannel/__tests__/__snapshots__/ChannelNamePanel.test.tsx.snap +++ b/public/pages/CreateChannel/__tests__/__snapshots__/ChannelNamePanel.test.tsx.snap @@ -11,13 +11,11 @@ exports[` spec renders errors 1`] = `
    -
    -

    - Name and description -

    -
    + Name and description +
    spec renders the component 1`] = `
    -
    -

    - Name and description -

    -
    + Name and description +
    spec renders the component 1`] = ` -
    -

    - Create channel -

    -
    + Create channel + `; diff --git a/public/pages/CreateChannel/components/modals/CreateRecipientGroupModal.tsx b/public/pages/CreateChannel/components/modals/CreateRecipientGroupModal.tsx index e3610256..5d7225ec 100644 --- a/public/pages/CreateChannel/components/modals/CreateRecipientGroupModal.tsx +++ b/public/pages/CreateChannel/components/modals/CreateRecipientGroupModal.tsx @@ -13,7 +13,6 @@ import { EuiModalHeader, EuiModalHeaderTitle, EuiOverlayMask, - EuiText, } from '@elastic/eui'; import React, { useContext, useState } from 'react'; import { CoreServicesContext } from '../../../../components/coreServices'; @@ -67,11 +66,7 @@ export function CreateRecipientGroupModal( - - -

    Create recipient group

    -     -     + Create recipient group     diff --git a/public/pages/CreateChannel/components/modals/CreateSESSenderModal.tsx b/public/pages/CreateChannel/components/modals/CreateSESSenderModal.tsx index 0ab58e6d..a674111a 100644 --- a/public/pages/CreateChannel/components/modals/CreateSESSenderModal.tsx +++ b/public/pages/CreateChannel/components/modals/CreateSESSenderModal.tsx @@ -13,7 +13,6 @@ import { EuiModalHeader, EuiModalHeaderTitle, EuiOverlayMask, - EuiText, } from '@elastic/eui'; import React, { useContext, useState } from 'react'; import { CoreServicesContext } from '../../../../components/coreServices'; @@ -71,11 +70,7 @@ export function CreateSESSenderModal(props: CreateSESSenderModalProps) { - - -

    Create SES sender

    -     -     + Create SES sender     diff --git a/public/pages/CreateChannel/components/modals/CreateSenderModal.tsx b/public/pages/CreateChannel/components/modals/CreateSenderModal.tsx index 7801aa18..9fe19221 100644 --- a/public/pages/CreateChannel/components/modals/CreateSenderModal.tsx +++ b/public/pages/CreateChannel/components/modals/CreateSenderModal.tsx @@ -13,7 +13,6 @@ import { EuiModalHeader, EuiModalHeaderTitle, EuiOverlayMask, - EuiText, } from '@elastic/eui'; import React, { useContext, useState } from 'react'; import { CoreServicesContext } from '../../../../components/coreServices'; @@ -69,11 +68,7 @@ export function CreateSenderModal(props: CreateSenderModalProps) { - - -

    Create SMTP sender

    -     -     + Create SMTP sender     diff --git a/public/pages/Emails/CreateRecipientGroup.tsx b/public/pages/Emails/CreateRecipientGroup.tsx index ee325fc0..f492f389 100644 --- a/public/pages/Emails/CreateRecipientGroup.tsx +++ b/public/pages/Emails/CreateRecipientGroup.tsx @@ -10,7 +10,7 @@ import { EuiFlexGroup, EuiFlexItem, EuiSpacer, - EuiText, + EuiTitle, } from '@elastic/eui'; import React, { useContext, useEffect, useState } from 'react'; import { RouteComponentProps } from 'react-router-dom'; @@ -111,9 +111,9 @@ export function CreateRecipientGroup(props: CreateRecipientGroupProps) { return ( <> {!getUseUpdatedUx() && ( - +

    {`${props.edit ? 'Edit' : 'Create'} recipient group`}

    -     + )} diff --git a/public/pages/Emails/CreateSESSender.tsx b/public/pages/Emails/CreateSESSender.tsx index 8b9f84ad..63a6c960 100644 --- a/public/pages/Emails/CreateSESSender.tsx +++ b/public/pages/Emails/CreateSESSender.tsx @@ -9,7 +9,7 @@ import { EuiFlexGroup, EuiFlexItem, EuiSpacer, - EuiText, + EuiTitle, } from '@elastic/eui'; import React, { useContext, useEffect, useState } from 'react'; import { RouteComponentProps } from 'react-router-dom'; @@ -102,9 +102,9 @@ export function CreateSESSender(props: CreateSESSenderProps) { return ( <> {!getUseUpdatedUx() && ( - +

    {`${props.edit ? 'Edit' : 'Create'} SES sender`}

    -     + )} diff --git a/public/pages/Emails/CreateSender.tsx b/public/pages/Emails/CreateSender.tsx index 31c5b282..15492bbd 100644 --- a/public/pages/Emails/CreateSender.tsx +++ b/public/pages/Emails/CreateSender.tsx @@ -9,7 +9,7 @@ import { EuiFlexGroup, EuiFlexItem, EuiSpacer, - EuiText, + EuiTitle, } from '@elastic/eui'; import React, { useContext, useEffect, useState } from 'react'; import { RouteComponentProps } from 'react-router-dom'; @@ -99,9 +99,9 @@ export function CreateSender(props: CreateSenderProps) { return ( <> {!getUseUpdatedUx() && ( - +

    {`${props.edit ? 'Edit' : 'Create'} SMTP sender`}

    -     + )} diff --git a/public/pages/Emails/EmailGroups.tsx b/public/pages/Emails/EmailGroups.tsx index 109e0808..90a5cd2c 100644 --- a/public/pages/Emails/EmailGroups.tsx +++ b/public/pages/Emails/EmailGroups.tsx @@ -3,7 +3,7 @@ * SPDX-License-Identifier: Apache-2.0 */ -import { EuiSpacer, EuiText } from '@elastic/eui'; +import { EuiSpacer, EuiTitle } from '@elastic/eui'; import React, { useContext, useEffect } from 'react'; import { RouteComponentProps } from 'react-router-dom'; import { CoreServicesContext } from '../../components/coreServices'; @@ -29,9 +29,9 @@ export function EmailGroups(props: EmailGroupsProps) { return ( <> {!getUseUpdatedUx() && ( - +

    Email recipient groups

    -     + )} diff --git a/public/pages/Emails/EmailSenders.tsx b/public/pages/Emails/EmailSenders.tsx index 05e7beb9..1c50387a 100644 --- a/public/pages/Emails/EmailSenders.tsx +++ b/public/pages/Emails/EmailSenders.tsx @@ -3,7 +3,7 @@ * SPDX-License-Identifier: Apache-2.0 */ -import { EuiSpacer, EuiText } from '@elastic/eui'; +import { EuiSpacer, EuiTitle } from '@elastic/eui'; import React, { useContext, useEffect } from 'react'; import { RouteComponentProps } from 'react-router-dom'; import { CoreServicesContext } from '../../components/coreServices'; @@ -25,7 +25,7 @@ export function EmailSenders(props: EmailSendersProps) { // Call the hook to manage URL updates useUpdateUrlWithDataSourceProperties(); - + useEffect(() => { setBreadcrumbs([ BREADCRUMBS.NOTIFICATIONS, @@ -37,9 +37,9 @@ export function EmailSenders(props: EmailSendersProps) { return ( <> {!getUseUpdatedUx() && ( - +

    Email senders

    -     + )} {mainStateContext.availableConfigTypes.includes('smtp_account') && ( <> diff --git a/public/pages/Emails/__tests__/__snapshots__/CreateRecipientGroup.test.tsx.snap b/public/pages/Emails/__tests__/__snapshots__/CreateRecipientGroup.test.tsx.snap index 4009d0b8..3bd7021d 100644 --- a/public/pages/Emails/__tests__/__snapshots__/CreateRecipientGroup.test.tsx.snap +++ b/public/pages/Emails/__tests__/__snapshots__/CreateRecipientGroup.test.tsx.snap @@ -1,11 +1,9 @@ // Jest Snapshot v1, https://goo.gl/fbAQLP exports[` spec renders the component 1`] = ` -
    -

    - Create recipient group -

    -
    + Create recipient group + `; diff --git a/public/pages/Emails/__tests__/__snapshots__/CreateSESSender.test.tsx.snap b/public/pages/Emails/__tests__/__snapshots__/CreateSESSender.test.tsx.snap index 97b70161..30e8145c 100644 --- a/public/pages/Emails/__tests__/__snapshots__/CreateSESSender.test.tsx.snap +++ b/public/pages/Emails/__tests__/__snapshots__/CreateSESSender.test.tsx.snap @@ -1,31 +1,25 @@ // Jest Snapshot v1, https://goo.gl/fbAQLP exports[` spec handles failed requests 1`] = ` -
    -

    - Edit SES sender -

    -
    + Edit SES sender + `; exports[` spec renders the component 1`] = ` -
    -

    - Create SES sender -

    -
    + Create SES sender + `; exports[` spec renders the component for editing 1`] = ` -
    -

    - Edit SES sender -

    -
    + Edit SES sender + `; diff --git a/public/pages/Emails/__tests__/__snapshots__/CreateSender.test.tsx.snap b/public/pages/Emails/__tests__/__snapshots__/CreateSender.test.tsx.snap index 68449f39..b6ff392a 100644 --- a/public/pages/Emails/__tests__/__snapshots__/CreateSender.test.tsx.snap +++ b/public/pages/Emails/__tests__/__snapshots__/CreateSender.test.tsx.snap @@ -1,31 +1,25 @@ // Jest Snapshot v1, https://goo.gl/fbAQLP exports[` spec handles failed requests 1`] = ` -
    -

    - Edit SMTP sender -

    -
    + Edit SMTP sender + `; exports[` spec renders the component 1`] = ` -
    -

    - Create SMTP sender -

    -
    + Create SMTP sender + `; exports[` spec renders the component for editing 1`] = ` -
    -

    - Edit SMTP sender -

    -
    + Edit SMTP sender + `; diff --git a/public/pages/Emails/__tests__/__snapshots__/EmailGroups.test.tsx.snap b/public/pages/Emails/__tests__/__snapshots__/EmailGroups.test.tsx.snap index a4b40b5b..ed8880f6 100644 --- a/public/pages/Emails/__tests__/__snapshots__/EmailGroups.test.tsx.snap +++ b/public/pages/Emails/__tests__/__snapshots__/EmailGroups.test.tsx.snap @@ -1,11 +1,9 @@ // Jest Snapshot v1, https://goo.gl/fbAQLP exports[` spec renders the component 1`] = ` -
    -

    - Email recipient groups -

    -
    + Email recipient groups + `; diff --git a/public/pages/Emails/__tests__/__snapshots__/EmailSenders.test.tsx.snap b/public/pages/Emails/__tests__/__snapshots__/EmailSenders.test.tsx.snap index 00e42cc1..1483fdaa 100644 --- a/public/pages/Emails/__tests__/__snapshots__/EmailSenders.test.tsx.snap +++ b/public/pages/Emails/__tests__/__snapshots__/EmailSenders.test.tsx.snap @@ -1,21 +1,17 @@ // Jest Snapshot v1, https://goo.gl/fbAQLP exports[` spec renders the component with SMTP config type 1`] = ` -
    -

    - Email senders -

    -
    + Email senders + `; exports[` spec renders the component without SMTP config type 1`] = ` -
    -

    - Email senders -

    -
    + Email senders + `; diff --git a/public/pages/Emails/__tests__/__snapshots__/RecipientGroupsTable.test.tsx.snap b/public/pages/Emails/__tests__/__snapshots__/RecipientGroupsTable.test.tsx.snap index fb2a223c..ebb37281 100644 --- a/public/pages/Emails/__tests__/__snapshots__/RecipientGroupsTable.test.tsx.snap +++ b/public/pages/Emails/__tests__/__snapshots__/RecipientGroupsTable.test.tsx.snap @@ -11,18 +11,16 @@ exports[` spec renders empty state 1`] = `
    -
    -

    - Recipient groups - - (0) - -

    -
    + Recipient groups + + (0) + +
    spec renders empty state 1`] = `
    -
    -

    - No recipient groups to display -

    -
    + No recipient groups to display + @@ -386,11 +382,7 @@ exports[` spec renders empty state 1`] = `
    -
    - Use an email group to manage a list of email addresses you frequently send at a time. You can select recipient groups when configuring email channels. -
    + Use an email group to manage a list of email addresses you frequently send at a time. You can select recipient groups when configuring email channels.
    spec renders empty state 1`] = `
    -
    -

    - SMTP senders - - (0) - -

    -
    + SMTP senders + + (0) + +
    spec renders empty state 1`] = `
    -
    -

    - No SMTP senders to display -

    -
    + No SMTP senders to display + @@ -488,11 +484,7 @@ exports[` spec renders empty state 1`] = `
    -
    - Set up an outbound email server by creating a sender. You will select a sender when configuring email channels. -
    + Set up an outbound email server by creating a sender. You will select a sender when configuring email channels.
    - - -

    {`Delete ${name}?`}

    -     -     + {`Delete ${name}?`}     - {message} + {message} {num >= 2 && ( <> @@ -72,7 +68,7 @@ export const DeleteRecipientGroupModal = ( )} - + To confirm delete, type delete in the field. { - - -

    {`Delete ${name}?`}

    -     -     + {`Delete ${name}?`}     - {message} + {message} {num >= 2 && ( <> @@ -62,7 +58,6 @@ export const DeleteSenderModal = (props: DeleteSenderModalProps) => {
  • {sender.name}
    •  @@ -70,7 +65,7 @@ export const DeleteSenderModal = (props: DeleteSenderModalProps) => { )} - + To confirm delete, type delete in the field.

    No recipient groups to display

    } - body={ - - Use an email group to manage a list of email addresses you frequently send at a time. You can select recipient groups when configuring email channels. - - } + title={

    No recipient groups to display

    } + body="Use an email group to manage a list of email addresses you frequently send at a time. You can select recipient groups when configuring email channels." actions={ Create recipient group } />} @@ -335,7 +330,6 @@ export class RecipientGroupsTable extends Component< { this.setState({ isPopoverOpen: false }); if (action.modal) { @@ -407,7 +401,7 @@ export class RecipientGroupsTable extends Component< } bodyStyles={{ padding: 'initial' }} title="Recipient groups" - titleSize="s" + titleSize="m" total={this.state.total} > {searchComponent} diff --git a/public/pages/Emails/components/tables/SESSendersTable.tsx b/public/pages/Emails/components/tables/SESSendersTable.tsx index 2b5059b7..aed3287b 100644 --- a/public/pages/Emails/components/tables/SESSendersTable.tsx +++ b/public/pages/Emails/components/tables/SESSendersTable.tsx @@ -16,7 +16,6 @@ import { EuiTableFieldDataColumnType, EuiTableSortingType, SortDirection, - EuiText, } from '@elastic/eui'; import { Criteria } from '@elastic/eui/src/components/basic_table/basic_table'; import { Pagination } from '@elastic/eui/src/components/basic_table/pagination_bar'; @@ -224,8 +223,8 @@ export class SESSendersTable extends Component< isSelectable={true} selection={selection} noItemsMessage={

    No SES senders to display

    } - body={Set up an outbound email server by creating a sender. You will select a sender when configuring email channels.} + title={

    No SES senders to display

    } + body="Set up an outbound email server by creating a sender. You will select a sender when configuring email channels." actions={ Create SES sender } />} @@ -260,7 +259,7 @@ export class SESSendersTable extends Component< } bodyStyles={{ padding: 'initial' }} title="SES senders" - titleSize="s" + titleSize="m" total={this.state.total} > @@ -289,7 +288,6 @@ export class SESSendersTable extends Component< { this.setState({ isPopoverOpen: false }); if (action.modal) { @@ -362,7 +360,7 @@ export class SESSendersTable extends Component< } bodyStyles={{ padding: 'initial' }} title="SES senders" - titleSize="s" + titleSize="m" total={this.state.total} > {searchComponent} diff --git a/public/pages/Emails/components/tables/SendersTable.tsx b/public/pages/Emails/components/tables/SendersTable.tsx index dba914f5..efee0179 100644 --- a/public/pages/Emails/components/tables/SendersTable.tsx +++ b/public/pages/Emails/components/tables/SendersTable.tsx @@ -15,7 +15,6 @@ import { EuiTableFieldDataColumnType, EuiTableSortingType, SortDirection, - EuiText, } from '@elastic/eui'; import { Criteria } from '@elastic/eui/src/components/basic_table/basic_table'; import { Pagination } from '@elastic/eui/src/components/basic_table/pagination_bar'; @@ -251,12 +250,8 @@ export class SendersTable extends Component< isSelectable={true} selection={selection} noItemsMessage={

    No SMTP senders to display

    } - body={ - - Set up an outbound email server by creating a sender. You will select a sender when configuring email channels. - - } + title={

    No SMTP senders to display

    } + body="Set up an outbound email server by creating a sender. You will select a sender when configuring email channels." actions={ Create SMTP sender } />} @@ -284,7 +279,7 @@ export class SendersTable extends Component< } bodyStyles={{ padding: 'initial' }} title="SMTP senders" - titleSize="s" + titleSize="m" total={this.state.total} > @@ -313,7 +308,6 @@ export class SendersTable extends Component< { this.setState({ isPopoverOpen: false }); if (action.modal) { @@ -386,7 +380,7 @@ export class SendersTable extends Component< } bodyStyles={{ padding: 'initial' }} title="SMTP senders" - titleSize="s" + titleSize="m" total={this.state.total} > {senderControlComponent} diff --git a/public/pages/Main/__tests__/__snapshots__/Main.test.tsx.snap b/public/pages/Main/__tests__/__snapshots__/Main.test.tsx.snap index 49182f1a..9f5714fe 100644 --- a/public/pages/Main/__tests__/__snapshots__/Main.test.tsx.snap +++ b/public/pages/Main/__tests__/__snapshots__/Main.test.tsx.snap @@ -116,18 +116,16 @@ exports[`
    spec renders the component 1`] = `
    -
    -

    - Channels - - (0) - -

    -
    + Channels + + (0) + +
    spec renders the component 1`] = `
    -
    -

    - No channels to display -

    -
    + No channels to display + @@ -613,11 +609,7 @@ exports[`
    spec renders the component 1`] = `
    -
    - "To send or receive notifications, you will need to create a notification channel." -
    + To send or receive notifications, you will need to create a notification channel.