Skip to content

Commit b135ea4

Browse files
Cleanup colors (#371)
* Cleanup colors * Final cleanup
1 parent 8012f7e commit b135ea4

5 files changed

Lines changed: 8 additions & 57 deletions

File tree

assets/theme-css/code-highlight.css

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -112,7 +112,7 @@ html[data-theme="dark"] {
112112
}
113113
/* NameTag */
114114
.chroma .nt {
115-
color: var(--colorCode);
115+
color: #ffb8d1;
116116
}
117117
/* Literal */
118118
.chroma .l {

assets/theme-css/dark-mode.css

Lines changed: 0 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -9,26 +9,3 @@
99
--pst-color-background: #14181e;
1010
}
1111
}
12-
13-
/* Once our colors have all been mapped to PST color names, this section can be removed. */
14-
@media (prefers-color-scheme: dark) {
15-
:root {
16-
/* Basically, we reassign the colors named in the non-dark theme. The
17-
exception is the background: it seems to look better as #111
18-
than as --colorPrimaryDark. */
19-
--colorBackgroundAlt: var(--colorPrimaryDark);
20-
21-
--colorContentLink: var(--colorPrimaryLight);
22-
23-
--colorShadow: var(--colorSecondary);
24-
--colorHeading: var(--colorSecondary);
25-
26-
/* For, e.g. .news-container */
27-
--colorTextInverse: var(--colorSecondaryLight);
28-
--colorBackgroundInverse: var(--colorPrimaryDark);
29-
--colorLinkInverse: var(--colorPrimaryLight);
30-
31-
--colorBlockBackground: var(--colorPrimaryDark);
32-
--colorCode: #ffb8d1;
33-
}
34-
}

assets/theme-css/light-mode.css

Lines changed: 0 additions & 20 deletions
This file was deleted.

assets/theme-css/news.css

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -4,18 +4,16 @@
44
align-items: center;
55
width: 100vw;
66
min-height: 45px;
7-
background-color: var(--colorBackgroundInverse);
8-
color: var(--colorTextInverse);
7+
background-color: var(--colorPrimaryDark);
8+
color: var(--colorSecondaryLight);
99
white-space: nowrap;
1010
}
1111

1212
.news-container a {
13-
/* FIXME: We need a named color for this, like --colorLinkInverse. */
14-
color: var(--colorSecondary);
13+
color: var(--colorSecondaryLight);
1514
}
1615

1716
.news-container a:hover {
18-
/* FIXME: We need a named color for this, too. */
1917
color: var(--colorPrimaryLight);
2018
}
2119

assets/theme-css/styles.css

Lines changed: 4 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -111,8 +111,8 @@ a > code {
111111
text-transform: uppercase;
112112
letter-spacing: 1.5px;
113113
font-weight: 500;
114-
color: var(--colorSecondary);
115-
background-color: var(--colorBackgroundInverse);
114+
color: var(--colorSecondaryLight);
115+
background-color: var(--colorPrimaryDark);
116116
border: none;
117117
border-radius: 25px;
118118
outline: none;
@@ -123,7 +123,7 @@ a > code {
123123
height: 475px;
124124
}
125125
#backtotop.visible #backtotop-color:hover {
126-
background: var(--colorHighlight);
126+
background: var(--colorYellow);
127127
}
128128

129129
.mt-30 {
@@ -212,10 +212,6 @@ a.footer-link {
212212
color: var(--colorSecondary);
213213
}
214214

215-
.icon {
216-
color: var(--colorIcon);
217-
}
218-
219215
.copyright {
220216
font-size: 12px;
221217
color: var(--colorSecondary);
@@ -327,7 +323,7 @@ a.navbar-item:visited {
327323
}
328324

329325
a.navbar-item:hover {
330-
color: var(--colorPrimaryLight);
326+
color: var(--pst-color-link-hover);
331327
}
332328

333329
.navbar-logo {

0 commit comments

Comments
 (0)