Re: [Nano-devel] [PATCH 1/2] display: let the title bar show when nano is in linting mode

[Brand Huntsman]

On Mon, 22 Oct 2018 20:56:36 +0200
Benno Schulenberg <address@hidden> wrote:

> since in linting mode no text can be marked, selectedcolor is a "free
> color".

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.


> > You are taking something that currently works and breaking it  
> 
> How is it broken?

I can't read selected text, I know what it said before selecting it and only 
need to see the boundaries of the selection. Using it for linter messages means 
I can't read them. Interface colors was a huge accessibility feature and 
overloading how those colors are used is a huge step backward. I understand 
your position though, most developers don't care about accessibility because it 
works fine for them. Life would be easier for everyone if disabilities didn't 
exist.

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. I made the promptcolor patch 
so prompts wouldn't look like the titlebar, purely decorative. But the 
yesnocolor patch was to highlight important prompt messages that need 
attention, just like errorcolor highlights important status messages. 
errorcolor, noticecolor and yesnocolor are accessibility features that benefit 
all users equally. And the purely decorative unicode arrows in help use more 
code and other resources than adding a couple new interface colors.


> > Those two are normal status messages, no different from other
> > status messages in nano.  
> 
> They are *not* normal status-bar messages

I'm an idiot, sorry about that. I didn't know the patch had the new napms fix 
and thought the NOTICE's were on the first/last messages.