jonathanchu / atom-one-dark-theme

Atom One Dark - An Emacs port of the Atom One Dark theme from Atom.io.
GNU General Public License v3.0
256 stars 52 forks source link

js2-mode: Initial support #21

Closed whitlockjc closed 7 years ago

whitlockjc commented 7 years ago

This commit adds initial support for js2-mode. This is not a 100% complete representation but that is more due to the differences between how Atom and Emacs theme in general and less about the work. For the most part this is straight forward but there is support for face-remapping to allow for a more accurate theme. This can be disabled if necessary and is documented in the source.

For the face-remapping, I copied the same approach I used in my own Atom Dark theme for emacs: https://github.com/whitlockjc/atom-dark-theme-emacs/blob/master/atom-dark-theme.el

Note: I was unable to make the code within the face-remapping hook work with your atom-one-dark-with-color-variables macro so I just copied/pasted the style (with comment) instead of referencing the theme-specific color by name. If I can figure this out, I will update the PR.

whitlockjc commented 7 years ago

Example

image

Discrepancies

Below are the noticeable discrepancies between Atom and Emacs (first occurrence type):

whitlockjc commented 7 years ago

bump

jonathanchu commented 7 years ago

@whitlockjc I am really sorry for the delay here. 👊 😵

Noted on what you did here. I think this a great start and thanks for laying out the discrepancies. I'm inclined to merge this and get this foundation in so we all can fix it up as we see fit in the near future.

Thanks and great work, as always 😍

whitlockjc commented 7 years ago

Happy to help. :)