[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Nano-devel] [PATCH 1/2] display: let the title bar show when nano i
From: |
David Ramsey |
Subject: |
Re: [Nano-devel] [PATCH 1/2] display: let the title bar show when nano is in linting mode |
Date: |
Mon, 22 Oct 2018 23:25:00 -0500 |
Chiming in here. I don't really use the linter, but some of the
proposed design decisions are problematic.
Benno Schulenberg:
> since in linting mode no text can be marked, selectedcolor is a "free
> color".
I'd agree with Brand here: using selectedcolor for text that isn't
actually selected is misleading. And if it impacts accessibility,
that's another problem.
Brand Huntsman:
> The status type is called NOTICE which implies any important message
> that should be more visible could be a NOTICE, even outside the
> linter. If this was only for the linter it would be called LINTER.
> Improper naming will lead to it getting used outside the linter at
> some point in the future.
Using NOTICE this way also seems odd to me if it's tied only to the
linter.
Brand Huntsman:
> You created an entirely new status type just for linter messages, why
> can't that type also have its own color, so that other more important
> non-linter non-error status messages could be more visible.
This is a good idea. If we have "errorcolor" for important error
messages, why not have a "noticecolor" for important non-error messages
with the NOTICE status type, instead of trying to stretch the existing
design?
- Re: [Nano-devel] [PATCH 1/2] display: let the title bar show when nano is in linting mode, (continued)
- Re: [Nano-devel] [PATCH 1/2] display: let the title bar show when nano is in linting mode, Brand Huntsman, 2018/10/18
- Re: [Nano-devel] [PATCH 1/2] display: let the title bar show when nano is in linting mode, Benno Schulenberg, 2018/10/21
- Re: [Nano-devel] [PATCH 1/2] display: let the title bar show when nano is in linting mode, Brand Huntsman, 2018/10/21
- Re: [Nano-devel] [PATCH 1/2] display: let the title bar show when nano is in linting mode, Benno Schulenberg, 2018/10/22
- Re: [Nano-devel] [PATCH 1/2] display: let the title bar show when nano is in linting mode, Brand Huntsman, 2018/10/22
- Re: [Nano-devel] [PATCH 1/2] display: let the title bar show when nano is in linting mode, Benno Schulenberg, 2018/10/23
- Re: [Nano-devel] [PATCH 1/2] display: let the title bar show when nano is in linting mode, Brand Huntsman, 2018/10/23
Re: [Nano-devel] [PATCH 1/2] display: let the title bar show when nano is in linting mode, Marco Diego Aurélio Mesquita, 2018/10/18
Re: [Nano-devel] [PATCH 1/2] display: let the title bar show when nano is in linting mode,
David Ramsey <=
Re: [Nano-devel] [PATCH 1/2] display: let the title bar show when nano is in linting mode, David Ramsey, 2018/10/23