Skip to content

Commit

Permalink
Merge pull request #261 from heorhi-deriv/fix-colors-in-breadcrumbs
Browse files Browse the repository at this point in the history
fix: 🚑 fix colors in Breadcrumbs
  • Loading branch information
shayan-deriv authored Oct 11, 2024
2 parents 5fa835d + f57237b commit 78ed22d
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 13 deletions.
8 changes: 0 additions & 8 deletions src/components/Breadcrumbs/Breadcrumbs.scss
Original file line number Diff line number Diff line change
Expand Up @@ -20,14 +20,6 @@
}
}

&__text {
color: var(--du-text-less-prominent, #999999);

&--active {
color: var(--du-text-prominent, #333333);
}
}

&__separator-container {
cursor: default;
}
Expand Down
7 changes: 3 additions & 4 deletions src/components/Breadcrumbs/index.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -35,11 +35,10 @@ export const Breadcrumbs = ({
className="deriv-breadcrumbs__item"
>
<Text
className={clsx("deriv-breadcrumbs__text", {
"deriv-breadcrumbs__text--active": isLastItem,
})}
size={textSize}
className="deriv-breadcrumbs__text"
color={isLastItem ? 'prominent' : 'less-prominent'}
onClick={() => handleOnClick(item)}
size={textSize}
>
{item.text}
</Text>
Expand Down
2 changes: 1 addition & 1 deletion src/components/Text/Text.scss
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
// TODO: Refactor this file to use css variables
$color-map: (
"prominent": var(--du-text-prominent),
"prominent": var(--du-text-prominent, #333333),
"less-prominent": var(--du-text-less-prominent, #999999),
"general": var(--du-text-general, #333333),
"primary": var(--du-text-primary, #999999),
Expand Down

0 comments on commit 78ed22d

Please sign in to comment.