Gnus development mailing list
 help / color / mirror / Atom feed
From: Didier Verna <didier@lrde.epita.fr>
Cc: ding@gnus.org
Subject: Re: Faces of the wrong order
Date: Thu, 30 Jun 2005 01:40:35 +0200	[thread overview]
Message-ID: <muxr7ekwx3g.fsf@uzeb.lrde.epita.fr> (raw)
In-Reply-To: <b9y3br0wy5h.fsf_-_@jpl.org> (Katsumi Yamaoka's message of "Thu, 30 Jun 2005 08:17:46 +0900")

Katsumi Yamaoka <yamaoka@jpl.org> wrote:

> I don't think you corrected the order. This article contains three Faces and
> three X-Faces in order of Face-1, Face-2, Face-3, X-Face-1, X-Face-2 and
> X-Face-3 in the header. WDYT?

        Sorry, I can't see them right now, I'm on a tty (who wants to hack
konsole to display the faces by modifying the background pixmap ? :-)

        However, I'm suspicious about your own order of face insertion. This
actually fixed a problem for me, and if you look at the code of
article-display[-x]-face, you'll see that previously, the faces and x-faces
lists were built in a different order. So my patch actually builds these lists
in the same order now (admitedly, in the opposite order from insertion in the
message, but AFAICR that was the case when this feature was first
implemented).

So, I'm not strongly opposed to revert the face order again (although I
believe that would break old code like mine), but then, the x-face order
should also be reverted in order to stay consistent.


-- 
Didier Verna, didier@lrde.epita.fr, http://www.lrde.epita.fr/~didier

EPITA / LRDE, 14-16 rue Voltaire   Tel.+33 (1) 44 08 01 85
94276 Le Kremlin-Bicêtre, France   Fax.+33 (1) 53 14 59 22   didier@xemacs.org



  reply	other threads:[~2005-06-29 23:40 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <E1DneB5-0007RY-00@quimby.gnus.org>
2005-06-29 17:37 ` Changes committed gnus/lisp (ChangeLog gnus-art.el) Romain Francoise
2005-06-29 23:25   ` Didier Verna
2005-06-30  5:37     ` Romain Francoise
2005-06-29 23:17 ` Faces of the wrong order (was Re: Changes committed gnus/lisp (ChangeLog gnus-art.el)) Katsumi Yamaoka
2005-06-29 23:40   ` Didier Verna [this message]
2005-06-29 23:59     ` Faces of the wrong order Katsumi Yamaoka
2005-06-30  3:18       ` Katsumi Yamaoka
2005-06-30 14:15         ` Adam Sjøgren
2005-07-01 14:34   ` Didier Verna
2005-07-01 16:05     ` Katsumi Yamaoka

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=muxr7ekwx3g.fsf@uzeb.lrde.epita.fr \
    --to=didier@lrde.epita.fr \
    --cc=ding@gnus.org \
    /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.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).