Skip to content

Commit

Permalink
feat(ProgressBar): Update colors from danger200/success200 to danger3…
Browse files Browse the repository at this point in the history
…00/success300 in dark mode. (#1500)

Co-authored-by: Nikita Orliak <[email protected]>
  • Loading branch information
nikitaorliak-cengage and Nikita Orliak authored Oct 21, 2024
1 parent facf8e9 commit 5a928ea
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 4 deletions.
5 changes: 5 additions & 0 deletions .changeset/feat-progressBar-update-colors.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
'react-magma-dom': minor
---

feat(ProgressBar): Update colors from danger200/success200 to danger300/success300 in dark mode.
Original file line number Diff line number Diff line change
Expand Up @@ -103,7 +103,7 @@ describe('ProgressBar', () => {

expect(container.querySelector('[role="progressbar"]')).toHaveStyleRule(
'background',
magma.colors.danger200
magma.colors.danger300
);
});

Expand All @@ -114,7 +114,7 @@ describe('ProgressBar', () => {

expect(container.querySelector('[role="progressbar"]')).toHaveStyleRule(
'background',
magma.colors.success200
magma.colors.success300
);
});

Expand Down
4 changes: 2 additions & 2 deletions packages/react-magma-dom/src/components/ProgressBar/index.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -60,9 +60,9 @@ function buildProgressBarBackground(props) {
} else if (props.color === ProgressBarColor.primary) {
return props.theme.colors.tertiary;
} else if (props.color === ProgressBarColor.danger) {
return props.theme.colors.danger200;
return props.theme.colors.danger300;
} else if (props.color === ProgressBarColor.success) {
return props.theme.colors.success200;
return props.theme.colors.success300;
}
}
if (
Expand Down

2 comments on commit 5a928ea

@github-actions
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@github-actions
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please sign in to comment.