[asterisk-bugs] [Asterisk 0013306]: [patch] Optionally use black on white for the terminal settings

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=13306 
====================================================================== 
Reported By:                Corydon76
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   13306
Category:                   Core/General
Reproducibility:            N/A
Severity:                   feature
Priority:                   normal
Status:                     ready for testing
Asterisk Version:           SVN 
SVN Branch (only for SVN checkouts, not tarball releases):  trunk 
SVN Revision (number only!):  
Disclaimer on File?:        N/A 
Request Review:              
====================================================================== 
Date Submitted:             2008-08-14 15:14 CDT
Last Modified:              2008-08-25 15:39 CDT
====================================================================== 
Summary:                    [patch] Optionally use black on white for the
terminal settings
Description: 
Jared Smith schrieb:
> On Thu, 2008-08-14 at 20:35 +0200, Philipp Kempgen wrote:
>> Whenever something spits out lines with a different background
>> color (of a varying runlength!) my eyes start to hurt.
> 
> You can turn of the ANSI color support completely by adding
> "nocolor=yes" to the [options] section of asterisk.conf and then
> restarting Asterisk.

Sure. But I want colored output on my default background color.
:-)

ls with dircolors works perfectly. So I was curious if there
is a reason for Asterisk to behave differently (forcing the
background color to black).

Grüße,
Philipp Kempgen
====================================================================== 

---------------------------------------------------------------------- 
 (0091716) pkempgen (reporter) - 2008-08-25 15:39
 http://bugs.digium.com/view.php?id=13306#c91716 
---------------------------------------------------------------------- 
> If ATTR_DIM is unreadable on your terminal

You don't have to make the patch so it looks best on *my* terminal. :-)
It's just that according to
http://en.wikipedia.org/wiki/ANSI_escape_sequences
dim (faint) is "not widely supported" and all I can say is that it
holds true for my terminal.

> I'm willing to change that to no attribute at all

Ok.

> but ATTR_BRIGHT on a light-colored background creates text that is
> difficult to read. I want to improve readability, not decrease it.

True if code 1 was rendered as bright. According to the Wikipedia page
some terminals render it bright, some bold. Mine does bold text which
increases readability of short light-colored strings (like "[NOTICE]").

But Terminal.app might not be the most popular terminal. So if it works
on a normal X.org terminal, go ahead. Your patch improves the color
handling and fixes the main issue of this bug. Secondly you can't make
everyone happy, and terminal colors are not the most important thing in
the world. :-) 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2008-08-25 15:39 pkempgen       Note Added: 0091716                          
======================================================================