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

feat: add dark mode #76

Merged
merged 5 commits into from
Mar 20, 2024

Conversation

sametpalitci
Copy link
Contributor

@sametpalitci sametpalitci commented Mar 14, 2024

Dark Mode:
dark

Light Mode:
light

@sametpalitci sametpalitci mentioned this pull request Mar 14, 2024
<div>
<div class="ml-4">
<h3 class="mb-2 text-2xl font-normal" id="input-types">Input Types</h3>
<h3 class="mb-2 text-2xl font-normal" id="input-types dark:text-white">Input Types</h3>
Copy link
Owner

Choose a reason for hiding this comment

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

Please do the same for output types on line 94. Right now it is of wrong color in dark mode

Comment on lines 92 to 95
<body class="bg-gray-200 dark:bg-black">
<div class="flex min-h-screen">
<div
class="sticky top-0 w-1/5 flex-shrink-0 h-screen p-4 px-6 mr-4 overflow-auto bg-white mac-h-screen"
class="sticky top-0 w-1/5 flex-shrink-0 h-screen p-4 px-6 overflow-auto bg-white dark:bg-black mac-h-screen"
Copy link
Owner

Choose a reason for hiding this comment

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

I would like to have same bg color for sidebar and content so match them to gray-800

Copy link
Owner

@pantharshit00 pantharshit00 left a comment

Choose a reason for hiding this comment

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

Thanks for the work! I have requested a few small changes then it should be good.

Also, please update the snapshots once after you have made the changes using npm run test -- -u. Right now CI is failing because of that.

Finally you can also maybe look into the code block sections and update those with dark theme using something like https://github.com/antfu/prism-theme-vars. That would be the final touch. You can also do a separate PR for that if you don't have enough time.

@sametpalitci
Copy link
Contributor Author

I've made the requested changes and updated the snapshot. I'm considering maybe making the Prism theme in another pull request at a later time.

Thank you.

@pantharshit00 pantharshit00 merged commit 90c10a0 into pantharshit00:master Mar 20, 2024
1 check passed
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