Skip to content

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
"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 <spenserblack01@gmail.com>
@o2sh o2sh merged commit 37b327b into main Mar 23, 2024
@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

2 participants