[m-rev.] for post-commit review: add color in det_report.m
Julien Fischer
jfischer at opturion.com
Wed May 1 23:49:04 AEST 2024
On Tue, 30 Apr 2024, Zoltan Somogyi wrote:
> On 2024-04-29 20:57 +10:00 AEST, "Zoltan Somogyi" <zoltan.somogyi at runbox.com> wrote:
>>> "int" and "uint8" are now colored, but nothing else is. I think there
>>> is a case to be made that the variable "N", which is the subject of the
>>> message should stand out mode.
>>
>> That is a good idea. Internally, we could call that color "color_subject",
>> as it denotes the subject of the error messsage. Does anyone have
>> a better name in mind? And what should the default color shade for be?
>> Some kind of blue, given that red, green and yellow are already in use
>> for other things? Or should we rejig the existing colors?
>
> Unless I hear some other opinions by then, about this time tomorrow I intend to
>
> - add color_subject, with the default shade being a light cyan, probably 8-bit color 87, and
color_subject is fine; I was going to suggest bright white as the color,
but light cyan works as well.
Julien.
More information about the reviews
mailing list