Re: [PATCH] emacs: notmuch-search: fix faces

Subject: Re: [PATCH] emacs: notmuch-search: fix faces

Date: Tue, 28 Aug 2012 01:11:05 +0200

To: Jameson Graef Rollins, notmuch@notmuchmail.org

Cc:

From: Michal Nazarewicz


> On Thu, Aug 23 2012, Michal Nazarewicz <mpn@google.com> wrote:
>> For some reason the faces do not get applied when 'face property is
>> used, but they work correctly with 'font-lock-face property.  This
>> commit changes notmuch-search to use the latter.

Jameson Graef Rollins <jrollins@finestructure.net> writes:
> Hi, Michal.  Can you say a bit more about what's motivating this?  This
> feature seems to work fine for me, so I would like to understand what's
> not working for you.  I'm certainly no expert on emacs font handling,
> though, so maybe this is the proper thing to do.  Thanks.

I'm not an expert either, but with this patch applied I see colours,
without this patch, I don't see colours, ie. everything is rendered
using the default face.

I'm also not entirely sure if that's the correct way of doing things
since 'face seems to be working in other modes (most notably
notmuch-show).  By posting, I'm also hoping that someone more
experienced will maybe take a look to see what's going on here... ;)

For some more info, in notmuch-show mode, I get-text-property of nil for
both 'face and 'font-lock-face, but in message-mode or in *scratch*
buffer I'm getting nil for 'font-lock-face but non-nil for 'face.  With
the patch, I'm getting the same non-nil for both 'face and
'font-lock-face; without the patch, I'm getting nil for both.

-- 
Best regards,                                         _     _
.o. | Liege of Serenely Enlightened Majesty of      o' \,=./ `o
..o | Computer Science,  Michał “mina86” Nazarewicz    (o o)
ooo +----<email/xmpp: mpn@google.com>--------------ooO--(_)--Ooo--

Thread: