-
-
Notifications
You must be signed in to change notification settings - Fork 176
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 'ansi-color-*' faces for Emacs 28.1 #433
Conversation
It seems to me there's room for improvement, but I don't have time to dive deeper into this. I guess we can refine the faces down the road. Thanks for tackling this! |
I was holding off merging this because they maybe should be the solarized 16 color palette rather than a custom one. But sure, lets go with this for now., There is some setting for a |
Roger that. I just figured that some Solarized colors were better than the default colors. Anyways, those are type of things we can always address incrementally. |
I'm not sure what you mean by the "16 color palette" @thomasf , but if you point me in the right direction I'm happy to try to do a follow up patch. |
From the official solarized page there is a termcol mapping column in the mapping list
However it does some times cause problems of it's own with some terminal programs so some times you might want something else and that was why I didn't opt for reverting this merge. It generally works very well though. |
I see, thanks! I was unaware of that. I can definitely understand wanting to match the official website, however the mappings of the brights colors to greys seem very odd to me (e.g. "bright green" to "base01"). |
That is how the original palette is designed and terminal emulator themes generally follows that convention. I like the subdued look of solarized and not using too many colors where it is avoidable is a pretty big part of it. |
Emacs 28 introduced new faces for ANSI color codes, that are applied by the
ansi-color-compilation-filter
function. This adds Solarized color mappings for those faces.I don't know whether there are great options for the 'black' and 'white' faces (and their 'bright' variants). I left 'white' at its default, and made the best choice I could for 'black', that hopefully makes sense for both dark and light palettes. I'm very open to alternate suggestions for those.
Default colors:
With this patch:
Before submitting a PR make sure the following things have been done (and denote this
by checking the relevant checkboxes):
Thanks!