-
Notifications
You must be signed in to change notification settings - Fork 33
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
Add dark theme #27
base: master
Are you sure you want to change the base?
Add dark theme #27
Conversation
As you can see I am not a pro designer but it looks better then the bright one to me. Do you want me to add a screenshot to README.md? |
ping |
Please darken the red background on the file name a bit. Otherwise this does the job. |
I don't like the colors, I'm not gonna merge this. make something more decent based on one of the popular dark themes and I'll merge it |
Thoughts? |
@nivir, any thoughts? |
@stoivo personally I never like the dark background and red color, not sure what you can improve. For me I am happy as it is now with Mac Classic.tmTheme :) |
Since I'm also a tubnil user, purple and pink are perfectly fine for me. I also considered the red to be too offensive and distracting before. |
I agree, it was too offensive. |
ping |
pong @aziz @FichteFoll |
I am tired of waiting, any feedback is better than this... |
Not my repo, but I mentioned earlier that I like the latest iteration of this. Generally, it would seem better to have the custom syntax definition use some of the standardized scope names instead of having to bundle custom color schemes, but this would need some exploration. Having a bright and dark scheme should work as a band-aid. |
@aziz hello |
ping @aziz |
Ideally the custom syntax would just use normal scope names and |
@FichteFoll, I might take a look at that. If/when I get the time. |
Honestly, why has this still not been implemented? The hard work was already taken care of. At least implement this temporary workaround. Such a crucial plugin needs better styling. Why can't this plugin simply leave the styling to the app's default, basically matching the user's currently applied Sublime Text theme? That could happen by not including any styling, or perhaps setting this plugin's styling to match the app's. |
closes #26