Skip to content

Commit

Permalink
Change disclosure icon color with prop (#3926)
Browse files Browse the repository at this point in the history
Co-authored-by: Ben Scott <227292+BPScott@users.noreply.github.com>
  • Loading branch information
Alex Page and BPScott committed Jan 27, 2021
1 parent 8b36453 commit 452210c
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 3 deletions.
2 changes: 2 additions & 0 deletions UNRELEASED.md
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,8 @@ Use [the changelog guidelines](https://git.io/polaris-changelog-guidelines) to f

### Enhancements

- Removed duplicate color definition from disclosure `Icon` in `Tabs` ([#3926](https://github.com/Shopify/polaris-react/pull/3926))

### Bug fixes

- Fixed an accessibility issue where high contrast styles wouldn’t be applied to the `Tag` component ([#3810](https://github.com/Shopify/polaris-react/pull/3810))
Expand Down
1 change: 0 additions & 1 deletion src/components/Tabs/Tabs.scss
Original file line number Diff line number Diff line change
Expand Up @@ -201,7 +201,6 @@ $focus-state-box-shadow-color: rgba(0, 0, 0, 0.8);
}

.DisclosureActivator {
@include recolor-icon(var(--p-icon-subdued));
@include focus-ring;
height: 100%;
background-color: transparent;
Expand Down
4 changes: 2 additions & 2 deletions src/components/Tabs/Tabs.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -125,10 +125,10 @@ class TabsInner extends PureComponent<CombinedProps, State> {
const disclosureButtonContent = hasCustomDisclosure ? (
<>
{disclosureText}
<Icon source={CaretDownMinor} color="base" />
<Icon source={CaretDownMinor} color="subdued" />
</>
) : (
<Icon source={HorizontalDotsMinor} />
<Icon source={HorizontalDotsMinor} color="subdued" />
);

const disclosureButton = (
Expand Down

0 comments on commit 452210c

Please sign in to comment.