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

Light Yellow instead of Yellow for Warning Color in Windows #979

Closed
M-Fabian opened this issue Aug 2, 2017 · 6 comments
Closed

Light Yellow instead of Yellow for Warning Color in Windows #979

M-Fabian opened this issue Aug 2, 2017 · 6 comments

Comments

@M-Fabian
Copy link

M-Fabian commented Aug 2, 2017

This is only a tiny issue. The non-light colors in Win32 Console are all very dim and boring. Hence, you already use the bright green and bright red. I wonder if you could also use bright yellow for warnings. You use Colour::Yellow for warnings and reconstructed expressions. I have no opinion about whether reconstructed expressions should be yellow or bright yellow.

Since I would like to use bright yellow in my reporter, I would already be happy if you added Colour::BrightYellow.

Also, it took me a while to understand that there are no colors when a debugger is active. I found --use-color, but I don't understand why color and the debugger could harm each other. Could you please explain this briefly?

@horenmar
Copy link
Member

horenmar commented Aug 2, 2017

I'll test how the different colours would look later, but

Also, it took me a while to understand that there are no colors when a debugger is active. I found --use-color, but I don't understand why color and the debugger could harm each other. Could you please explain this briefly?

You are not the first one to ask this, see #590. Looking through that and performing a cursory google search says that some debug consoles (in IDEs and stuff) can run into trouble with escape codes.

I actually disabled that check for Catch 2 and it is likely to stay disabled (users with trouble can run catch with --use-colour no).

@M-Fabian
Copy link
Author

M-Fabian commented Aug 2, 2017

I see. Fingers crossed for BrightYellow ;-)

@M-Fabian
Copy link
Author

Now that version 2 is out, I had hoped for BrightYellow. It is actually quite simple:

case Colour::BrightYellow: return setTextAttribute( FOREGROUND_RED | FOREGROUND_GREEN | FOREGROUND_INTENSITY);

Would you please consider adding BrightYellow, as the normal Yellow in Windows Console is muddy and dim...

@M-Fabian
Copy link
Author

M-Fabian commented Feb 1, 2018

Awesome, thank you! Catch continues to be my favorite Unittest framework :-)

@horenmar
Copy link
Member

horenmar commented Feb 1, 2018

Thanks, and sorry for the long wait. The issue kinda slipped my mind and the backlog is still huge.

@M-Fabian
Copy link
Author

Could you please also add BrightYellow to Catch1?
It seems that Catch1 is still under active development, which I appreciate since I cannot switch to Catch2 yet (because of my old compiler I am bound to use in my current projects),
Please consider it for the next release 1.12.3.

I cannot reopen this issue. Do I need to make a new issue for this request?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants