Skip to content

Commit 78c59f7

Browse files
authored
Refactor CSS names and merge colors: neutral background. (#8077)
1 parent 1f628e2 commit 78c59f7

File tree

8 files changed

+33
-26
lines changed

8 files changed

+33
-26
lines changed

pkg/web_css/lib/dartdoc.scss

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@
3131

3232
/* Note: light-theme was added to prevent non-visible table text in dartdoc's dark mode. */
3333
.light-theme .markdown-body table tr {
34-
background-color: var(--pub-default-background-color);
34+
background-color: var(--pub-neutral-bgColor);
3535
border-top: 1px solid #c6cbd1;
3636
}
3737

pkg/web_css/lib/src/_base.scss

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@
1010
}
1111

1212
body {
13-
background-color: var(--pub-default-background-color);
13+
background-color: var(--pub-neutral-bgColor);
1414
color: var(--pub-default-text-color);
1515
font-size: 14px;
1616
line-height: 1.6;
@@ -96,9 +96,10 @@ a {
9696
opacity: 0.8;
9797
}
9898

99+
// TODO: fix style to not use bgColor as text and text color as background
99100
&.link-button {
100101
background: var(--pub-link-text-color);
101-
color: var(--pub-default-background-color);
102+
color: var(--pub-neutral-bgColor);
102103
display: inline-block;
103104
padding: 4px 12px;
104105
}

pkg/web_css/lib/src/_home.scss

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -100,7 +100,7 @@
100100
}
101101

102102
.mini-list-item {
103-
background: var(--pub-home_card-background-color);
103+
background: var(--pub-neutral-bgColor);
104104
border-radius: 4px;
105105
box-shadow: 0px 2px 7px 0px var(--pub-home_card-box_shadow-color);
106106
padding: 28px 30px 30px; // title's top gap is about 2px (30-2 => 28)
@@ -206,7 +206,7 @@
206206
top: 0;
207207
bottom: 0;
208208
right: 0;
209-
background: var(--pub-home_card_fadeout-background-value);
209+
background: linear-gradient(transparent 90%, var(--pub-neutral-bgColor));
210210
}
211211
}
212212
}

pkg/web_css/lib/src/_list.scss

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,7 @@
6060
.sort-control-option {
6161
display: block;
6262
width: 100%;
63-
background: var(--pub-sort_control-background-color);
63+
background: var(--pub-neutral-bgColor);
6464
color: var(--pub-sort_control-text-color);
6565
font-size: 14px;
6666
padding: 12px 12px;

pkg/web_css/lib/src/_pkg.scss

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44

55
.package-badge {
66
display: inline-block;
7-
background: var(--pub-default-background-color);
7+
background: var(--pub-neutral-bgColor);
88
border: 1px solid var(--pub-badge-default-color);
99
border-radius: 20px;
1010
color: var(--pub-badge-default-color);

pkg/web_css/lib/src/_search.scss

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -112,7 +112,7 @@
112112
width: 25ex;
113113
border: 1px solid #000;
114114
max-height: 20em;
115-
background-color: var(--pub-default-background-color);
115+
background-color: var(--pub-neutral-bgColor);
116116
font-size: 16px;
117117
margin-left: 48px; // counter padding-left on .search-bar>.input
118118
margin-top: -14px; // counter padding-top on .search-bar>.input

pkg/web_css/lib/src/_tags.scss

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,7 @@
5858
>.-discontinued-replacedby {
5959
display: inline-block;
6060
padding: 2px 6px;
61-
background: var(--pub-default-background-color);
61+
background: var(--pub-neutral-bgColor);
6262
color: var(--pub-default-text-color);
6363
}
6464
}

pkg/web_css/lib/src/_variables.scss

Lines changed: 23 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -3,16 +3,25 @@
33
// color definitions:
44
// `--pub-color-<qualifier><baseColor>`
55
//
6-
// components:
6+
// theme roles:
7+
// `--pub-<role>[-<modifier>]-<property>`, where:
8+
// - `<role>` may be: `neutral`, `inset`
9+
// - `<modifier>` may be: `hover`, `selected`
10+
// - `<property>` may be: `bgColor`, `textColor`, `linkColor`
11+
//
12+
// components (**deprecated**):
713
// `--pub-[component]-[role]-[property]`, where
814
// - `[component]` may be: `default`, `button`, `link`, `card`, ...
915
// - `[role]` may be the `background`, `text`, `border`, ... (multiple parts are separated by `_`)
1016
// - `[property]` may be `color`, `opacity` for specific values, or a `value` for multi-part properties.
1117

1218
:root {
19+
--pub-color-white: #ffffff;
20+
1321
--pub-color-dangerRed: #ff4242;
1422

15-
--pub-default-background-color: #ffffff;
23+
--pub-neutral-bgColor: var(--pub-color-white);
24+
1625
--pub-default-headline-font_family: "Google Sans Display", "Google Sans", "Roboto", sans-serif;
1726
--pub-default-text-color: hsl(0, 0%, 29%);
1827
--pub-default-text-font_family: "Google Sans Text", "Google Sans", "Roboto", sans-serif;
@@ -26,9 +35,9 @@
2635
--pub-carousel_nav-background-color: rgba(0,0,0,0.7);
2736
--pub-carousel_nav_hover-background-color: #4285f4;
2837
--pub-carousel_screenshot-background-color: rgba(0,0,0,0.7);
29-
--pub-carousel_screenshot-text-color: #ffffff;
38+
--pub-carousel_screenshot-text-color: var(--pub-color-white);
3039
--pub-collections_icon-background-color: #aeaeae;
31-
--pub-thumbnail_container-background-color: #ffffff;
40+
--pub-thumbnail_container-background-color: var(--pub-color-white); // same in dark mode
3241
--pub-copy_feedback-background-color: #fafaff;
3342
--pub-detail_tab-background-color: var(--pub-code-background-color);
3443
--pub-detail_tab-text-color: var(--pub-default-text-color);
@@ -40,11 +49,9 @@
4049
--pub-footer-text-color: #f8f9fa;
4150
--pub-home_title-text-color: #254a76;
4251
--pub-home_announcement-background-color: #e7f8ff;
43-
--pub-home_banner-text-color: #ffffff;
52+
--pub-home_banner-text-color: var(--pub-color-white);
4453
--pub-home_banner-link-color: #31b0fc;
45-
--pub-home_card-background-color: #ffffff;
4654
--pub-home_card-box_shadow-color: rgba(0, 0, 0, 0.3);
47-
--pub-home_card_fadeout-background-value: linear-gradient(transparent 90%, white);
4855
--pub-home_card_title-text-color: #1967d2;
4956
--pub-home_card_hover-background-color: #fafafa;
5057
--pub-home_card_hover-box_shadow-color: rgba(0, 0, 0, 0.4);
@@ -56,7 +63,7 @@
5663
--pub-pkg_list_item_hover-background-color: #fafafa;
5764
--pub-pkg_list_recent_item-text-color: #6d7278;
5865
--pub-remove_button-background-color: var(--pub-color-dangerRed);
59-
--pub-remove_button-text-color: #ffffff;
66+
--pub-remove_button-text-color: var(--pub-color-white);
6067
--pub-report_header_score_error-text-color: #e13701;
6168
--pub-report_header_score_warning-text-color: #ffa500;
6269
--pub-report-hover-background-color: #f0f0f0;
@@ -65,7 +72,7 @@
6572
--pub-searchbar-background-color: #132030;
6673
--pub-searchbar-text-color: #8d9399;
6774
--pub-searchbar_input-background-color: #35404d;
68-
--pub-searchbar_input-text-color: #ffffff;
75+
--pub-searchbar_input-text-color: var(--pub-color-white);
6976
--pub-search_completition_selected-background-color: #ccc;
7077
--pub-session_warning-background-color: #ffffaa;
7178
--pub-site_header_banner-background-color: #1C2834;
@@ -74,14 +81,13 @@
7481
--pub-site_header_popup-background-color: #1f3044;
7582
--pub-site_header_popup-text-color: #f8f9fa;
7683
--pub-site_header_popup-border-color: #4a5868; // mix of bg+fg color
77-
--pub-sort_control-background-color: #ffffff;
7884
--pub-sort_control-text-color: var(--pub-default-text-color);
7985
--pub-sort_control_hover-background-color: #f5f5f7;
8086
--pub-sort_control_hover-text-color: var(--pub-default-text-color);
8187
--pub-sort_control_selected-background-color: #e7f8ff;
8288
--pub-sort_control_selected-text-color: var(--pub-default-text-color);
8389
--pub-spinner_frame-background-color: rgba(0, 0, 0, 0.2);
84-
--pub-summary_hover-background-color: color-mix(in srgb, var(--pub-link-text-color), var(--pub-default-background-color) 80%);
90+
--pub-summary_hover-background-color: color-mix(in srgb, var(--pub-link-text-color), var(--pub-neutral-bgColor) 80%);
8591
--pub-tag_simplebadge-background-color: #f0f0f0;
8692
--pub-tag_simplebadge-text-color: #444444;
8793
--pub_tag_simplebadge_warning-background-color: #c0392b;
@@ -104,7 +110,10 @@
104110
}
105111

106112
.dark-theme {
107-
--pub-default-background-color: #212121;
113+
--pub-color-darkGunmetal: #1f262a; // close to #1d2026
114+
115+
--pub-neutral-bgColor: var(--pub-color-darkGunmetal);
116+
108117
--pub-default-text-color: #e0e0e0;
109118
--pub-code-background-color: #474747;
110119
--pub-code-text-color: var(--pub-default-text-color);
@@ -118,9 +127,7 @@
118127
--pub-detail_tab-active-color: color-mix(in srgb, var(--pub-link-text-color), var(--pub-detail_tab-background-color) 20%);
119128
--pub-detail_tab-admin-color: #e03030;
120129
--pub-home_title-text-color: #31b0fc;
121-
--pub-home_card-background-color: #303030;
122130
--pub-home_card-box_shadow-color: rgba(255, 255, 255, 0.2);
123-
--pub-home_card_fadeout-background-value: linear-gradient(transparent 90%, var(--pub-home_card-background-color));
124131
--pub-home_card_title-text-color: var(--pub-home_title-text-color);
125132
--pub-home_card_hover-background-color: #383838;
126133
--pub-home_card_hover-box_shadow-color: rgba(255, 255, 255, 0.3);
@@ -132,21 +139,20 @@
132139
--pub-report-hover-background-color: #333333;
133140
--pub-score_label-text-color: #a0b0b8;
134141
--pub-score_value-text-color: var(--pub-link-text-color);
135-
--pub-sort_control-background-color: #333333;
136142
--pub-sort_control-text-color: var(--pub-default-text-color);
137143
--pub-sort_control_hover-background-color: var(--pub-code-background-color);
138144
--pub-sort_control_hover-text-color: var(--pub-default-text-color);
139145
--pub-sort_control_selected-background-color: #206080;
140146
--pub-sort_control_selected-text-color: var(--pub-default-text-color);
141-
--pub-summary_hover-background-color: color-mix(in srgb, var(--pub-link-text-color), var(--pub-default-background-color) 80%);
147+
--pub-summary_hover-background-color: color-mix(in srgb, var(--pub-link-text-color), var(--pub-neutral-bgColor) 80%);
142148
--pub-tag_simplebadge-background-color: var(--pub-code-background-color);
143149
--pub-tag_simplebadge-text-color: var(--pub-default-text-color);
144150
--pub-tag_sdkbadge-background-color: #206080;
145151
--pub-tag_sdkbadge-separator-color: var(--pub-default-text-color);
146152
--pub-tag_sdkbadge-text-color: var(--pub-default-text-color);
147153

148154
// Material Design theme customizations
149-
--mdc-theme-surface: var(--pub-default-background-color);
155+
--mdc-theme-surface: var(--pub-neutral-bgColor);
150156
--mdc-theme-on-primary: var(--pub-default-text-color);
151157
--mdc-theme-on-secondary: var(--pub-default-text-color);
152158
--mdc-theme-on-surface: var(--pub-default-text-color);

0 commit comments

Comments
 (0)