87/74c460483bfda76426d610fd472899ff8e890a

   1 Return-Path: <cworth@cworth.org>\r
   2 X-Original-To: notmuch@notmuchmail.org\r
   3 Delivered-To: notmuch@notmuchmail.org\r
   4 Received: from localhost (localhost [127.0.0.1])\r
   5         by olra.theworths.org (Postfix) with ESMTP id 00BB5431FBF;\r
   6         Sun, 22 Nov 2009 04:02:37 -0800 (PST)\r
   7 X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
   8 Received: from olra.theworths.org ([127.0.0.1])\r
   9         by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
  10         with ESMTP id yR0JaVgeA9zK; Sun, 22 Nov 2009 04:02:36 -0800 (PST)\r
  11 Received: from cworth.org (localhost [127.0.0.1])\r
  12         by olra.theworths.org (Postfix) with ESMTP id 87B32431FAE;\r
  13         Sun, 22 Nov 2009 04:02:35 -0800 (PST)\r
  14 From: Carl Worth <cworth@cworth.org>\r
  15 To: "Aneesh Kumar K.V" <aneesh.kumar@linux.vnet.ibm.com>,\r
  16         notmuch@notmuchmail.org\r
  17 In-Reply-To:\r
  18  <1258879076-31883-1-git-send-email-aneesh.kumar@linux.vnet.ibm.com>\r
  19 References:\r
  20  <1258879076-31883-1-git-send-email-aneesh.kumar@linux.vnet.ibm.com>\r
  21 Date: Sun, 22 Nov 2009 13:02:21 +0100\r
  22 Message-ID: <87hbsmg436.fsf@yoom.home.cworth.org>\r
  23 MIME-Version: 1.0\r
  24 Content-Type: text/plain; charset=us-ascii\r
  25 Subject: Re: [notmuch] [PATCH -v2] notmuch.el: Add face support to search\r
  26  and show mode\r
  27 X-BeenThere: notmuch@notmuchmail.org\r
  28 X-Mailman-Version: 2.1.12\r
  29 Precedence: list\r
  30 List-Id: "Use and development of the notmuch mail system."\r
  31         <notmuch.notmuchmail.org>\r
  32 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  33         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  34 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  35 List-Post: <mailto:notmuch@notmuchmail.org>\r
  36 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  37 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  38         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  39 X-List-Received-Date: Sun, 22 Nov 2009 12:02:37 -0000\r
  40 \r
  41 On Sun, 22 Nov 2009 14:07:56 +0530, "Aneesh Kumar K.V" <aneesh.kumar@linux.vnet.ibm.com> wrote:\r
  42 > This add two faces, notmuch-show-subject-face and\r
  43 > notmuch-tag-unread-face. The first face is used to show the subject\r
  44 > line in the notmuch-show-mode and the second one to show the unread\r
  45 > tag in the notmuch-search-mode. We can set additional tags by setting\r
  46 > notmuch-tag-face-alist as below\r
  47 \r
  48 Hi Aneesh,\r
  49 \r
  50 I've been meaning to follow-up with you on this since I did get it\r
  51 working.\r
  52 \r
  53 > (setq notmuch-tag-face-alist '(("unread" . 'notmuch-tag-unread-face)\r
  54 >                                ("inbox" . 'notmuch-tag-inbox-face)))\r
  55 \r
  56 It's interesting to be able to highlight specific tags like this. But I\r
  57 think that first I'd like to have a single 'notmuch-tag face to make it\r
  58 easier to see all the tag names, since they're hard to see among the\r
  59 subject. And then maybe after that have s 'notmuch-tag-important-face or\r
  60 something. I'm guessing that just that much would cover most people's\r
  61 needs, before needing to choose custom colors for each tag name.\r
  62 \r
  63 > +(defface notmuch-show-subject-face\r
  64 > + '((((class color) (background light)) (:foreground "yellow" :bold t))\r
  65 > +    (((class color) (background dark)) (:foreground "yellow" :bold t)))\r
  66 > +  "Notmuch show mode face used to highligh subject line."\r
  67 > +  :group 'notmuch)\r
  68 \r
  69 Color selection can be pretty difficult, and I don't think we're ever\r
  70 all going to agree on things. But I can at least say that I can't read\r
  71 yellow on white at all, (maybe yellow on black works out a bit better, I\r
  72 don't know).\r
  73 \r
  74 So I'd definitely like to hear people's ideas on what the presentation\r
  75 should look like here.\r
  76 \r
  77 First, the current reverse-video is definitely not something I really\r
  78 like. That was just a placeholder since I didn't know how to configure\r
  79 other faces, (and I needed _something_ to distinguish one message from\r
  80 another).\r
  81 \r
  82 Second, from the time I spent with sup, I found that it was hard for me\r
  83 to read entire lines of bold text. So I think I'd rather just have the\r
  84 header name ("Subject:", for example) in bold and the rest of the\r
  85 subject not bold. Hmm... maybe we could use a "From:" at the beginning\r
  86 of the one line summary.\r
  87 \r
  88 Finally, for separating one message from the other, I think I'd really\r
  89 just like a one-pixel line drawn between the last line of text of one\r
  90 message and the first line of text of the next message. Does anybody\r
  91 know how to achieve that?\r
  92 \r
  93 Back to the patch, I think I'll be able to push it as soon as I can\r
  94 apply it and still _see_ the subject lines. ;-)\r
  95 \r
  96 -Carl\r