Re: Setting term-default-fg-color/term-default-bg-color has no effect

[Peter Povinec]

>   > > The problem is that the screen is updated in a lot of places in
>   > > term.el not only term-handle-colors-array. So this could not have
>   > > worked correctly in emacs 21 either.
>   > That is quite possible.  The patch I submitted was intended to merely
> remove
>   > the regression in functionality introduced in Emacs 22.  I believe it
> does
>   > that and is useful on its own, whether additional color handling fixes
> are
>   > applied or not. 
> 
> It's hard to call a regression replacing one type of broken
> behavior with another type of just slightly different broken
> behavior.
It is a complete loss of service to a user who only wanted to customize the
default fg color.  The regression is really that the settings are not honored,
not whether or not setting them results in expected behavior in all the cases.
 As I said, the patch attempts to restore the behavior to Emacs 21 level.  It
does not attempt to fix all related bugs that may exist in term.el.  For all I
know, the fact that the background of empty or short lines is not displayed
with the customized bg color may have been a design limitation, and considered
acceptable when the feature was first introduced.  

> I only see 2 possible fixes for this: either a patch that makes
> changing the default fg and bg for M-x term work correctly, or simply
> removing this configurability for the user. Anything else seems to be
> just busy work. As I said, I am inclined to believe that the right
> thing to do is the latter.
Sure, the full support would be preferred and if you could do that it would be
great.  But removing this configuration functionality completely just because
it continues to have the same unusual behavior in certain cases as it had in
prior releases would seem like a busy work to me...

--Peter



       
____________________________________________________________________________________
Looking for a deal? Find great prices on flights and hotels with Yahoo! 
FareChase.
http://farechase.yahoo.com/