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

A question about the colors used in atom-one-dark-theme.el #15

Open
ghost opened this issue Jun 2, 2016 · 5 comments
Open

A question about the colors used in atom-one-dark-theme.el #15

ghost opened this issue Jun 2, 2016 · 5 comments
Assignees

Comments

@ghost
Copy link

ghost commented Jun 2, 2016

Hi,

I was browsing through the source code of one-dark-syntax and I couldn't find the following colors that have been used in atom-one-dark-theme:

https://github.com/jonathanchu/atom-one-dark-theme/blob/master/atom-one-dark-theme.el#L22
https://github.com/jonathanchu/atom-one-dark-theme/blob/master/atom-one-dark-theme.el#L23
https://github.com/jonathanchu/atom-one-dark-theme/blob/master/atom-one-dark-theme.el#L36
https://github.com/jonathanchu/atom-one-dark-theme/blob/master/atom-one-dark-theme.el#L37

Is it the case that I wasn't able to find these colors in the one-dark-syntax repo or did you use these colors independently?

Thanks,
sixpointzero

@jonathanchu
Copy link
Owner

Hey @sixpointzero! Hmm, it could be one of three things here:

  1. The official one-dark-syntax color was modified since porting over this theme. If that's the case, these can be updated on our end.
  2. It was my closest approximation to the color match and might be slightly off, hence the color not found.
  3. I messed up :)

I can take a look at these in more detail tonight and verify the colors. However, feel free to correct these if they don't seem right to you!

@jonathanchu jonathanchu self-assigned this Jun 2, 2016
@jonathanchu jonathanchu reopened this Nov 2, 2016
@jonathanchu
Copy link
Owner

Whoops, accidentally closed this one out. Re-opened.

@jakubdonovan
Copy link

I cannot find this package in melpa. Where did it go and was this fixed?

@jonathanchu
Copy link
Owner

@jakubdonovan hey! I haven't verified if these colors are exact or up-to-date with the latest official Atom One Dark theme yet.

And also, yes, it is still on MELPA - https://melpa.org/#/atom-one-dark-theme

@jhacksworth
Copy link
Collaborator

TL;DR: The only color variable that I can't account for is atom-one-dark-bg-1.

I amended the links in @sixpointzero's original question to point to the state the repo was in at the time of the question, Jun 2, 2016, instead of master, so that we can understand what he's referring to. I'm also including the names of the color variables in the link names, for ease of reference:

(These links are to an outdated version of this theme)

Here is the current version of the theme at the time of this writing, on Jan 31, 2018 (Note that the values for atom-one-dark-silver and atom-one-dark-black have been updated):

I created a gist with a mapping of Atom One Dark colors to this theme's colors in this gist:

Here is a sample output of that mapping (when plugged into the LESS CSS compiler):

The only color variable that I can't account for is atom-one-dark-bg-1. If we want to only use colors defined in the Atom theme, I think that a good candidate for atom-one-dark-bg-1 is @input-background-color, which has a hex color value of: #1b1d23

I hope this makes sense and isn't a confusing mess. 😃

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