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

[onefetch.dev] switch to dark theme #1297

Merged
merged 9 commits into from
Mar 23, 2024
Merged

[onefetch.dev] switch to dark theme #1297

merged 9 commits into from
Mar 23, 2024

Conversation

o2sh
Copy link
Owner

@o2sh o2sh commented Mar 23, 2024

Switching to a dark theme to make the site easier on the eyes,

@o2sh o2sh added the doc label Mar 23, 2024
@o2sh o2sh changed the title [onefetch.dev] use dark theme [onefetch.dev] switch to dark theme Mar 23, 2024
@o2sh o2sh requested a review from spenserblack March 23, 2024 14:16
docs/vercel/README.md Outdated Show resolved Hide resolved
"check:svelte": "svelte-check",
"check:prettier": "prettier --check **/*.{ts,js,svelte,css,html,json}",
"check:lint": "eslint 'src/**/*.{ts,js,svelte}'"
"check:lint": "eslint src/**/*.{ts,js,svelte}"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I forget exactly why this nuance is important, but quoting the glob ensures that eslint receives it as a string instead of the glob getting expanded to several arguments IIRC

Copy link
Owner Author

@o2sh o2sh Mar 23, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

With the quotes, i was getting the following error: ""No files matching the pattern "'src/**/*.{ts,js,svelte}'" were found.""

@@ -1,3 +1,4 @@
/* $color-text: #dedce5; */
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Was this file manually edited, or did you grab one of the other stylesheets?

Copy link
Owner Author

@o2sh o2sh Mar 23, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I renamed the css file to sakura-dark.css

Co-authored-by: Spenser Black <[email protected]>
@o2sh o2sh merged commit 37b327b into main Mar 23, 2024
2 checks passed
@o2sh o2sh deleted the site/use-dark-theme branch March 23, 2024 17:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants