Skip to content

Commit

Permalink
[Discover] Use new icons for the sidebar toggle button (#167625)
Browse files Browse the repository at this point in the history
A follow up for #165866

## Summary

This PR replaces icons for the sidebar toggle button.

<img width="200" alt="Screenshot 2023-09-29 at 13 00 39"
src="https://github.com/elastic/kibana/assets/1415710/6ed10562-9a50-48ce-b6b2-030ab7b11e11">
<img width="200" alt="Screenshot 2023-09-29 at 13 00 47"
src="https://github.com/elastic/kibana/assets/1415710/e286e152-a6bb-4b21-a97a-44419757dafb">
  • Loading branch information
jughosta authored Sep 30, 2023
1 parent b03b2fd commit 3550650
Showing 1 changed file with 2 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,6 @@ export const SidebarToggleButton: React.FC<SidebarToggleButtonProps> = ({
buttonSize,
onChange,
}) => {
// TODO: replace with new Eui icons once available
return (
<div data-test-subj={dataTestSubj}>
<IconButtonGroup
Expand All @@ -48,7 +47,7 @@ export const SidebarToggleButton: React.FC<SidebarToggleButtonProps> = ({
label: i18n.translate('unifiedFieldList.fieldListSidebar.expandSidebarButton', {
defaultMessage: 'Show sidebar',
}),
iconType: 'menuRight',
iconType: 'transitionLeftIn',
'data-test-subj': `${dataTestSubj}-expand`,
onClick: () => onChange(false),
},
Expand All @@ -58,7 +57,7 @@ export const SidebarToggleButton: React.FC<SidebarToggleButtonProps> = ({
label: i18n.translate('unifiedFieldList.fieldListSidebar.collapseSidebarButton', {
defaultMessage: 'Hide sidebar',
}),
iconType: 'menuLeft',
iconType: 'transitionLeftOut',
'data-test-subj': `${dataTestSubj}-collapse`,
onClick: () => onChange(true),
},
Expand Down

0 comments on commit 3550650

Please sign in to comment.