Skip to content

Commit e1b1c46

Browse files
authored
feat(buttons): new light/dark mode colors (#1807)
1 parent 5282298 commit e1b1c46

File tree

6 files changed

+192
-98
lines changed

6 files changed

+192
-98
lines changed

packages/buttons/src/styled/StyledAnchor.spec.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77

88
import React from 'react';
99
import { render, renderRtl } from 'garden-test-utils';
10-
import { getColorV8 } from '@zendeskgarden/react-theming';
10+
import { PALETTE } from '@zendeskgarden/react-theming';
1111
import { StyledAnchor } from './StyledAnchor';
1212

1313
describe('StyledAnchor', () => {
@@ -26,7 +26,7 @@ describe('StyledAnchor', () => {
2626
it('renders danger styling if provided', () => {
2727
const { container } = render(<StyledAnchor isDanger />);
2828

29-
expect(container.firstChild).toHaveStyleRule('color', getColorV8('dangerHue'));
29+
expect(container.firstChild).toHaveStyleRule('color', PALETTE.red[700]);
3030
});
3131

3232
it('renders expected RTL direction', () => {

packages/buttons/src/styled/StyledButton.spec.tsx

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77

88
import React from 'react';
99
import { render } from 'garden-test-utils';
10-
import { PALETTE_V8 } from '@zendeskgarden/react-theming';
10+
import { PALETTE } from '@zendeskgarden/react-theming';
1111
import { StyledButton } from './StyledButton';
1212

1313
describe('StyledButton', () => {
@@ -32,13 +32,15 @@ describe('StyledButton', () => {
3232
it('renders danger styling if provided', () => {
3333
const { container } = render(<StyledButton isDanger />);
3434

35-
expect(container.firstChild).toHaveStyleRule('color', PALETTE_V8.red[600]);
35+
expect(container.firstChild).toHaveStyleRule('color', PALETTE.red[700]);
3636
});
3737

3838
it('renders disabled styling if provided', () => {
3939
const { container } = render(<StyledButton disabled />);
4040

41-
expect(container.firstChild).toHaveStyleRule('color', PALETTE_V8.grey[600]);
41+
expect(container.firstChild).toHaveStyleRule('color', PALETTE.grey[600], {
42+
modifier: '&:disabled'
43+
});
4244
});
4345

4446
it('renders link styling if provided', () => {
@@ -51,13 +53,13 @@ describe('StyledButton', () => {
5153
it('renders primary styling if provided', () => {
5254
const { container } = render(<StyledButton isPrimary />);
5355

54-
expect(container.firstChild).toHaveStyleRule('background-color', PALETTE_V8.blue[600]);
56+
expect(container.firstChild).toHaveStyleRule('background-color', PALETTE.blue[700]);
5557
});
5658

5759
it('renders neutral styling if provided', () => {
5860
const { container } = render(<StyledButton isNeutral />);
5961

60-
expect(container.firstChild).toHaveStyleRule('border-color', PALETTE_V8.grey[300]);
62+
expect(container.firstChild).toHaveStyleRule('border-color', PALETTE.grey[400]);
6163
});
6264

6365
it('renders pill styling if provided', () => {

0 commit comments

Comments
 (0)