Error message color does not have enough contrast with default console backgrounds about knack HOT 7 CLOSED

@derekbekoe I would like to take this up.

from knack.

@invinciblycool that would be great!

from knack.

The library should honor the hosting console's (the user's preferences) display colors and not choose an arbitrary color.

This is an accessibility issue and should be given appropriate priority. It should really not be considered a feature request.

from knack.

I made this change to my local repo, but am not sure how to go about testing it where I want to see the change ultimately effected.

Is there a script that will make a locally usable knack that I can use in a local copy of azure-cli, that i can then build and run? Or should I just pip install the knack package into the azure-cli and edit there?

from knack.

If you have cloned the repo, you should be able to go to the root of that repo and run pip install -e <path to knack setup.py file> --ignore-installed.

The other option would be to modify the code in you virtual environment's site-packages folder where you have the CLI installed.

from knack.

@StingyJack Here is the difference in the two colors (LIGHTRED_EX on top, RED at bottom) on windows command prompt. I also tried on powershell and the LIGHTRED_EX seems much better in contrast.

from knack.

Fixed by PR #147.

from knack.