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

Remove all dark mode tailwind references #449

Merged
merged 2 commits into from
Dec 30, 2024
Merged

Conversation

anchit-chandran
Copy link
Contributor

@anchit-chandran anchit-chandran commented Dec 30, 2024

Signed-off-by: anchit-chandran [email protected]

Overview

I have darkmode as my machine's system preference, tailwind was still detecting this and applying dark styles, but we don't have a full dark mode theme yet (still WIP) so couple places looks a bit odd.

cc @reece for visibility

Code changes

Just removes the few styles where dark: is specified

Documentation changes (done or required as a result of this PR)

  • Also refactors some docs to add Useful scripts page with s/watch-tailwind script

Related Issues

#108
#427

Mentions

@mentions of the person or team responsible for reviewing proposed changes.

@anchit-chandran anchit-chandran merged commit bded7ac into live Dec 30, 2024
1 check passed
@anchit-chandran anchit-chandran deleted the anchit/update-navbar branch December 30, 2024 11:58
@mbarton
Copy link
Member

mbarton commented Dec 30, 2024

Seen on STAGING (merged by @anchit-chandran 6 minutes and 24 seconds ago) Please check your changes!

@reece
Copy link

reece commented Dec 30, 2024

cc @reece for visibility

@anchit-chandran : You got the wrong Reece. I think you meant @reecehill . Probably no matter here since this PR was merged, but perhaps useful in the future.

@anchit-chandran
Copy link
Contributor Author

cc @reece for visibility

@anchit-chandran : You got the wrong Reece. I think you meant @reecehill . Probably no matter here since this PR was merged, but perhaps useful in the future.

Many apologies @reece !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants