Skip to content

Commit

Permalink
chore(chrome): Remove OSD breadcrumb styling and classes
Browse files Browse the repository at this point in the history
Now that the breacrumb styling is updated in OUI v1.2.0

Signed-off-by: Josh Romero <[email protected]>
  • Loading branch information
joshuarrrr committed May 30, 2023
1 parent 0977902 commit 1c2605e
Show file tree
Hide file tree
Showing 4 changed files with 10 additions and 132 deletions.

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

107 changes: 0 additions & 107 deletions src/core/public/chrome/ui/header/header_breadcrumbs.scss

This file was deleted.

11 changes: 1 addition & 10 deletions src/core/public/chrome/ui/header/header_breadcrumbs.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,6 @@ interface Props {
export function HeaderBreadcrumbs({ appTitle$, breadcrumbs$, isDarkMode }: Props) {
const appTitle = useObservable(appTitle$, 'OpenSearch Dashboards');
const breadcrumbs = useObservable(breadcrumbs$, []);
const className = isDarkMode ? 'osdHeaderBreadcrumbs--dark' : 'osdHeaderBreadcrumbs';
let crumbs = breadcrumbs;

if (breadcrumbs.length === 0 && appTitle) {
Expand All @@ -61,15 +60,7 @@ export function HeaderBreadcrumbs({ appTitle$, breadcrumbs$, isDarkMode }: Props
i === 0 && 'first',
i === breadcrumbs.length - 1 && 'last'
),
className: classNames('osdBreadcrumbs'),
}));

return (
<EuiHeaderBreadcrumbs
breadcrumbs={crumbs}
max={10}
data-test-subj="breadcrumbs"
className={className}
/>
);
return <EuiHeaderBreadcrumbs breadcrumbs={crumbs} max={10} data-test-subj="breadcrumbs" />;
}

0 comments on commit 1c2605e

Please sign in to comment.