Skip to content

Commit

Permalink
Revert "[Fit & Finish] Updated Fit and Finish guidelines (opensearch-…
Browse files Browse the repository at this point in the history 
…project#256) (opensearch-project#259)"

This reverts commit fb48dbc.

Signed-off-by: Joanne Wang <[email protected]>
  • Loading branch information
@jowg-amazon
jowg-amazon committed Sep 24, 2024
1 parent 6f68631 commit dd92b6f
Show file tree
Hide file tree
Showing 38 changed files with 222 additions and 476 deletions.
14 changes: 7 additions & 7 deletions public/components/ContentPanel/ContentPanel.tsx
View file
Original file line number Diff line number Diff line change
Expand Up @@ -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;
Expand All @@ -25,7 +25,7 @@ interface ContentPanelProps {

const ContentPanel: React.SFC<ContentPanelProps> = ({
title = '',
titleSize = 's',
titleSize = 'l',
total = undefined,
bodyStyles = {},
panelStyles = {},
Expand All @@ -40,16 +40,16 @@ const ContentPanel: React.SFC<ContentPanelProps> = ({
alignItems="center"
>
<EuiFlexItem>
<EuiText size={titleSize}>
<h2>
<EuiTitle size={titleSize}>
<h3>
{title}
{total !== undefined ? (
<span
style={{ color: '#9f9f9f', fontWeight: 300 }}
>{` (${total})`}</span>
) : null}
</h2>
</EuiText>
</h3>
</EuiTitle>
</EuiFlexItem>
{actions ? (
<EuiFlexItem grow={false}>
Expand Down
20 changes: 8 additions & 12 deletions public/components/__tests__/__snapshots__/ContentPanel.test.tsx.snap
View file
Original file line number Diff line number Diff line change
Expand Up @@ -11,13 +11,11 @@ exports[`<ContentPanel /> spec renders the component 1`] = `
<div
class="euiFlexItem"
>
<div
class="euiText euiText--small"
<h3
class="euiTitle euiTitle--large"
>
<h2>
Testing
</h2>
</div>
Testing
</h3>
</div>
<div
class="euiFlexItem euiFlexItem--flexGrowZero"
Expand Down Expand Up @@ -59,13 +57,11 @@ exports[`<ContentPanel /> spec renders with empty actions 1`] = `
<div
class="euiFlexItem"
>
<div
class="euiText euiText--small"
<h3
class="euiTitle euiTitle--large"
>
<h2>
Testing
</h2>
</div>
Testing
</h3>
</div>
</div>
<hr
Expand Down
9 changes: 4 additions & 5 deletions public/pages/Channels/Channels.tsx
View file
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,6 @@ import {
EuiTableSortingType,
EuiTitle,
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';
Expand Down Expand Up @@ -258,8 +257,8 @@ export class Channels extends MDSEnabledComponent<ChannelsProps, ChannelsState>
isSelectable={true}
selection={selection}
noItemsMessage={<EuiEmptyPrompt
title={<EuiText size="s"><h2>No channels to display</h2></EuiText>}
body={<EuiText size="s">"To send or receive notifications, you will need to create a notification channel."</EuiText>}
title={<h2>No channels to display</h2>}
body="To send or receive notifications, you will need to create a notification channel."
actions={<EuiSmallButton href={`#${ROUTES.CREATE_CHANNEL}`}>
Create channel
</EuiSmallButton>} />}
Expand Down Expand Up @@ -310,7 +309,7 @@ export class Channels extends MDSEnabledComponent<ChannelsProps, ChannelsState>
}
bodyStyles={{ padding: 'initial' }}
title="Channels"
titleSize="s"
titleSize="m"
total={this.state.total}
>
{channelControlsComponent}
Expand All @@ -319,7 +318,7 @@ export class Channels extends MDSEnabledComponent<ChannelsProps, ChannelsState>
</ContentPanel>
)}
</>

);
}
};
Expand Down
70 changes: 28 additions & 42 deletions public/pages/Channels/__tests__/__snapshots__/ChannelDetails.test.tsx.snap
View file
Original file line number Diff line number Diff line change
Expand Up @@ -15,13 +15,11 @@ exports[`<ChannelDetails/> spec handles a non-existing channel 1`] = `
<div
class="euiFlexItem euiFlexItem--flexGrowZero"
>
<div
class="euiText euiText--small"
<h1
class="euiTitle euiTitle--large"
>
<h1>
-
</h1>
</div>
-
</h1>
</div>
<div
class="euiFlexItem euiFlexItem--flexGrowZero"
Expand Down Expand Up @@ -57,13 +55,11 @@ exports[`<ChannelDetails/> spec handles a non-existing channel 1`] = `
<div
class="euiFlexItem"
>
<div
class="euiText euiText--small"
<h3
class="euiTitle euiTitle--small"
>
<h2>
Name and description
</h2>
</div>
Name and description
</h3>
</div>
</div>
<hr
Expand Down Expand Up @@ -157,13 +153,11 @@ exports[`<ChannelDetails/> spec handles a non-existing channel 1`] = `
<div
class="euiFlexItem"
>
<div
class="euiText euiText--small"
<h3
class="euiTitle euiTitle--small"
>
<h2>
Configurations
</h2>
</div>
Configurations
</h3>
</div>
</div>
<hr
Expand Down Expand Up @@ -191,13 +185,11 @@ exports[`<ChannelDetails/> spec renders a specific channel 1`] = `
<div
class="euiFlexItem euiFlexItem--flexGrowZero"
>
<div
class="euiText euiText--small"
<h1
class="euiTitle euiTitle--large"
>
<h1>
-
</h1>
</div>
-
</h1>
</div>
<div
class="euiFlexItem euiFlexItem--flexGrowZero"
Expand Down Expand Up @@ -233,13 +225,11 @@ exports[`<ChannelDetails/> spec renders a specific channel 1`] = `
<div
class="euiFlexItem"
>
<div
class="euiText euiText--small"
<h3
class="euiTitle euiTitle--small"
>
<h2>
Name and description
</h2>
</div>
Name and description
</h3>
</div>
</div>
<hr
Expand Down Expand Up @@ -333,13 +323,11 @@ exports[`<ChannelDetails/> spec renders a specific channel 1`] = `
<div
class="euiFlexItem"
>
<div
class="euiText euiText--small"
<h3
class="euiTitle euiTitle--small"
>
<h2>
Configurations
</h2>
</div>
Configurations
</h3>
</div>
</div>
<hr
Expand All @@ -366,13 +354,11 @@ exports[`<ChannelDetails/> spec renders the component 1`] = `
<div
class="euiFlexItem euiFlexItem--flexGrowZero"
>
<div
class="euiText euiText--small"
<h1
class="euiTitle euiTitle--large"
>
<h1>
-
</h1>
</div>
-
</h1>
</div>
<div
class="euiFlexItem euiFlexItem--flexGrowZero"
Expand Down
36 changes: 14 additions & 22 deletions public/pages/Channels/__tests__/__snapshots__/Channels.test.tsx.snap
View file
Original file line number Diff line number Diff line change
Expand Up @@ -11,18 +11,16 @@ exports[`<Channels/> spec renders the empty component 1`] = `
<div
class="euiFlexItem"
>
<div
class="euiText euiText--small"
<h3
class="euiTitle euiTitle--medium"
>
<h2>
Channels
<span
style="color: rgb(159, 159, 159); font-weight: 300;"
>
(0)
</span>
</h2>
</div>
Channels
<span
style="color: rgb(159, 159, 159); font-weight: 300;"
>
(0)
</span>
</h3>
</div>
<div
class="euiFlexItem euiFlexItem--flexGrowZero"
Expand Down Expand Up @@ -492,13 +490,11 @@ exports[`<Channels/> spec renders the empty component 1`] = `
<div
class="euiEmptyPrompt"
>
<div
class="euiText euiText--small euiTitle euiTitle--medium"
<h2
class="euiTitle euiTitle--medium"
>
<h2>
No channels to display
</h2>
</div>
No channels to display
</h2>
<span
class="euiTextColor euiTextColor--subdued"
>
Expand All @@ -508,11 +504,7 @@ exports[`<Channels/> spec renders the empty component 1`] = `
<div
class="euiText euiText--medium"
>
<div
class="euiText euiText--small"
>
"To send or receive notifications, you will need to create a notification channel."
</div>
To send or receive notifications, you will need to create a notification channel.
</div>
</span>
<div
Expand Down
52 changes: 6 additions & 46 deletions public/pages/Channels/__tests__/__snapshots__/DetailsListModal.test.tsx.snap
View file
Original file line number Diff line number Diff line change
Expand Up @@ -82,13 +82,7 @@ exports[`<DetailsListModal /> spec renders the component 1`] = `
<div
class="euiModalHeader__title"
>
<div
class="euiText euiText--small"
>
<h2>
test header
</h2>
</div>
test header
</div>
</div>
<div
Expand Down Expand Up @@ -319,13 +313,7 @@ exports[`<DetailsListModal /> spec renders the component 1`] = `
<div
class="euiModalHeader__title"
>
<div
class="euiText euiText--small"
>
<h2>
test header
</h2>
</div>
test header
</div>
</div>
<div
Expand Down Expand Up @@ -488,13 +476,7 @@ exports[`<DetailsListModal /> spec renders the component 1`] = `
<div
class="euiModalHeader__title"
>
<div
class="euiText euiText--small"
>
<h2>
test header
</h2>
</div>
test header
</div>
</div>
<div
Expand Down Expand Up @@ -657,13 +639,7 @@ exports[`<DetailsListModal /> spec renders the component 1`] = `
<div
class="euiModalHeader__title"
>
<div
class="euiText euiText--small"
>
<h2>
test header
</h2>
</div>
test header
</div>
</div>
<div
Expand Down Expand Up @@ -814,13 +790,7 @@ exports[`<DetailsListModal /> spec renders the component 1`] = `
<div
class="euiModalHeader__title"
>
<div
class="euiText euiText--small"
>
<h2>
test header
</h2>
</div>
test header
</div>
</div>
<div
Expand Down Expand Up @@ -1025,17 +995,7 @@ exports[`<DetailsListModal /> spec renders the component 1`] = `
<div
className="euiModalHeader__title"
>
<EuiText
size="s"
>
<div
className="euiText euiText--small"
>
<h2>
test header
</h2>
</div>
</EuiText>
test header
</div>
</EuiModalHeaderTitle>
</div>
Expand Down
Loading

0 comments on commit dd92b6f

Please sign in to comment.