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

Missing font face definitions from Alchemist, maybe port them in? #11

Open
jsmestad opened this issue May 17, 2021 · 3 comments
Open

Missing font face definitions from Alchemist, maybe port them in? #11

jsmestad opened this issue May 17, 2021 · 3 comments

Comments

@jsmestad
Copy link

When using doom-themes, I was hitting an issue where the compilation buffer output was difficult to read. I was going to customize the font-lock values in the compilation buffer, but it appears that the buffer does not have any defined.

Looking at alchemist.el's implementation, it looks like it had several faces defined.

I am not an expert on defface and all that, so please correct me if I am mistaken on my approach to customizing the output.

Example

Screen Shot 2021-05-17 at 09 10 03

Screen Shot 2021-05-17 at 09 25 31

@ananthakumaran
Copy link
Owner

ananthakumaran commented May 17, 2021

These colors come from the mix test ... command output, not via emacs font-locking. When we run mix test, it displays colors via ANSI escape codes

image

which then get converted to colors by ansi-color-apply-on-region

image

The same issue exists on the terminal as well, the filename is not readable on dark themes. The defface you pointed out on alchemist is not controlling the test output, but used in other places. If you really want to disable the ANSI colors, you could do (setq exunit-mix-test-default-options '("--no-color")), but other useful things like error diffs won't get colors as well. Another option is to change the color used by mix cli formatter

@jsmestad
Copy link
Author

Good to know. I was not sure how to even start debugging this 👏

jsmestad referenced this issue in axelson/exunit.el Nov 20, 2021
@ryoung786
Copy link

ryoung786 commented Jul 6, 2023

I ended up fixing this in my emacs (v30.0.50) by customizing ansi-color-black. MediumPurple2 worked best for my dark theme, but ymmv.

(custom-set-faces
 ;; other customizations ...
 '(ansi-color-black ((t (:background "MediumPurple2" :foreground "MediumPurple2")))))

The entire Ansi-Colors customization group is helpful too.

Screenshot 2023-07-06 at 11 21 52

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