Skip to content

Commit

Permalink
Remove wrapping React.Fragment from EuiNavDrawerGroup in navbar to re…
Browse files Browse the repository at this point in the history
…-include tooltips (elastic#48810) (elastic#48919)
  • Loading branch information
chandlerprall authored Oct 22, 2019
1 parent 9640c5a commit ff0ebd5
Showing 1 changed file with 14 additions and 16 deletions.
30 changes: 14 additions & 16 deletions src/core/public/chrome/ui/header/header.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -410,22 +410,20 @@ class HeaderUI extends Component<Props, State> {
defaultMessage: 'Primary',
})}
>
<>
<EuiNavDrawerGroup
listItems={recentLinksArray}
aria-label={i18n.translate('core.ui.recentLinks.screenReaderLabel', {
defaultMessage: 'Recently viewed links, navigation',
})}
/>
<EuiHorizontalRule margin="none" />
<EuiNavDrawerGroup
data-test-subj="navDrawerAppsMenu"
listItems={navLinksArray}
aria-label={i18n.translate('core.ui.primaryNavList.screenReaderLabel', {
defaultMessage: 'Primary navigation links',
})}
/>
</>
<EuiNavDrawerGroup
listItems={recentLinksArray}
aria-label={i18n.translate('core.ui.recentLinks.screenReaderLabel', {
defaultMessage: 'Recently viewed links, navigation',
})}
/>
<EuiHorizontalRule margin="none" />
<EuiNavDrawerGroup
data-test-subj="navDrawerAppsMenu"
listItems={navLinksArray}
aria-label={i18n.translate('core.ui.primaryNavList.screenReaderLabel', {
defaultMessage: 'Primary navigation links',
})}
/>
</EuiNavDrawer>
</header>
);
Expand Down

0 comments on commit ff0ebd5

Please sign in to comment.