Thomas Rast <trast@xxxxxxxxxxxxxxx> writes:
>> I think it is a bug that "git show --word-diff" gives the colored format
>> output when I have "color.ui" configuration.
>>
>> Even though I may have "color.ui = auto" in the configuration, I am
>> telling the command to do a --word-diff, not --color-words, from the
>> command line, and that should override color.ui settings.
>
> I don't really see why the user would forfeit the convenience and
> readability of a color-words diff when the terminal supports colors.
There is one difference between other uses of colors and color-words, but
I can imagine that ordinary people may not have even realized nor thought
about it.
To people who are somewhat but not completely color-challenged (like
myself), it still helps to paint hunk headers in a color that is different
from the body text to make the boundary of each hunk more visible. Even
without the perception of exact color/hue, the contrast alone helps in
that case.
The body of the diff is painted in deleted and added colors; the color is
used only as a way to additionally enhance the output, while still keeping
the plus/minus/SP at the leftmost column. Even to somebody who is
completely color challenged, the necessary information is still there.
Compare --color-words with these. The output uses color as the _only_ way
to say which words appear before and after. The contrast among the three
colors used for the plain body, deleted and added text may still help, or
it may not, to a partially color challenged person.
I also happen to see "--word-diff=color" more as just a customization to
the "plain" output formatting to say "use these byte sequences that happen
to be ANSI color codes, instead of '{+', '+}', etc." than as a part of
what the ui.color switch wants to specify, which is "are various parts of
the output colored to further help distinguishing them visually?"
So color me somewhat biased, but there is a case where the suggestion in
the message you are responding to makes a difference.
I also think --color --word-diff=plain could show "{+new+}" in green and
"[-old-]" in red and that may make things more consistent.
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html