Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[EuiDescriptionList] Update display to grid for vertical column option with new props #7062

Merged
merged 56 commits into from
Aug 28, 2023
Merged
Show file tree
Hide file tree
Changes from 50 commits
Commits
Show all changes
56 commits
Select commit Hold shift + click to select a range
58ab7cc
add display grid
kyrspl Aug 9, 2023
1fd6bbd
add eui variables for column gap
kyrspl Aug 9, 2023
040cf19
Merge branch 'main' into descriptionList_update_vertical
kyrspl Aug 9, 2023
9d46054
Merge branch 'main' into descriptionList_update_vertical
kyrspl Aug 10, 2023
28ef754
add total max width
kyrspl Aug 10, 2023
d64db92
Update src/components/description_list/description_list_types.ts
kyrspl Aug 11, 2023
c53c8a8
Merge branch 'main' into descriptionList_update_vertical
kyrspl Aug 11, 2023
9bd8385
update test for column_gap
kyrspl Aug 11, 2023
94c5f68
add changelog
kyrspl Aug 11, 2023
672f564
update documentation
kyrspl Aug 11, 2023
0ee59c5
fix typo
kyrspl Aug 11, 2023
b25bc8f
fix typo in test
kyrspl Aug 11, 2023
64e192c
Merge branch 'main' into descriptionList_update_vertical
kyrspl Aug 12, 2023
c0c9e1f
update test desc
kyrspl Aug 14, 2023
3896845
update keyboard snapshot
kyrspl Aug 14, 2023
f160fc9
Merge branch 'main' into descriptionList_update_vertical
kyrspl Aug 14, 2023
8f2c8ef
Merge branch 'main' into descriptionList_update_vertical
kyrspl Aug 14, 2023
fd1e4f7
Merge branch 'main' into descriptionList_update_vertical
kyrspl Aug 15, 2023
79cc572
fix typos
kyrspl Aug 15, 2023
8a59812
Update src-docs/src/views/description_list/description_list_example.js
kyrspl Aug 15, 2023
db9e57a
Update src-docs/src/views/description_list/description_list_example.js
kyrspl Aug 15, 2023
dde01e8
[PR feedback] Conditionally render CSS that only applies to `display:…
cee-chen Aug 15, 2023
f8faa78
[PR feedback] Remove unnecessary mobile breakpoint CSS
cee-chen Aug 15, 2023
c11b79f
[PR feedback] Remove `row-gap` CSS
cee-chen Aug 15, 2023
2bd11ee
[PR feedback] Remove `columnGap` from context
cee-chen Aug 15, 2023
d35e84e
prettier lint fix
cee-chen Aug 15, 2023
1028d81
Update downstream snapshots
cee-chen Aug 15, 2023
2c7e070
Merge branch 'main' into descriptionList_update_vertical
kyrspl Aug 16, 2023
507139a
refactor final
kyrspl Aug 21, 2023
fc12fe6
update context
kyrspl Aug 21, 2023
4ec06df
update tests
kyrspl Aug 21, 2023
a38d4c4
Merge branch 'main' into descriptionList_update_vertical
kyrspl Aug 21, 2023
d3d8416
Merge branch 'main' into descriptionList_update_vertical
kyrspl Aug 22, 2023
56a791d
update tests
kyrspl Aug 22, 2023
5311a63
Merge branch 'main' into descriptionList_update_vertical
kyrspl Aug 22, 2023
521b4d7
Fix lint complaint
cee-chen Aug 22, 2023
a78ec7a
CSS cleanup
cee-chen Aug 22, 2023
3e1ad21
[REFACTOR] Remove `responsiveColumn` CSS entirely, and instead use JS…
cee-chen Aug 22, 2023
0f462d5
[REFACTOR] Remove `responsiveColumns` CSS & logic from child components
cee-chen Aug 22, 2023
7388ea2
[refactor] update child types and tests
cee-chen Aug 22, 2023
0624a3c
misc newline nits
cee-chen Aug 22, 2023
95641d0
Fix row/column gap tests
cee-chen Aug 22, 2023
06ba99a
[BREAKING CHANGE] Prop renames
cee-chen Aug 22, 2023
3e4d1f1
Update datagrid usage
cee-chen Aug 22, 2023
e4669e5
Merge branch 'main' into descriptionList_update_vertical
kyrspl Aug 23, 2023
c1f74d3
Merge branch 'main' into descriptionList_update_vertical
kyrspl Aug 23, 2023
5710c75
Update src/components/description_list/description_list_title.styles.ts
kyrspl Aug 23, 2023
2b6a0cb
Update src/components/description_list/description_list_title.styles.ts
kyrspl Aug 23, 2023
3f49cad
Update upcoming_changelogs/7062.md
kyrspl Aug 23, 2023
eae95f4
[docs] Fix prop names
cee-chen Aug 23, 2023
024eb73
Update src/components/description_list/description_list_title.styles.ts
kyrspl Aug 24, 2023
4808e81
removed max-inline-size
kyrspl Aug 24, 2023
0f4bed3
changelog
cee-chen Aug 25, 2023
66354b7
Merge branch 'main' into descriptionList_update_vertical
kyrspl Aug 25, 2023
a1ffc4b
Merge branch 'main' into descriptionList_update_vertical
kyrspl Aug 28, 2023
a7a5f7c
Merge branch 'main' into descriptionList_update_vertical
kyrspl Aug 28, 2023
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ const favoriteVideoGames = [
{
title: 'TIE Fighter',
description:
'The sequel to XWING, join the dark side and fly for the Emporer.',
'The sequel to XWING, join the dark side and fly for the Emperor.',
},
{
title: 'Quake 2',
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ const favoriteVideoGames = [
{
title: 'TIE Fighter',
description:
'The sequel to XWING, join the dark side and fly for the Emporer.',
'The sequel to XWING, join the dark side and fly for the Emperor.',
cee-chen marked this conversation as resolved.
Show resolved Hide resolved
},
{
title: 'Quake 2',
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ const favoriteVideoGames = [
{
title: 'TIE Fighter',
description:
'The sequel to XWING, join the dark side and fly for the Emporer.',
'The sequel to XWING, join the dark side and fly for the Emperor.',
},
{
title: 'Quake 2',
Expand Down
28 changes: 21 additions & 7 deletions src-docs/src/views/description_list/description_list_example.js
kyrspl marked this conversation as resolved.
Show resolved Hide resolved
Original file line number Diff line number Diff line change
Expand Up @@ -239,13 +239,27 @@ export const DescriptionListExample = {
},
],
text: (
<p>
Using the <EuiCode>align</EuiCode> and <EuiCode>compressed</EuiCode>{' '}
props you can further tailor the look of a description list. This
works with column and inline types. You can also adjust the{' '}
<EuiCode>gutterSize</EuiCode> prop to increase and decrease vertical
spacing between <EuiCode>EuiDescriptionList</EuiCode> elements.
</p>
<>
<h4>Alignment & Compression</h4>
<p>
Utilize the <EuiCode>align</EuiCode> and
<EuiCode>compressed</EuiCode> props to refine the presentation of
your description list. These are compatible with both column and
inline types.
</p>
<h4>Vertical spacing</h4>
<p>
Modify the <EuiCode>rowGutterSize</EuiCode> prop to control the
vertical spacing between <EuiCode>EuiDescriptionList</EuiCode>{' '}
elements. This will not affect inline list types.
</p>
<h4>Horizontal spacing</h4>
<p>
Adjust the spacing between the title and description with the{' '}
<EuiCode>columnGutterSize</EuiCode> prop. This will not affect
inline or row types.
</p>
</>
),
snippet: descriptionListStylingSnippet,
demo: <DescriptionListStyling />,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ const favoriteVideoGames = [
{
title: 'TIE Fighter',
description:
'The sequel to XWING, join the dark side and fly for the Emporer.',
'The sequel to XWING, join the dark side and fly for the Emperor.',
},
{
title: 'Quake 2',
Expand Down
52 changes: 35 additions & 17 deletions src-docs/src/views/description_list/description_list_styling.js
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ export default () => {
{
title: 'TIE Fighter',
description:
'The sequel to XWING, join the dark side and fly for the Emporer.',
'The sequel to XWING, join the dark side and fly for the Emperor.',
},
{
title: 'Quake 2',
Expand All @@ -40,11 +40,7 @@ export default () => {

const [alignSelected, setAlignSelected] = useState('center');

const alignOnChange = (id) => {
setAlignSelected(id);
};

const gutterToggleButtons = [
const rowGutterSizeOptions = [
{
id: 's',
label: 'Small',
Expand All @@ -55,11 +51,20 @@ export default () => {
},
];

const [gutterSelected, setGutterSelected] = useState('m');
const [rowGutterSize, setRowGutterSize] = useState('m');

const gutterOnChange = (id) => {
setGutterSelected(id);
};
const columnGutterSizeOptions = [
{
id: 's',
label: 'Small',
},
{
id: 'm',
label: 'Medium',
},
];

const [columnGutterSize, setColumnGutterSize] = useState('m');

const [compressed, setCompressed] = useState(true);

Expand All @@ -78,7 +83,7 @@ export default () => {
legend="Toggle for the EuiDescription align prop"
options={alignToggleButtons}
idSelected={alignSelected}
onChange={(id) => alignOnChange(id)}
onChange={(id) => setAlignSelected(id)}
/>
</EuiFlexItem>

Expand All @@ -87,10 +92,22 @@ export default () => {
<h3>Row gutter sizes</h3>
</EuiTitle>
<EuiButtonGroup
legend="Toggle for the EuiDescription gutterSize prop"
options={gutterToggleButtons}
idSelected={gutterSelected}
onChange={(id) => gutterOnChange(id)}
legend="Toggle for the EuiDescription rowGutterSize prop"
options={rowGutterSizeOptions}
idSelected={rowGutterSize}
onChange={(id) => setRowGutterSize(id)}
/>
</EuiFlexItem>

<EuiFlexItem>
<EuiTitle size="xxs">
<h3>Column gap sizes</h3>
</EuiTitle>
<EuiButtonGroup
legend="Toggle for the EuiDescription columnGutterSize prop"
options={columnGutterSizeOptions}
idSelected={columnGutterSize}
onChange={(id) => setColumnGutterSize(id)}
/>
</EuiFlexItem>

Expand All @@ -112,7 +129,7 @@ export default () => {
listItems={favoriteVideoGames}
align={alignSelected}
compressed={compressed}
gutterSize={gutterSelected}
rowGutterSize={rowGutterSize}
/>

<EuiSpacer size="l" />
Expand All @@ -122,7 +139,8 @@ export default () => {
type="column"
align={alignSelected}
compressed={compressed}
gutterSize={gutterSelected}
rowGutterSize={rowGutterSize}
columnGutterSize={columnGutterSize}
/>

<EuiSpacer size="l" />
Expand Down
2 changes: 1 addition & 1 deletion src-docs/src/views/description_list/playground.js
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ export const descriptionListConfig = () => {
{
title: 'TIE Fighter',
description:
'The sequel to XWING, join the dark side and fly for the Emporer.',
'The sequel to XWING, join the dark side and fly for the Emperor.',
},
]`;
propsToUse.listItems = {
Expand Down
2 changes: 1 addition & 1 deletion src-docs/src/views/text/text.js
Original file line number Diff line number Diff line change
Expand Up @@ -169,7 +169,7 @@ export default () => (
<dd>The opening music alone evokes such strong memories.</dd>
<dt>TIE Fighter</dt>
<dd>
The sequel to XWING, join the dark side and fly for the Emporer.
The sequel to XWING, join the dark side and fly for the Emperor.
</dd>
<dt>Quake 2</dt>
<dd>The game that made me drop out of college.</dd>
Expand Down
2 changes: 1 addition & 1 deletion src-docs/src/views/text/text_small.js
Original file line number Diff line number Diff line change
Expand Up @@ -94,7 +94,7 @@ const exampleText = (
<dt>The Elder Scrolls: Morrowind</dt>
<dd>The opening music alone evokes such strong memories.</dd>
<dt>TIE Fighter</dt>
<dd>The sequel to XWING, join the dark side and fly for the Emporer.</dd>
<dd>The sequel to XWING, join the dark side and fly for the Emperor.</dd>
<dt>Quake 2</dt>
<dd>The game that made me drop out of college.</dd>
</dl>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -84,83 +84,83 @@ exports[`useDataGridKeyboardShortcuts returns a popover containing a list of key
>
<dl
aria-labelledby="generated-id"
class="euiDescriptionList emotion-euiDescriptionList-column-center"
class="euiDescriptionList emotion-euiDescriptionList-column-center-s-s"
data-type="column"
>
<dt
class="euiDescriptionList__title emotion-euiDescriptionList__title-column-s-compressed-right"
class="euiDescriptionList__title emotion-euiDescriptionList__title-column-compressed-right"
>
<kbd>
Up arrow
</kbd>
</dt>
<dd
class="euiDescriptionList__description emotion-euiDescriptionList__description-column-normal-left-s"
class="euiDescriptionList__description emotion-euiDescriptionList__description-column-normal-left"
>
Move one cell up
</dd>
<dt
class="euiDescriptionList__title emotion-euiDescriptionList__title-column-s-compressed-right"
class="euiDescriptionList__title emotion-euiDescriptionList__title-column-compressed-right"
>
<kbd>
Down arrow
</kbd>
</dt>
<dd
class="euiDescriptionList__description emotion-euiDescriptionList__description-column-normal-left-s"
class="euiDescriptionList__description emotion-euiDescriptionList__description-column-normal-left"
>
Move one cell down
</dd>
<dt
class="euiDescriptionList__title emotion-euiDescriptionList__title-column-s-compressed-right"
class="euiDescriptionList__title emotion-euiDescriptionList__title-column-compressed-right"
>
<kbd>
Right arrow
</kbd>
</dt>
<dd
class="euiDescriptionList__description emotion-euiDescriptionList__description-column-normal-left-s"
class="euiDescriptionList__description emotion-euiDescriptionList__description-column-normal-left"
>
Move one cell right
</dd>
<dt
class="euiDescriptionList__title emotion-euiDescriptionList__title-column-s-compressed-right"
class="euiDescriptionList__title emotion-euiDescriptionList__title-column-compressed-right"
>
<kbd>
Left arrow
</kbd>
</dt>
<dd
class="euiDescriptionList__description emotion-euiDescriptionList__description-column-normal-left-s"
class="euiDescriptionList__description emotion-euiDescriptionList__description-column-normal-left"
>
Move one cell left
</dd>
<dt
class="euiDescriptionList__title emotion-euiDescriptionList__title-column-s-compressed-right"
class="euiDescriptionList__title emotion-euiDescriptionList__title-column-compressed-right"
>
<kbd>
Home
</kbd>
</dt>
<dd
class="euiDescriptionList__description emotion-euiDescriptionList__description-column-normal-left-s"
class="euiDescriptionList__description emotion-euiDescriptionList__description-column-normal-left"
>
Move to the first cell of the current row
</dd>
<dt
class="euiDescriptionList__title emotion-euiDescriptionList__title-column-s-compressed-right"
class="euiDescriptionList__title emotion-euiDescriptionList__title-column-compressed-right"
>
<kbd>
End
</kbd>
</dt>
<dd
class="euiDescriptionList__description emotion-euiDescriptionList__description-column-normal-left-s"
class="euiDescriptionList__description emotion-euiDescriptionList__description-column-normal-left"
>
Move to the last cell of the current row
</dd>
<dt
class="euiDescriptionList__title emotion-euiDescriptionList__title-column-s-compressed-right"
class="euiDescriptionList__title emotion-euiDescriptionList__title-column-compressed-right"
>
<kbd>
Ctrl
Expand All @@ -171,12 +171,12 @@ exports[`useDataGridKeyboardShortcuts returns a popover containing a list of key
</kbd>
</dt>
<dd
class="euiDescriptionList__description emotion-euiDescriptionList__description-column-normal-left-s"
class="euiDescriptionList__description emotion-euiDescriptionList__description-column-normal-left"
>
Move to the first cell of the current page
</dd>
<dt
class="euiDescriptionList__title emotion-euiDescriptionList__title-column-s-compressed-right"
class="euiDescriptionList__title emotion-euiDescriptionList__title-column-compressed-right"
>
<kbd>
Ctrl
Expand All @@ -187,55 +187,55 @@ exports[`useDataGridKeyboardShortcuts returns a popover containing a list of key
</kbd>
</dt>
<dd
class="euiDescriptionList__description emotion-euiDescriptionList__description-column-normal-left-s"
class="euiDescriptionList__description emotion-euiDescriptionList__description-column-normal-left"
>
Move to the last cell of the current page
</dd>
<dt
class="euiDescriptionList__title emotion-euiDescriptionList__title-column-s-compressed-right"
class="euiDescriptionList__title emotion-euiDescriptionList__title-column-compressed-right"
>
<kbd>
Page Up
</kbd>
</dt>
<dd
class="euiDescriptionList__description emotion-euiDescriptionList__description-column-normal-left-s"
class="euiDescriptionList__description emotion-euiDescriptionList__description-column-normal-left"
>
Go to the last row of the previous page
</dd>
<dt
class="euiDescriptionList__title emotion-euiDescriptionList__title-column-s-compressed-right"
class="euiDescriptionList__title emotion-euiDescriptionList__title-column-compressed-right"
>
<kbd>
Page Down
</kbd>
</dt>
<dd
class="euiDescriptionList__description emotion-euiDescriptionList__description-column-normal-left-s"
class="euiDescriptionList__description emotion-euiDescriptionList__description-column-normal-left"
>
Go to the first row of the next page
</dd>
<dt
class="euiDescriptionList__title emotion-euiDescriptionList__title-column-s-compressed-right"
class="euiDescriptionList__title emotion-euiDescriptionList__title-column-compressed-right"
>
<kbd>
Enter
</kbd>
</dt>
<dd
class="euiDescriptionList__description emotion-euiDescriptionList__description-column-normal-left-s"
class="euiDescriptionList__description emotion-euiDescriptionList__description-column-normal-left"
>
Open cell details and actions
</dd>
<dt
class="euiDescriptionList__title emotion-euiDescriptionList__title-column-s-compressed-right"
class="euiDescriptionList__title emotion-euiDescriptionList__title-column-compressed-right"
>
<kbd>
Escape
</kbd>
</dt>
<dd
class="euiDescriptionList__description emotion-euiDescriptionList__description-column-normal-left-s"
class="euiDescriptionList__description emotion-euiDescriptionList__description-column-normal-left"
>
Close cell details and actions
</dd>
Expand Down
1 change: 0 additions & 1 deletion src/components/datagrid/controls/keyboard_shortcuts.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -53,7 +53,6 @@ export const useDataGridKeyboardShortcuts = (): {
type="column"
align="center"
compressed
gutterSize="s"
listItems={[
{
title: (
Expand Down
Loading
Loading