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

Dark Mode for All Existing Color Themes #465

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

Conversation

Thunder-Blaze
Copy link
Contributor

@Thunder-Blaze Thunder-Blaze commented Jan 10, 2025

📝 Description

Added Dark mode for all the colors

📸 Screenshots / 📹 Videos

image
image
image

🔗 Related Issues

✅ Checklist

  • I have read and followed the Contributing Guidelines.
  • I have tested my changes by installing them as an extension (not just running via localhost) to ensure it builds, installs, and works as expected.
  • I have tested these changes in at least Chrome and Firefox (other browsers if applicable).
  • Attached visual evidence of changes (screenshots or videos) if applicable.

@prem-k-r prem-k-r linked an issue Jan 10, 2025 that may be closed by this pull request
@Thunder-Blaze Thunder-Blaze changed the title Feature/darkmode Dark Mode for All Existing Color Themes Jan 10, 2025
@prem-k-r prem-k-r added the enhancement New feature or request label Jan 10, 2025
@Thunder-Blaze
Copy link
Contributor Author

Fixed the issue ig
image

Copy link
Collaborator

@itz-rj-here itz-rj-here left a comment

Choose a reason for hiding this comment

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

Great work. We need to check for every issues we find here..

@Thunder-Blaze
Copy link
Contributor Author

Great work. We need to check for every issues we find here..

I think there is one issue, that screen blinking one when dark mode is on (we might need to put the logic in preload.js)
other than that lmk if you encounter any issues

@prem-k-r
Copy link
Collaborator

what it look like for black(dark) theme?

@Thunder-Blaze
Copy link
Contributor Author

what it look like for black(dark) theme?

lol
image
hehe

wait a min ig I can fix this

@prem-k-r
Copy link
Collaborator

we can ignore any changes if it's black theme

@Thunder-Blaze
Copy link
Contributor Author

Thunder-Blaze commented Jan 11, 2025

we can ignore any changes if it's black theme

yes, done :)

style.css Outdated Show resolved Hide resolved
@prem-k-r
Copy link
Collaborator

Is color palette still white?
image

@Thunder-Blaze
Copy link
Contributor Author

Nope it should be black now I think

@prem-k-r
Copy link
Collaborator

Ok, nice

Copy link
Collaborator

@prem-k-r prem-k-r left a comment

Choose a reason for hiding this comment

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

Great work!
some of the themes looks ugly.. cant be helped ig

Is it possible to use #1d1d1d instead of #000, too much constrast for black parts

can you please exclude bookmarks favicons from color inverting

@prem-k-r
Copy link
Collaborator

prem-k-r commented Jan 11, 2025

And the preload bg color is still previous ones from light themes

@prem-k-r prem-k-r added the changes required PR requires updates based on review feedback label Jan 11, 2025
@Thunder-Blaze
Copy link
Contributor Author

Great work! some of the themes looks ugly.. cant be helped ig

well it was a shortcut rather than adding more colors XD

Is it possible to use #1d1d1d instead of #000, too much constrast for black parts

Seems possible will take a bit of time though ig

can you please exclude bookmarks favicons from color inverting

yeah sure

@Thunder-Blaze
Copy link
Contributor Author

And the preload bg color is still prevoius ones from light themes

will try to fix (rn have no ideas execpt for storing a hasWallaper item and then fetching in preload)

@prem-k-r
Copy link
Collaborator

ok, take your time

@prem-k-r
Copy link
Collaborator

Great work! some of the themes looks ugly.. cant be helped ig

well it was a shortcut rather than adding more colors XD

Yeah, fantastic idea

@prem-k-r prem-k-r added the work in progress The issue is on progress to fix label Jan 11, 2025
@prem-k-r
Copy link
Collaborator

And the preload bg color is still prevoius ones from light themes

will try to fix (rn have no ideas execpt for storing a hasWallaper item and then fetching in preload)

fetch LoadingScreenColor from localstorage, then perform invert(1) hue-rotate(180deg) operation if toggle checked?
it sounds simple but operation part can be complicated

@prem-k-r
Copy link
Collaborator

When using color picker.. there is lots of processing.. so it lags lol
dont know if we can do anything if we want to show real time color

@Thunder-Blaze
Copy link
Contributor Author

When using color picker.. there is lots of processing.. so it lags lol dont know if we can do anything if we want to show real time color

I might a way of easier theme making and the processing will be low in it imo but will try after theme code has been refactored rn can't understand most of it

btw #1d1d1d is mixing with the other colors, is it fine ?
image

@prem-k-r
Copy link
Collaborator

When using color picker.. there is lots of processing.. so it lags lol dont know if we can do anything if we want to show real time color

I might a way of easier theme making and the processing will be low in it imo but will try after theme code has been refactored rn can't understand most of it

ok,
I once was also working on it #162 (comment) needed css fixes and I'm not good at it.. meanwhile xengshi implemented current color picker.. It better for most user.. so I quit that

btw #1d1d1d is mixing with the other colors, is it fine ? image

ah no, please use a shade in between them

@Thunder-Blaze
Copy link
Contributor Author

When using color picker.. there is lots of processing.. so it lags lol dont know if we can do anything if we want to show real time color

I might a way of easier theme making and the processing will be low in it imo but will try after theme code has been refactored rn can't understand most of it

ok, I once was also working on it #162 (comment) needed css fixes and I'm not good at it.. meanwhile xengshi implemented current color picker.. It better for most user.. so I quit that

btw #1d1d1d is mixing with the other colors, is it fine ? image

ah no, please use a shade in between them

oh it's a nice one
but I was talking about something like current one

ah no, please use a shade in between them

between black and 1d1d1d ?

@prem-k-r
Copy link
Collaborator

ah no, please use a shade in between them

between black and 1d1d1d ?

yeah, any that looks ok to you

@prem-k-r
Copy link
Collaborator

Btw, won't it be better if the toggle is available near the color palette..

IMG_20250112_201804
Well this doesn't looks good,
umm.. can't think of any UI idea

@Thunder-Blaze
Copy link
Contributor Author

Btw, won't it be better if the toggle is available near the color palette..

I also thought the same but there was no space 😞
In your design how about only one of icon ->

  • If light mode is selected, show moon
  • Otherwise show sun

@prem-k-r
Copy link
Collaborator

prem-k-r commented Jan 12, 2025

First I thought of that, some user might not understand it's clickable..
But we can go with it anyways lol

I mean only one icon.. opposite one

@Thunder-Blaze
Copy link
Contributor Author

First I thought of that, some user might not understand it's clickable.. But we can go with it anyways lol

I mean only one icon.. opposite one

yeah that's also true 😅

@prem-k-r
Copy link
Collaborator

if u need svg https://www.transfernow.net/dl/20250112i9KY6Sdn
or u can use diff one

@Thunder-Blaze
Copy link
Contributor Author

if u need svg https://www.transfernow.net/dl/20250112i9KY6Sdn or u can use diff one

thanks :)

@prem-k-r
Copy link
Collaborator

Maybe we can go ahead with this, if you exclude bookmarks favicons from color inverting, as this is still in experimental phase..
Later we can resolve related issues like loading color, toggle.. etc

@prem-k-r prem-k-r linked an issue Jan 15, 2025 that may be closed by this pull request
@Thunder-Blaze
Copy link
Contributor Author

Maybe we can go ahead with this, if you exclude bookmarks favicons from color inverting, as this is still in experimental phase.. Later we can resolve related issues like loading color, toggle.. etc

hmm sure 👍🏻

@prem-k-r prem-k-r removed work in progress The issue is on progress to fix changes required PR requires updates based on review feedback labels Jan 16, 2025
@prem-k-r prem-k-r self-requested a review January 16, 2025 19:20
Copy link
Collaborator

@prem-k-r prem-k-r left a comment

Choose a reason for hiding this comment

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

seems all fine to me (except the screen loading color), great work!

@prem-k-r prem-k-r requested a review from itz-rj-here January 17, 2025 07:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Dark Mode possibly? Dark theme toggle on custom color
3 participants