Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Set explicit background colours for website #1361

Merged
merged 2 commits into from
Sep 19, 2024
Merged

Conversation

kriswest
Copy link
Contributor

@kriswest kriswest commented Sep 19, 2024

The website doesn't currently explicitly set background colors - rather it assume the browser's default background is white (which is a user preference in some browsers, while many electron-based products use a dark background by default - leaving the page impossible to read).

This PR sets an explicit background colour for each mode (dark and light)

@kriswest kriswest requested a review from a team as a code owner September 19, 2024 16:39
Copy link

netlify bot commented Sep 19, 2024

Deploy Preview for fdc3 ready!

Name Link
🔨 Latest commit fb1d954
🔍 Latest deploy log https://app.netlify.com/sites/fdc3/deploys/66ec74bfb5948000079c852c
😎 Deploy Preview https://deploy-preview-1361--fdc3.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@kriswest kriswest requested a review from robmoffat September 19, 2024 16:41
@bingenito
Copy link
Member

bingenito commented Sep 19, 2024

Any thoughts on a bit lighter for dark theme like #1b1b1d? You can see with dark theme on docusaurus.io for an example.

bingenito
bingenito previously approved these changes Sep 19, 2024
@kriswest
Copy link
Contributor Author

@bingenito adjusted to #1b1b1b, does seem a little less stark

@kriswest kriswest requested review from bingenito and a team September 19, 2024 19:04
@kriswest kriswest merged commit b8b4065 into main Sep 19, 2024
10 checks passed
@kriswest kriswest deleted the website-background branch September 19, 2024 19:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants