Skip to content

Commit 62d97b1

Browse files
refactor: Typography, action hover and table head colors (#4046)
* Adjust primary text color * refactor: Typography and table head colors
1 parent a01ab27 commit 62d97b1

File tree

3 files changed

+11
-9
lines changed

3 files changed

+11
-9
lines changed

site/src/components/NavbarView/NavbarView.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -187,7 +187,7 @@ const useStyles = makeStyles((theme) => ({
187187
// NavLink adds this class when the current route matches.
188188
"&.active": {
189189
position: "relative",
190-
color: theme.palette.primary.contrastText,
190+
color: theme.palette.text.primary,
191191
fontWeight: "bold",
192192

193193
"&::before": {

site/src/theme/overrides.ts

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@ export const getOverrides = ({ palette, breakpoints }: Theme): Overrides => {
1111
backgroundImage: `linear-gradient(to right bottom, ${palette.background.default}, ${colors.gray[17]})`,
1212
backgroundRepeat: "no-repeat",
1313
backgroundAttachment: "fixed",
14-
letterSpacing: "-0.015em",
1514
},
1615
":root": {
1716
colorScheme: palette.type,
@@ -81,11 +80,6 @@ export const getOverrides = ({ palette, breakpoints }: Theme): Overrides => {
8180
},
8281
},
8382
},
84-
MuiTableHead: {
85-
root: {
86-
display: "table-header-group",
87-
},
88-
},
8983
MuiTableContainer: {
9084
root: {
9185
borderRadius,
@@ -119,6 +113,7 @@ export const getOverrides = ({ palette, breakpoints }: Theme): Overrides => {
119113
fontSize: 14,
120114
color: palette.text.secondary,
121115
fontWeight: 600,
116+
background: palette.background.paperLight,
122117
},
123118
root: {
124119
fontSize: 16,

site/src/theme/palettes.ts

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,12 @@ import { colors } from "./colors"
44
// Couldn't find a type for this so I made one. We can extend the palette if needed with module augmentation.
55
export type PaletteIndex = "primary" | "secondary" | "info" | "success" | "error" | "warning"
66

7+
declare module "@material-ui/core/styles/createPalette" {
8+
interface TypeBackground {
9+
paperLight: string
10+
}
11+
}
12+
713
export const darkPalette: PaletteOptions = {
814
type: "dark",
915
primary: {
@@ -20,9 +26,10 @@ export const darkPalette: PaletteOptions = {
2026
background: {
2127
default: colors.gray[17],
2228
paper: colors.gray[16],
29+
paperLight: colors.gray[15],
2330
},
2431
text: {
25-
primary: colors.gray[4],
32+
primary: colors.gray[1],
2633
secondary: colors.gray[5],
2734
},
2835
divider: colors.gray[13],
@@ -46,6 +53,6 @@ export const darkPalette: PaletteOptions = {
4653
contrastText: colors.gray[4],
4754
},
4855
action: {
49-
hover: colors.gray[13],
56+
hover: colors.gray[14],
5057
},
5158
}

0 commit comments

Comments
 (0)