Skip to content

Commit 620970e

Browse files
committed
Migrate ntp- to ds- color variables
1 parent ec5c2db commit 620970e

File tree

9 files changed

+55
-83
lines changed

9 files changed

+55
-83
lines changed

special-pages/pages/new-tab/app/components/Popover.module.css

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@
1515
}
1616

1717
.content {
18-
background: var(--ntp-surface-tertiary);
18+
background: var(--ds-color-surface-tertiary);
1919
border-radius: 16px;
2020
flex: auto;
2121
padding: var(--sp-4) var(--sp-11) var(--sp-4) var(--sp-5);
@@ -26,7 +26,7 @@
2626
background: none;
2727
border-radius: 100%;
2828
border: none;
29-
color: var(--ntp-icons-primary);
29+
color: var(--ds-color-icons-primary);
3030
cursor: pointer;
3131
padding: 0;
3232
position: absolute;
@@ -74,22 +74,22 @@
7474
button {
7575
background: none;
7676
border: none;
77-
color: var(--ntp-accent-primary);
77+
color: var(--ds-color-accent-primary);
7878
cursor: pointer;
7979
padding: 0;
8080
transform: translateZ(0); /* Fix shadow rendering issue on .popover in WebKit */
8181

8282
&:hover {
83-
color: var(--ntp-accent-secondary);
83+
color: var(--ds-color-accent-secondary);
8484
}
8585

8686
&:active {
87-
color: var(--ntp-accent-tertiary);
87+
color: var(--ds-color-accent-tertiary);
8888
}
8989
}
9090
}
9191

9292
.arrow {
93-
color: var(--ntp-surface-tertiary);
93+
color: var(--ds-color-surface-tertiary);
9494
margin-right: -1px;
9595
}

special-pages/pages/new-tab/app/customizer/components/VisibilityMenu.module.css

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@
2121
height: calc(28 * var(--px-in-rem));
2222

2323
&:has(input[disabled]) .title {
24-
color: var(--ntp-text-tertiary)
24+
color: var(--ds-color-text-tertiary)
2525
}
2626
}
2727

special-pages/pages/new-tab/app/omnibar/components/AiChatForm.module.css

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@
1010
background: none;
1111
border: none;
1212
box-sizing: content-box;
13-
color: var(--ntp-text-primary);
13+
color: var(--ds-color-text-primary);
1414
line-height: 16px;
1515
max-height: 10lh;
1616
padding: var(--sp-3) var(--sp-3) 0;
@@ -21,7 +21,7 @@
2121
}
2222

2323
&::placeholder {
24-
color: var(--ntp-text-tertiary);
24+
color: var(--ds-color-text-tertiary);
2525
}
2626

2727
[data-platform="windows"] &::selection {
@@ -34,7 +34,7 @@
3434
}
3535

3636
.hasScroll & {
37-
border-bottom: 1px solid var(--ntp-decoration-tertiary);
37+
border-bottom: 1px solid var(--ds-color-surface-decoration-tertiary);
3838
padding-bottom: var(--sp-3);
3939
}
4040
}
@@ -52,10 +52,10 @@
5252
.submitButton {
5353
align-items: center;
5454
animation: fadeIn 0.3s ease-in-out;
55-
background: var(--ntp-accent-primary);
55+
background: var(--ds-color-accent-primary);
5656
border-radius: 100%;
5757
border: none;
58-
color: var(--ntp-accent-content-primary);
58+
color: var(--ds-color-accent-content-primary);
5959
cursor: pointer;
6060
display: flex;
6161
height: var(--sp-7);
@@ -66,16 +66,16 @@
6666

6767
&[disabled] {
6868
background: none;
69-
color: color-mix(in srgb, var(--ntp-icons-primary) 30%, transparent);
69+
color: color-mix(in srgb, var(--ds-color-icons-primary) 30%, transparent);
7070
cursor: default;
7171
}
7272

7373
&:hover:not([disabled]) {
74-
background: var(--ntp-accent-secondary);
74+
background: var(--ds-color-accent-secondary);
7575
}
7676

7777
&:active:not([disabled]) {
78-
background: var(--ntp-accent-tertiary);
78+
background: var(--ds-color-accent-tertiary);
7979
}
8080

8181
&:focus-visible {

special-pages/pages/new-tab/app/omnibar/components/Omnibar.module.css

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -39,12 +39,12 @@
3939

4040
/* Rest */
4141
.popup {
42-
background: var(--ntp-surface-tertiary);
42+
background: var(--ds-color-surface-tertiary);
4343
border-radius: 12px;
4444
box-shadow: 0 1px 4px 0 rgba(0, 0, 0, 0.1), 0 4px 8px 0 rgba(0, 0, 0, 0.08); /* Elevation 30 */
4545
display: flow-root; /* Prevent margin collapse on .field */
4646
margin: 3px;
47-
outline: 1px solid var(--ntp-controls-raised-backdrop);
47+
outline: 1px solid var(--ds-color-control-raised-backdrop);
4848

4949
[data-background-kind="color"] [data-theme="light"] &,
5050
[data-background-kind="gradient"] [data-theme="light"] &,
@@ -74,10 +74,10 @@
7474
/* Focused */
7575
.root:has(input:focus:placeholder-shown, textarea:focus:placeholder-shown) .popup {
7676
border-radius: 15px;
77-
border: 2px solid var(--ntp-accent-primary);
77+
border: 2px solid var(--ds-color-accent-primary);
7878
box-shadow: 0 4px 12px 0 rgba(0, 0, 0, 0.1), 0 20px 40px 0 rgba(0, 0, 0, 0.08); /* Elevation 90 */
7979
margin: 0;
80-
outline: 2px solid var(--ntp-accent-glow);
80+
outline: 2px solid var(--ds-color-accent-glow-primary);
8181

8282
.field {
8383
margin: 1px; /* Use margin on .field instead of padding on .popup so that position: absolute respects the spacing */

special-pages/pages/new-tab/app/omnibar/components/SearchForm.module.css

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@
99
.input {
1010
background: none;
1111
border: none;
12-
color: var(--ntp-text-primary);
12+
color: var(--ds-color-text-primary);
1313
font: var(--input-font);
1414
left: 0;
1515
padding: var(--sp-3);
@@ -18,7 +18,7 @@
1818
right: 0;
1919

2020
&::placeholder {
21-
color: var(--ntp-text-tertiary);
21+
color: var(--ds-color-text-tertiary);
2222
}
2323

2424
[data-platform='windows'] &::selection {
@@ -39,7 +39,7 @@
3939
}
4040

4141
.suffix {
42-
color: var(--ntp-accent-primary);
42+
color: var(--ds-color-accent-primary);
4343
flex: none;
4444
font: var(--input-font);
4545
}
@@ -49,7 +49,7 @@
4949
background: none;
5050
border-radius: 9px;
5151
border: none;
52-
color: var(--ntp-text-primary);
52+
color: var(--ds-color-text-primary);
5353
cursor: pointer;
5454
display: flex;
5555
flex: none;
@@ -63,11 +63,11 @@
6363
visibility: hidden;
6464

6565
&:hover {
66-
background: var(--ntp-controls-fill-primary);
66+
background: var(--ds-color-control-fill-primary);
6767
}
6868

6969
&:active {
70-
background: var(--ntp-controls-fill-secondary);
70+
background: var(--ds-color-control-fill-secondary);
7171
}
7272

7373
&:focus-visible {

special-pages/pages/new-tab/app/omnibar/components/SuggestionsList.module.css

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@
2121
background: none;
2222
border-radius: 4px;
2323
border: none;
24-
color: var(--ntp-text-primary);
24+
color: var(--ds-color-text-primary);
2525
cursor: pointer;
2626
display: flex;
2727
height: var(--sp-8);
@@ -35,10 +35,10 @@
3535

3636
&[aria-selected="true"],
3737
&:active {
38-
color: var(--ntp-accent-content-primary);
38+
color: var(--ds-color-accent-content-primary);
3939

4040
.suffix {
41-
color: var(--ntp-accent-content-primary);
41+
color: var(--ds-color-accent-content-primary);
4242
}
4343

4444
.badge {
@@ -47,11 +47,11 @@
4747
}
4848

4949
&[aria-selected="true"] {
50-
background: var(--ntp-accent-primary);
50+
background: var(--ds-color-accent-primary);
5151
}
5252

5353
&:active {
54-
background: var(--ntp-accent-secondary);
54+
background: var(--ds-color-accent-secondary);
5555
}
5656
}
5757

@@ -64,7 +64,7 @@
6464
}
6565

6666
.suffix {
67-
color: var(--ntp-text-tertiary);
67+
color: var(--ds-color-text-tertiary);
6868
flex: initial;
6969
font-size: 12px;
7070
overflow: hidden;
@@ -74,7 +74,7 @@
7474

7575
.badge {
7676
align-items: center;
77-
background: var(--ntp-controls-raised-backdrop);
77+
background: var(--ds-color-control-raised-backdrop);
7878
border-radius: var(--border-radius-sm);
7979
display: flex;
8080
flex: none;

special-pages/pages/new-tab/app/omnibar/components/TabSwitcher.module.css

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
.tabSwitcher {
2-
background: var(--ntp-controls-raised-backdrop);
2+
background: var(--ds-color-control-raised-backdrop);
33
border-radius: 99px;
44
display: grid;
55
gap: var(--sp-0_5);
@@ -17,7 +17,7 @@
1717
align-items: center;
1818
background: none;
1919
border: none;
20-
color: var(--ntp-text-primary);
20+
color: var(--ds-color-text-primary);
2121
cursor: pointer;
2222
display: flex;
2323
gap: 6px;
@@ -29,7 +29,7 @@
2929
z-index: 1;
3030

3131
&:hover:not([aria-selected="true"])::before {
32-
background: var(--ntp-controls-fill-primary);
32+
background: var(--ds-color-control-fill-primary);
3333
border-radius: 99px;
3434
content: '';
3535
inset: 0;
@@ -42,9 +42,9 @@
4242
}
4343

4444
.blob {
45-
background-color: var(--ntp-controls-raised-fill-primary);
45+
background-color: var(--ds-color-control-raised-fill-primary);
4646
border-radius: 18px;
47-
box-shadow: 0 1px 0 0 var(--ntp-tone-tint) inset, 0 1px 4px 0 var(--ntp-shadow-secondary), 0 4px 8px 0 var(--ntp-shadow-secondary);
47+
box-shadow: 0 1px 0 0 var(--ds-color-tone-tint-primary) inset, 0 1px 4px 0 var(--ds-color-shadow-secondary), 0 4px 8px 0 var(--ds-color-shadow-secondary);
4848
height: var(--sp-8);
4949
left: 0;
5050
position: absolute;
@@ -56,7 +56,7 @@
5656
z-index: 0;
5757

5858
[data-theme="dark"] & {
59-
box-shadow: 0 4px 4px var(--ntp-shadow-secondary), 0 1px 2px var(--ntp-shadow-secondary), inset 0 1px 0 rgba(255, 255, 255, 0.06);
59+
box-shadow: 0 4px 4px var(--ds-color-shadow-secondary), 0 1px 2px var(--ds-color-shadow-secondary), inset 0 1px 0 rgba(255, 255, 255, 0.06);
6060
}
6161

6262
@media (prefers-reduced-motion: reduce) {

special-pages/pages/new-tab/app/privacy-stats/components/PrivacyStats.module.css

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -76,7 +76,7 @@
7676
}
7777

7878
.title {
79-
color: var(--ntp-text-primary);
79+
color: var(--ds-color-text-primary);
8080
font-size: 40px;
8181
font-weight: 400;
8282
grid-area: title;

special-pages/pages/new-tab/app/styles/ntp-theme.css

Lines changed: 16 additions & 44 deletions
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@
3737
--focus-ring-thin: 0px 0px 0px 1px var(--ntp-focus-outline-color), 0px 0px 0px 1px var(--color-white);
3838
}
3939

40-
[data-theme-variant="default"] [data-theme="light"] {
40+
[data-theme="light"] {
4141
--ds-color-surface-primary: var(--ds-color-default-light-surface-primary);
4242
--ds-color-surface-secondary: var(--ds-color-default-light-surface-secondary);
4343
--ds-color-surface-tertiary: var(--ds-color-default-light-surface-tertiary);
@@ -78,9 +78,16 @@
7878
--ds-color-accent-glow-primary: var(--ds-color-default-light-accent-glow-primary);
7979
--ds-color-accent-glow-secondary: var(--ds-color-default-light-accent-glow-secondary);
8080
--ds-color-highlight-primary: var(--ds-color-default-light-highlight-primary);
81+
82+
/* Only exist in default variant: */
83+
--ds-color-tone-tint-primary: var(--ds-color-default-light-tone-tint-primary);
84+
--ds-color-control-raised-backdrop: var(--ds-color-default-light-control-raised-backdrop);
85+
--ds-color-control-raised-fill-primary: var(--ds-color-default-light-control-raised-fill-primary);
86+
--ds-color-shadow-primary: var(--ds-color-default-light-shadow-primary);
87+
--ds-color-shadow-secondary: var(--ds-color-default-light-shadow-secondary);
8188
}
8289

83-
[data-theme-variant="default"] [data-theme="dark"] {
90+
[data-theme="dark"] {
8491
--ds-color-surface-primary: var(--ds-color-default-dark-surface-primary);
8592
--ds-color-surface-secondary: var(--ds-color-default-dark-surface-secondary);
8693
--ds-color-surface-tertiary: var(--ds-color-default-dark-surface-tertiary);
@@ -121,6 +128,13 @@
121128
--ds-color-accent-glow-primary: var(--ds-color-default-dark-accent-glow-primary);
122129
--ds-color-accent-glow-secondary: var(--ds-color-default-dark-accent-glow-secondary);
123130
--ds-color-highlight-primary: var(--ds-color-default-dark-highlight-primary);
131+
132+
/* Only exist in default variant: */
133+
--ds-color-tone-tint-primary: var(--ds-color-default-dark-tone-tint-primary);
134+
--ds-color-control-raised-backdrop: var(--ds-color-default-dark-control-raised-backdrop);
135+
--ds-color-control-raised-fill-primary: var(--ds-color-default-dark-control-raised-fill-primary);
136+
--ds-color-shadow-primary: var(--ds-color-default-dark-shadow-primary);
137+
--ds-color-shadow-secondary: var(--ds-color-default-dark-shadow-secondary);
124138
}
125139

126140
[data-theme-variant="coolGray"] [data-theme="light"] {
@@ -781,27 +795,6 @@ body[data-theme-variant="desert"] {
781795
--focus-ring-primary: 0px 0px 0px 1px var(--color-white), 0px 0px 0px 3px var(--ntp-color-primary);
782796
--ntp-selection-color: var(--color-white);
783797
--ntp-selection-background-color: #3A69EF;
784-
785-
/* Global Colors & Styles - https://www.figma.com/design/3W4vi0zX8hrpQc7zInQQB6/%F0%9F%8E%A8-Global-Colors---Styles?node-id=11-1&p=f&vars=1&m=dev */
786-
/* @todo: This palette is what the native apps use, we should gradually move over to it */
787-
--ntp-surface-tertiary: #FFFFFF;
788-
--ntp-text-primary: rgba(0, 0, 0, 0.96);
789-
--ntp-text-tertiary: rgba(0, 0, 0, 0.42);
790-
--ntp-icons-primary: rgba(0, 0, 0, 0.84);
791-
--ntp-icons-tertiary: rgba(0, 0, 0, 0.36);
792-
--ntp-tone-tint: rgba(255, 255, 255, 0.48);
793-
--ntp-accent-primary: var(--color-blue-50);
794-
--ntp-accent-secondary: var(--color-blue-60);
795-
--ntp-accent-tertiary: var(--color-blue-70);
796-
--ntp-accent-glow: rgba(57, 105, 239, 0.2);
797-
--ntp-accent-content-primary: #FFFFFF;
798-
--ntp-controls-fill-primary: rgba(31, 31, 31, 0.09);
799-
--ntp-controls-fill-secondary: rgba(31, 31, 31, 0.12);
800-
--ntp-controls-raised-fill-primary: #FFFFFF;
801-
--ntp-controls-raised-backdrop: rgba(0, 0, 0, 0.09);
802-
--ntp-decoration-tertiary: rgba(0, 0, 0, 0.09);
803-
--ntp-shadow-primary: rgba(0, 0, 0, 0.05);
804-
--ntp-shadow-secondary: rgba(0, 0, 0, 0.08);
805798
}
806799

807800
[data-theme=dark] {
@@ -820,27 +813,6 @@ body[data-theme-variant="desert"] {
820813
--focus-ring-primary: 0px 0px 0px 1px var(--default-dark-background-color), 0px 0px 0px 3px var(--ntp-color-primary);
821814
--ntp-selection-color: var(--color-white);
822815
--ntp-selection-background-color: var(--color-blue-30);
823-
824-
/* Global Colors & Styles - https://www.figma.com/design/3W4vi0zX8hrpQc7zInQQB6/%F0%9F%8E%A8-Global-Colors---Styles?node-id=11-1&p=f&vars=1&m=dev */
825-
/* @todo: This palette is what the native apps use, we should gradually move over to it */
826-
--ntp-surface-tertiary: #474747;
827-
--ntp-text-primary: rgba(255, 255, 255, 0.96);
828-
--ntp-text-tertiary: rgba(255, 255, 255, 0.36);
829-
--ntp-icons-primary: rgba(255, 255, 255, 0.78);
830-
--ntp-icons-tertiary: rgba(255, 255, 255, 0.24);
831-
--ntp-tone-tint: rgba(249, 249, 249, 0.12);
832-
--ntp-accent-primary: var(--color-blue-30);
833-
--ntp-accent-secondary: var(--color-blue-40);
834-
--ntp-accent-tertiary: var(--color-blue-50);
835-
--ntp-accent-glow: rgba(114, 149, 246, 0.2);
836-
--ntp-accent-content-primary: var(--color-blue-80);
837-
--ntp-controls-fill-primary: rgba(249, 249, 249, 0.12);
838-
--ntp-controls-fill-secondary: rgba(249, 249, 249, 0.18);
839-
--ntp-controls-raised-fill-primary: rgba(255, 255, 255, 0.18);
840-
--ntp-controls-raised-backdrop: rgba(255, 255, 255, 0.12);
841-
--ntp-decoration-tertiary: rgba(255, 255, 255, 0.12);
842-
--ntp-shadow-primary: rgba(0, 0, 0, 0.16);
843-
--ntp-shadow-secondary: rgba(0, 0, 0, 0.24);
844816
}
845817

846818
/* This comes from the application settings */

0 commit comments

Comments
 (0)