[PATCH] color: issue warning in yellow
Yuya Nishihara
yuya at tcha.org
Tue Aug 21 09:15:56 EDT 2018
On Mon, 20 Aug 2018 11:14:23 -0700, Martin von Zweigbergk via Mercurial-devel wrote:
> On Mon, Aug 20, 2018 at 9:26 AM Kyle Lippincott <spectral at google.com> wrote:
>
> > Bright/bold yellow should be considered unavailable. "Normal" yellow,
> > which is closer to brown or maybe gold on many screens, is fine.
> >
>
> Ah, and we already use yellow for a few things (as you also pointed out).
> Sorry I didn't even check that.
>
>
> > At Google, and I think other tools like clang, generally use magenta I
> > believe, but I have no strong preference. Keep in mind that ui.prompt is
> > also yellow a couple lines below.
> >
>
> Good point about ui.prompt. Yellow seems like the obvious choice for
> warnings, so do we want to change the color for the prompt?
FWIW, I don't like the red-colored "error" since it's hard to spot in
white-on-black screen. Well, it's readable, but not significant. A plain
"yellow" (i.e. dark yellow) would have the same effect.
More information about the Mercurial-devel
mailing list