From: "Aneesh Kumar K.V" <aneesh.kumar@linux.vnet.ibm.com>
To: Carl Worth <cworth@cworth.org>
Cc: notmuch@notmuchmail.org
Subject: Re: [PATCH -v2] notmuch.el: Add face support to search and show mode
Date: Sun, 22 Nov 2009 17:51:06 +0530 [thread overview]
Message-ID: <20091122122106.GA3531@skywalker.linux.vnet.ibm.com> (raw)
In-Reply-To: <87hbsmg436.fsf@yoom.home.cworth.org>
On Sun, Nov 22, 2009 at 01:02:21PM +0100, Carl Worth wrote:
> On Sun, 22 Nov 2009 14:07:56 +0530, "Aneesh Kumar K.V" <aneesh.kumar@linux.vnet.ibm.com> wrote:
> > This add two faces, notmuch-show-subject-face and
> > notmuch-tag-unread-face. The first face is used to show the subject
> > line in the notmuch-show-mode and the second one to show the unread
> > tag in the notmuch-search-mode. We can set additional tags by setting
> > notmuch-tag-face-alist as below
>
> Hi Aneesh,
>
> I've been meaning to follow-up with you on this since I did get it
> working.
>
> > (setq notmuch-tag-face-alist '(("unread" . 'notmuch-tag-unread-face)
> > ("inbox" . 'notmuch-tag-inbox-face)))
>
> It's interesting to be able to highlight specific tags like this. But I
> think that first I'd like to have a single 'notmuch-tag face to make it
> easier to see all the tag names, since they're hard to see among the
> subject. And then maybe after that have s 'notmuch-tag-important-face or
> something. I'm guessing that just that much would cover most people's
> needs, before needing to choose custom colors for each tag name.
The default search pattern for notmuch is inbox and unread right. So adding
both of them to the notmuch-tag-face-alist should color the tags properly.
>
> > +(defface notmuch-show-subject-face
> > + '((((class color) (background light)) (:foreground "yellow" :bold t))
> > + (((class color) (background dark)) (:foreground "yellow" :bold t)))
> > + "Notmuch show mode face used to highligh subject line."
> > + :group 'notmuch)
>
> Color selection can be pretty difficult, and I don't think we're ever
> all going to agree on things. But I can at least say that I can't read
> yellow on white at all, (maybe yellow on black works out a bit better, I
> don't know).
>
> So I'd definitely like to hear people's ideas on what the presentation
> should look like here.
>
> First, the current reverse-video is definitely not something I really
> like. That was just a placeholder since I didn't know how to configure
> other faces, (and I needed _something_ to distinguish one message from
> another).
>
> Second, from the time I spent with sup, I found that it was hard for me
> to read entire lines of bold text. So I think I'd rather just have the
> header name ("Subject:", for example) in bold and the rest of the
> subject not bold. Hmm... maybe we could use a "From:" at the beginning
> of the one line summary.
>
> Finally, for separating one message from the other, I think I'd really
> just like a one-pixel line drawn between the last line of text of one
> message and the first line of text of the next message. Does anybody
> know how to achieve that?
>
> Back to the patch, I think I'll be able to push it as soon as I can
> apply it and still _see_ the subject lines. ;-)
One of the easy part with emacs is we can use customize-face and select
notmuch-show-subject-face to select what ever suites your configuration.
For me with black background i found yellow with bold to be nice.
-aneesh
next prev parent reply other threads:[~2009-11-22 12:21 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-11-22 8:37 [PATCH -v2] notmuch.el: Add face support to search and show mode Aneesh Kumar K.V
2009-11-22 12:02 ` Carl Worth
2009-11-22 12:21 ` Aneesh Kumar K.V [this message]
2009-11-22 22:48 ` Carl Worth
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
List information: https://notmuchmail.org/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20091122122106.GA3531@skywalker.linux.vnet.ibm.com \
--to=aneesh.kumar@linux.vnet.ibm.com \
--cc=cworth@cworth.org \
--cc=notmuch@notmuchmail.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.
Code repositories for project(s) associated with this public inbox
https://yhetil.org/notmuch.git/
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 read-only IMAP folder(s) and NNTP newsgroup(s).