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 Plug-In Issue #23

Open
mekaspencer31 opened this issue Mar 5, 2021 · 10 comments
Open

Dark Mode Plug-In Issue #23

mekaspencer31 opened this issue Mar 5, 2021 · 10 comments

Comments

@mekaspencer31
Copy link

Hi, when I go to put in the hex colors for dark mode it does not make the change to the background "BG" or Text does not change the same color as white.

Screen Shot 2021-03-05 at 5 51 01 PM

Screen Shot 2021-03-05 at 5 49 51 PM

@eddiesigner
Copy link
Owner

Hi,

are you sure that the layers of your document are actually using those colors variables? The plugin ignores colors that are not Color Variables or Color Assets. If not then you have to assign the proper Color Variable or Color Asset in the color inspector for every layer.

https://github.com/eddiesigner/sketch-dark-mode/wiki/How-to-use-Sketch-Dark-Mode#2-use-your-colors-in-the-layers-of-your-document

@mekaspencer31
Copy link
Author

mekaspencer31 commented Mar 8, 2021 via email

@eddiesigner
Copy link
Owner

I can't see the image, please try to upload it directly through Github.
What version of Sketch are you using?

@mekaspencer31
Copy link
Author

Oh yes my apologies. Here are the images I tried to upload earlier.
unnamed-1

unnamed

@eddiesigner
Copy link
Owner

It looks good to me, please make sure to save the color palette and also your document changes. After that you can generate the dark mode version of your Artboard or your current Page by running one of the plugin commands: https://github.com/eddiesigner/sketch-dark-mode/wiki/How-to-use-Sketch-Dark-Mode#4-generate-the-dark-mode-version-of-your-design

@mekaspencer31
Copy link
Author

But it's not changing the colors to the dark mode colors for BG and the blue body copy.

@eddiesigner
Copy link
Owner

That means that it works with the rest of the colors? I think the issue is because you have repeated colors. Blue Background, Blue Bar and Blue Copy are the same. You should try to reorganise your colors and remove the repeated ones because you don't need them after all, you can create a single color nammed Blue or something. Also you should consider removing Text because is the same color in dark mode.

Please remove all the repeated colors (Text as well) and try again.

@mekaspencer31
Copy link
Author

Still doesn't work. It's actually made it worse. please see my screen recording here. This is a link to my dropbox since the 1 min video is huge. https://www.dropbox.com/sh/u45xgukvffczylx/AABcc9gZZNnMORnm07leJfFva?dl=0

@eddiesigner
Copy link
Owner

That happens because you still have two repeated colors: blue-bg and body-copy.

At the moment you can't have repeated colors even if they have a different name, this is because the only info I can get from layers styles is the hex color value, so internally the plugin searches for that specific value and replaces it with the dark color value, if you have two or more repeated colors then the plugin always convert one of them correctly but not all of them.

Usually this is not a problem because normally in a theme color palette you only have unique colors. Actually I wonder why you want to have the same color twice in light mode and use a different color for each of them in dark mode, normally you want to have a 1:1 mapping to make your design more consistent.

I would suggest to keep only one variation of that blue color.

I will see if I can come up with a solution for this limitation as soon as I have some free time.

@isax5
Copy link

isax5 commented May 5, 2021

I'm having the same issue, I have a few colors that are the same (code) depending if the color is dynamic or not, so I can't give the behavior a want

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

No branches or pull requests

3 participants