Skip to content

Commit c9b9900

Browse files
remove grayscale treatment (#3024)
As per https://twitter.com/sliminality/status/1269033413488459776, there's feedback that the contrast isn't great for reading. I myself have trouble with low contrast text, (and long sightedness), and I thought it was fine. I was clearly mistaken. Removing this for now to get back to a non harmful position, and we can decide later what to do instead.
1 parent 6bb0d33 commit c9b9900

File tree

8 files changed

+2
-18
lines changed

8 files changed

+2
-18
lines changed

src/components/Container/Container.js

+1-8
Original file line numberDiff line numberDiff line change
@@ -15,20 +15,13 @@ import type {Node} from 'react';
1515
* This component wraps page content sections (eg header, footer, main).
1616
* It provides consistent margin and max width behavior.
1717
*/
18-
const Container = ({
19-
children,
20-
grayscale,
21-
}: {
22-
children: Node,
23-
grayscale: boolean,
24-
}) => (
18+
const Container = ({children}: {children: Node}) => (
2519
<div
2620
css={{
2721
paddingLeft: 20,
2822
paddingRight: 20,
2923
marginLeft: 'auto',
3024
marginRight: 'auto',
31-
filter: grayscale ? 'grayscale(100%)' : '',
3225
[media.greaterThan('medium')]: {
3326
width: '90%',
3427
},

src/components/LayoutFooter/FooterNav.js

-1
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,6 @@ const FooterNav = ({children, title, layoutHasSidebar = false}: Props) => (
3434
css={{
3535
display: 'inline-flex',
3636
flexDirection: 'column',
37-
filter: 'grayscale(100%)',
3837
}}>
3938
{children}
4039
</div>

src/components/LayoutHeader/Header.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -61,7 +61,7 @@ const Header = ({location}: {location: Location}) => (
6161
</a>
6262
</div>
6363
</Container>
64-
<Container grayscale={true}>
64+
<Container>
6565
<div
6666
css={{
6767
display: 'flex',

src/components/MarkdownPage/MarkdownPage.js

-1
Original file line numberDiff line numberDiff line change
@@ -87,7 +87,6 @@ const MarkdownPage = ({
8787
<div
8888
css={{
8989
marginTop: 15,
90-
filter: 'grayscale(100%)', // BLM
9190
}}>
9291
{date}{' '}
9392
{hasAuthors && (

src/pages/index.js

-1
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,6 @@ class Home extends Component {
5757
css={{
5858
width: '100%',
5959
marginTop: 60, // BLM
60-
filter: 'grayscale(100%)', // BLM
6160
}}>
6261
<header
6362
css={{

src/templates/components/NavigationFooter/NavigationFooter.js

-2
Original file line numberDiff line numberDiff line change
@@ -15,8 +15,6 @@ const NavigationFooter = ({next, prev, location}) => {
1515
return (
1616
<div
1717
css={{
18-
filter: 'grayscale(100%)', // BLM
19-
2018
background: colors.dark,
2119
color: colors.white,
2220
paddingTop: 50,

src/templates/components/Sidebar/Sidebar.js

-2
Original file line numberDiff line numberDiff line change
@@ -37,8 +37,6 @@ class Sidebar extends Component {
3737
direction="column"
3838
halign="stretch"
3939
css={{
40-
filter: 'grayscale(100%)', // BLM
41-
4240
width: '100%',
4341
paddingLeft: 20,
4442
position: 'relative',

src/theme.js

-2
Original file line numberDiff line numberDiff line change
@@ -132,8 +132,6 @@ const sharedStyles = {
132132
},
133133
},
134134
content: {
135-
filter: 'grayscale(100%)', // BLM
136-
137135
marginTop: 40,
138136
marginBottom: 120,
139137

0 commit comments

Comments
 (0)