all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Peter Povinec <pp_publiclists@yahoo.com>
To: Dan Nicolaescu <dann@ics.uci.edu>
Cc: bug-gnu-emacs@gnu.org
Subject: Re: Setting term-default-fg-color/term-default-bg-color has no effect
Date: Thu, 2 Aug 2007 23:35:48 -0700 (PDT)	[thread overview]
Message-ID: <774795.43084.qm@web57303.mail.re1.yahoo.com> (raw)
In-Reply-To: <200708020228.l722SeaF022313@oogie-boogie.ics.uci.edu>

>   > > 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/

      reply	other threads:[~2007-08-03  6:35 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-06-30  0:17 Setting term-default-fg-color/term-default-bg-color has no effect Peter Povinec
2007-07-02 17:28 ` Dan Nicolaescu
2007-07-23 19:22   ` Peter Povinec
2007-08-01  4:10     ` Dan Nicolaescu
2007-08-02  0:50       ` Peter Povinec
2007-08-02  2:28         ` Dan Nicolaescu
2007-08-03  6:35           ` Peter Povinec [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=774795.43084.qm@web57303.mail.re1.yahoo.com \
    --to=pp_publiclists@yahoo.com \
    --cc=bug-gnu-emacs@gnu.org \
    --cc=dann@ics.uci.edu \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.