Skip to content

[feat] : added darkmode#42

Open
onlybond wants to merge 3 commits intopmndrs:mainfrom
onlybond:main
Open

[feat] : added darkmode#42
onlybond wants to merge 3 commits intopmndrs:mainfrom
onlybond:main

Conversation

@onlybond
Copy link
Copy Markdown

Summary: This PR addresses issue #9 by introducing dark mode across the entire application, providing users with a more comfortable viewing experience in low-light environments.

Challenges Faced: During implementation, I encountered challenges with theme customization. The application relies on pre-built themes, which limited direct color adjustments. To ensure consistency, I matched the dark mode colors with the existing theme used by the Leva component.

Status: Dark mode is now fully implemented and integrated across the application. Please review and provide feedback.

image
image

@vercel
Copy link
Copy Markdown

vercel Bot commented Aug 26, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
gltf ✅ Ready (Inspect) Visit Preview 💬 Add feedback Sep 11, 2024 7:32am

@SaraVieira
Copy link
Copy Markdown
Member

Hey!

Your screenshots look great but when I open the website it opens like this:

image

Have you had this issue? It may be a linux thing

@onlybond
Copy link
Copy Markdown
Author

onlybond commented Sep 11, 2024

Hey!

Your screenshots look great but when I open the website it opens like this:

image

Have you had this issue? It may be a linux thing

sure lemme see through it

edit: maybe its a issue with how i am implementing the dark mode and have not optimized it for all devices.

@onlybond
Copy link
Copy Markdown
Author

@SaraVieira can you review it now

@onlybond
Copy link
Copy Markdown
Author

hey there @SaraVieira its been a long time. this is just a small pr, can you recheck the changes i have done

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