On 2021-05-17 at 16:48:04, Ævar Arnfjörð Bjarmason wrote: > This looks much better. > > I wonder a good follow-up (hint, hint! :) would be to have > exec_man_man() and exec_man_cmd() in builtin/help.c set this depending > on color.ui (so we'd do it by default with "auto"). > > Then e.g. "git help git" would look prettier than "man git". As I mentioned on the patch itself, I'd prefer if Git didn't do this. I have my own colors configured and don't want Git to render its man output differently from what I have. Even if I didn't, I wouldn't want Git to change the output of man(1) to be different from what's on the system. I should point out that I have my shell configuration set up to use different colors depending on the capability of the terminal, such as using a 256-color palette when that's supported and a 16-color palette when it's not, so there is literally no configuration that Git can provide here that matches my existing settings. Additionally, colors tend to pose accessibility problems for a lot of people. I have normal color vision, but because I use a transparent background which renders as grey, the standard terminal red is nearly illegible for me. I also know people with colorblindness who have problems with various colors or any colors at all. -- brian m. carlson (he/him or they/them) Houston, Texas, US
Attachment:
signature.asc
Description: PGP signature