On Thu, Dec 31, 2015 at 10:07:42AM -0500, S. Gilles wrote:
> A number of terminal emulators (xterm with TERM=xterm-256color and st,
> off the top of my head), interpret the A_BLINK attribute by actually
> blinking, not setting the background color to the bright version. A
> quick way to test this is
> 
>     :color indicator black brightblack
> 
> which on some terms (urxvt) will cause black-on-gray text, and on
> others (xterm) will cause an unreadable blinking line.

...while both terminals support blinking...
 
> This patch changes that behavior by avoiding A_BLINK and directly
> using the bright versions of colors if they are available. I've tested
> it on urxvt, xterm (with TERM=xterm-256color), and st, and it behaves
> as I expect. On xterm (with TERM=xterm), blinking still persists, but
> it always did that anyway, so at least this patch doesn't seem to
> break that case.

...a still better improvement would be to remove the A_BLINK code altogether.

-- 
Thomas E. Dickey <[email protected]>
http://invisible-island.net
ftp://invisible-island.net

Attachment: signature.asc
Description: Digital signature

Reply via email to