[PATCH 4 of 4] [RFC] color: support different styling depending on color support (BC)
Jun Wu
quark at fb.com
Mon Jul 10 00:32:21 EDT 2017
I think the series is a good improvement and can be queued with some small
in-flight fixes.
Excerpts from Gregory Szorc's message of 2017-07-09 16:46:17 -0700:
> diff --git a/mercurial/color.py b/mercurial/color.py
> --- a/mercurial/color.py
> +++ b/mercurial/color.py
> @@ -135,13 +135,31 @@ except ImportError:
> 'diff.inserted': 'green',
> 'diff.tab': '',
> 'diff.trailingwhitespace': 'bold red_background',
> - 'changeset.public' : '',
> - 'changeset.draft' : '',
> - 'changeset.secret' : '',
> + 'changeset.public': {
> + 8: '',
> + 16: 'brightred',
This is mostly nitpicking, but red seems to be mostly about errors or
warnings. I'd vote "brightgreen" for public changesets. Or make public
brightyellow and draft non-bright yellow.
> [...]
Not only in colortable, it seems config support is also worthwhile, like:
[color]
changeset.public:8 = ...
changeset.public:16 = ...
That could be a future patch.
More information about the Mercurial-devel
mailing list