Skip to content

Commit

Permalink
change(breadcrumb): removed padding from breadcrumb added by container
Browse files Browse the repository at this point in the history
- Fix home icon missing a divider in the breadcrumb story
  • Loading branch information
ichim-david committed May 12, 2022
1 parent 134da52 commit c5df567
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 16 deletions.
3 changes: 3 additions & 0 deletions src/ui/Breadcrumbs/Breadcrumb.stories.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -45,6 +45,9 @@ const Template = (args) => (
<Breadcrumb.Section key="home" href={args.root}>
<Image src={homeSVG} alt="home" />
</Breadcrumb.Section>
<Breadcrumb.Divider>
<Icon className="ri-arrow-right-s-line"></Icon>
</Breadcrumb.Divider>
{args.sections.map((section, index, sections) => [
index !== 0 && (
<Breadcrumb.Divider key={index}>
Expand Down
16 changes: 0 additions & 16 deletions theme/themes/eea/collections/breadcrumb.overrides
Original file line number Diff line number Diff line change
Expand Up @@ -5,25 +5,10 @@
.ui.breadcrumb {
display: flex;
align-items: center;
padding-right: 1rem;
padding-left: 1rem;

//a.section {
// color: #0074a3;
//}
//
//.section.active {
// color: #4e7372;
//}

.home.icon {
margin-right: 0;
}

@media only screen and (min-width: @tabletBreakpoint) {
padding-right: 0;
padding-left: 0;
}
}
/*******************************
Theme Overrides
Expand All @@ -49,4 +34,3 @@
font-size: @breadcrumbIconFontSize;
}
}

0 comments on commit c5df567

Please sign in to comment.