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

[WIP] Add monokai theme #51

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Holzhaus
Copy link
Contributor

This is my first attempt to create a monokai theme (based on the gruvbox
theme and using the colors from terminal.sexy:

http://terminal.sexy/#Jygi8evrSEg-3CVmj8Ap1MluVbzOk1j-VrelrK2hdnFe-idyp-Iu59t1ZtnuroL_Zu_Vz9DC

I then made some changes to dark.css to improve readability.

This is my first attempt to create a monokai theme (based on the gruvbox
theme and using the colors from terminal.sexy:

http://terminal.sexy/#Jygi8evrSEg-3CVmj8Ap1MluVbzOk1j-VrelrK2hdnFe-idyp-Iu59t1ZtnuroL_Zu_Vz9DC

I then made some changes to dark.css to improve readability.
@alphapapa
Copy link
Owner

Hi,

Thanks, this is cool! I ran make screenshots and looked at some of the ones for monokai, and some of them have some readability issues (not referring to the ones that are basically blank because of JavaScript), so I guess it needs some more tweaking. Of course, you did say WIP. :)

I then made some changes to dark.css to improve readability.

Just to be clear, you mean dark.styl, right? Because the CSS files should never be modified by hand.

Please do continue improving this new theme! When it's ready, I'll be happy to merge it.

@Holzhaus
Copy link
Contributor Author

Thanks!

Just to be clear, you mean dark.styl, right? Because the CSS files should never be modified by hand.

Yes, I meant dark.styl. Sorry, I was in a hurry. :-)

Please do continue improving this new theme! When it's ready, I'll be happy to merge it.

Yup, I'll continue working on it as soon as I find the time ;-)

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

Successfully merging this pull request may close these issues.

None yet

2 participants