Skip to content

Commit 8f5f029

Browse files
authored
fix(theming): align success, warning, and danger border color variables with design (#1936)
1 parent f4b6fd4 commit 8f5f029

File tree

3 files changed

+9
-9
lines changed

3 files changed

+9
-9
lines changed

packages/notifications/src/styled/StyledBase.spec.tsx

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -29,11 +29,11 @@ describe('StyledBase', () => {
2929

3030
it.each<{ mode: 'light' | 'dark'; type: Type; color: string }>([
3131
{ mode: 'light', type: 'success', color: PALETTE.green[300] },
32-
{ mode: 'dark', type: 'success', color: PALETTE.green[800] },
32+
{ mode: 'dark', type: 'success', color: PALETTE.green[900] },
3333
{ mode: 'light', type: 'error', color: PALETTE.red[300] },
34-
{ mode: 'dark', type: 'error', color: PALETTE.red[800] },
34+
{ mode: 'dark', type: 'error', color: PALETTE.red[900] },
3535
{ mode: 'light', type: 'warning', color: PALETTE.yellow[300] },
36-
{ mode: 'dark', type: 'warning', color: PALETTE.yellow[800] },
36+
{ mode: 'dark', type: 'warning', color: PALETTE.yellow[900] },
3737
{ mode: 'light', type: 'info', color: PALETTE.grey[300] },
3838
{ mode: 'dark', type: 'info', color: PALETTE.grey[800] }
3939
])('renders $mode mode $type border color', ({ mode, type, color }) => {

packages/theming/src/elements/theme/__snapshots__/index.spec.ts.snap

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -50,16 +50,16 @@ exports[`DEFAULT_THEME matches snapshot 1`] = `
5050
"warningEmphasis": "warningHue.600",
5151
},
5252
"border": {
53-
"danger": "dangerHue.800",
53+
"danger": "dangerHue.900",
5454
"dangerEmphasis": "dangerHue.600",
5555
"default": "neutralHue.800",
5656
"disabled": "neutralHue.800",
5757
"emphasis": "neutralHue.600",
5858
"primaryEmphasis": "primaryHue.600",
5959
"subtle": "neutralHue.900",
60-
"success": "successHue.800",
60+
"success": "successHue.900",
6161
"successEmphasis": "successHue.600",
62-
"warning": "warningHue.800",
62+
"warning": "warningHue.900",
6363
"warningEmphasis": "warningHue.600",
6464
},
6565
"foreground": {

packages/theming/src/elements/theme/index.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -66,9 +66,9 @@ const colors = {
6666
default: 'neutralHue.800',
6767
emphasis: 'neutralHue.600',
6868
subtle: 'neutralHue.900',
69-
success: 'successHue.800',
70-
warning: 'warningHue.800',
71-
danger: 'dangerHue.800',
69+
success: 'successHue.900',
70+
warning: 'warningHue.900',
71+
danger: 'dangerHue.900',
7272
primaryEmphasis: 'primaryHue.600',
7373
successEmphasis: 'successHue.600',
7474
warningEmphasis: 'warningHue.600',

0 commit comments

Comments
 (0)