Skip to content
This repository was archived by the owner on Oct 10, 2023. It is now read-only.

Update UI variables and add theme colors#2

Merged
NSHenry merged 14 commits intoNSHenry:masterfrom
jackiehluo:solarized
Mar 12, 2016
Merged

Update UI variables and add theme colors#2
NSHenry merged 14 commits intoNSHenry:masterfrom
jackiehluo:solarized

Conversation

@jackiehluo
Copy link

Hey, @NSHenry! I'm part of the Nylas team, and we're about to land a new visual theme picker that'll allow people to preview and view color palettes for themes. We pull a few major colors from the theme's UI variables to create the palette, but we've also added a way to override them manually with a theme-colors.less file for themes (like yours) that aren't conveyed well with the variables we selected.

Before:
screen shot 2016-03-03 at 12 23 05 pm

After:
screen shot 2016-03-03 at 12 24 19 pm

I also pulled everything into the root directory for the project, since we've had a lot of people who are confused when they try to install using that directory and it doesn't work, and it might also be easier for them to git pull updates there. Let me know what you think!

@jackiehluo jackiehluo changed the title Solarized Update UI variables and add theme colors Mar 3, 2016
NSHenry added a commit that referenced this pull request Mar 12, 2016
Update UI variables and add theme colors
@NSHenry NSHenry merged commit da2ec89 into NSHenry:master Mar 12, 2016
@NSHenry
Copy link
Owner

NSHenry commented Mar 12, 2016

Apologies for the ridiculously late reply I didn't realize that it wasn't notifying me. I am so sorry. I'll commit to more actively stay on top of this.

@jackiehluo
Copy link
Author

@NSHenry No worries, thanks for merging! 👍

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants