[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: |
Brand Huntsman |
Subject: |
Re: [Nano-devel] [PATCH 1/2] display: let the title bar show when nano is in linting mode |
Date: |
Mon, 22 Oct 2018 19:30:50 -0600 |
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.
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