Re: [PATCH] help: colorize man pages

On 21/05/2021 23:20, Felipe Contreras wrote:

Igor Djordjevic wrote:

quoted

If I may, NO_COLOR approach seems to be rather straightforward to me, 
as per description on their homepage[1] - make all software supporting 
it behave as colors are an opt-in feature, thus disabled by default.

May I ask you how you interpret this?

  It is reasonable to configure certain software such as a text editor
  to use color ... sparingly

Sure, but to make the point (hopefully) even more obvious, let me 
quote the whole part:

  It is reasonable to configure certain software such as a text editor 
  to use color or other ANSI attributes sparingly (such as the reverse 
  attribute for a status bar) while still desiring that other software 
  not add color unless configured to. It should be up to the user 
  whether color is used, not the software author.

I understand it exactly as (I think) it says - it is reasonable to 
allow (the user, not developer!) to configure certain software to 
(still) use color (fully or sparingly should not even matter, and it 
may depend on what kind of granular configuration software allows in 
the first place, if any), even if his (user's) general ("default") 
preference is to have no colors.

Thus color should be user opt-in - NO_COLOR turns all of it off by 
default (for all software supporting it), and user decides which color 
to turn back on through each specific software color configuration.

That last sentence should make it clear - "it should be up to the 
user whether color is used, not the software author".

So it shouldn't matter what does software author think about which 
parts of software should be (fully or sparingly) colored (by default) 
- NO_COLOR's idea is to give the ultimate power to the user to 
decide, and on a global level, starting with no colors by default, 
then allowing colors where desired, per each specific software config 
(instead of vice-versa, being required to turn color off per each 
specific software, where color is otherwise used by default).

At least that's how I understand all of it, making sense to me, but I 
don't mind discussing it further, if needed.