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

Light Mode Support #67

Open
wants to merge 9 commits into
base: main
Choose a base branch
from
Open

Light Mode Support #67

wants to merge 9 commits into from

Conversation

harrywang
Copy link

Our first version of the light mode with manual light/dark mode toggle. #23

Screenshot 2023-05-23 at 8 40 47 AM

@cruhl
Copy link
Contributor

cruhl commented May 23, 2023

Awesome work! As someone with astigmatism, I really appreciate this! ❤️

We'll be reviewing this as soon as we can. 🫡

@cruhl cruhl added important We really care about this issue! new feature This is a new feature! labels May 23, 2023
@@ -7,7 +7,8 @@ export function Next() {
<div className="flex flex-col">
<span className="text-lg font-medium">StableStudio</span>
<span className="-mt-1 text-xs font-light">
<span className="text-white/75">by </span>stability.ai
<span className="text-black/75 dark:text-white/75">by </span>

Choose a reason for hiding this comment

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

smaller note: indentation fix required so span tags are aligned

Copy link
Author

Choose a reason for hiding this comment

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

fixed, thanks.

@harrywang
Copy link
Author

Awesome work! As someone with astigmatism, I really appreciate this! ❤️

We'll be reviewing this as soon as we can. 🫡

@cruhl I know your must be busy - any idea when this can be reviewed? Thanks!

@harrywang
Copy link
Author

@KAJdev @cruhl hi guys, any updates?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
important We really care about this issue! new feature This is a new feature!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants