Re: [PATCH] emacs: add default value to notmuch-search-line-faces
authorJani Nikula <jani@nikula.org>
Thu, 26 Jan 2012 21:58:51 +0000 (23:58 +0200)
committerW. Trevor King <wking@tremily.us>
Fri, 7 Nov 2014 17:43:23 +0000 (09:43 -0800)
2b/5714baef3aeadbe81584a89374c96ff1001e56 [new file with mode: 0644]

diff --git a/2b/5714baef3aeadbe81584a89374c96ff1001e56 b/2b/5714baef3aeadbe81584a89374c96ff1001e56
new file mode 100644 (file)
index 0000000..11dc106
--- /dev/null
@@ -0,0 +1,123 @@
+Return-Path: <jani@nikula.org>\r
+X-Original-To: notmuch@notmuchmail.org\r
+Delivered-To: notmuch@notmuchmail.org\r
+Received: from localhost (localhost [127.0.0.1])\r
+       by olra.theworths.org (Postfix) with ESMTP id 8F264431FAF\r
+       for <notmuch@notmuchmail.org>; Thu, 26 Jan 2012 13:58:57 -0800 (PST)\r
+X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
+X-Spam-Flag: NO\r
+X-Spam-Score: -0.7\r
+X-Spam-Level: \r
+X-Spam-Status: No, score=-0.7 tagged_above=-999 required=5\r
+       tests=[RCVD_IN_DNSWL_LOW=-0.7] autolearn=disabled\r
+Received: from olra.theworths.org ([127.0.0.1])\r
+       by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
+       with ESMTP id gtd-MBqyC1Vy for <notmuch@notmuchmail.org>;\r
+       Thu, 26 Jan 2012 13:58:57 -0800 (PST)\r
+Received: from mail-lpp01m010-f53.google.com (mail-lpp01m010-f53.google.com\r
+       [209.85.215.53]) (using TLSv1 with cipher RC4-SHA (128/128 bits))\r
+       (No client certificate requested)\r
+       by olra.theworths.org (Postfix) with ESMTPS id 9FED2431FAE\r
+       for <notmuch@notmuchmail.org>; Thu, 26 Jan 2012 13:58:56 -0800 (PST)\r
+Received: by lahd3 with SMTP id d3so672226lah.26\r
+       for <notmuch@notmuchmail.org>; Thu, 26 Jan 2012 13:58:55 -0800 (PST)\r
+Received: by 10.112.100.2 with SMTP id eu2mr1094964lbb.97.1327615135010;\r
+       Thu, 26 Jan 2012 13:58:55 -0800 (PST)\r
+Received: from localhost (dsl-hkibrasgw4-fe50f800-253.dhcp.inet.fi.\r
+       [84.248.80.253])\r
+       by mx.google.com with ESMTPS id nu4sm4160892lab.4.2012.01.26.13.58.52\r
+       (version=SSLv3 cipher=OTHER); Thu, 26 Jan 2012 13:58:53 -0800 (PST)\r
+From: Jani Nikula <jani@nikula.org>\r
+To: Austin Clements <amdragon@MIT.EDU>\r
+Subject: Re: [PATCH] emacs: add default value to notmuch-search-line-faces\r
+In-Reply-To: <20120126194146.GG1940@mit.edu>\r
+References: <1327605679-15213-1-git-send-email-jani@nikula.org>\r
+       <20120126194146.GG1940@mit.edu>\r
+User-Agent: Notmuch/0.11+95~g1c29b96 (http://notmuchmail.org) Emacs/23.3.1\r
+       (i686-pc-linux-gnu)\r
+Date: Thu, 26 Jan 2012 23:58:51 +0200\r
+Message-ID: <87liounn1g.fsf@nikula.org>\r
+MIME-Version: 1.0\r
+Content-Type: text/plain; charset=us-ascii\r
+Cc: notmuch@notmuchmail.org\r
+X-BeenThere: notmuch@notmuchmail.org\r
+X-Mailman-Version: 2.1.13\r
+Precedence: list\r
+List-Id: "Use and development of the notmuch mail system."\r
+       <notmuch.notmuchmail.org>\r
+List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
+       <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
+List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
+List-Post: <mailto:notmuch@notmuchmail.org>\r
+List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
+List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
+       <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
+X-List-Received-Date: Thu, 26 Jan 2012 21:58:57 -0000\r
+\r
+On Thu, 26 Jan 2012 14:41:46 -0500, Austin Clements <amdragon@MIT.EDU> wrote:\r
+> Quoth Jani Nikula on Jan 26 at  9:21 pm:\r
+> > Add default value to notmuch-search-line-faces to show "unread"\r
+> > messages in bold, and "flagged" messages in red, to have some visual\r
+> > indication of important messages in search results. This should be\r
+> > helpful for new users.\r
+> > \r
+> > "unread" tag is quite obvious, and handled specially both in the lib\r
+> > and emacs ui. "flagged" is synced to maildir F flag in the lib. If one\r
+> > syncs the maildir to IMAP, this also translates to corresponding IMAP\r
+> > flag. (This is "starred" in GMail and Android.)\r
+> > \r
+> > Signed-off-by: Jani Nikula <jani@nikula.org>\r
+> \r
+> While I'm sure this can be bikeshod to death, I do have one legitimate\r
+> concern (and one illegitimate).\r
+> \r
+> I completely agree with what you said on IRC, though, that setting\r
+> this to non-nil is more about making this capability more discoverable\r
+> to new users than trying to come up with the perfect faces.\r
+\r
+Thanks for repeating that here too before full blown bikeshedding. I'm\r
+just trying to help other people find this option quicker than I and\r
+some others did...\r
+\r
+> > ---\r
+> >  emacs/notmuch.el |    3 ++-\r
+> >  1 files changed, 2 insertions(+), 1 deletions(-)\r
+> > \r
+> > diff --git a/emacs/notmuch.el b/emacs/notmuch.el\r
+> > index 6b2c252..551ea9d 100644\r
+> > --- a/emacs/notmuch.el\r
+> > +++ b/emacs/notmuch.el\r
+> > @@ -662,7 +662,8 @@ This function advances the next thread when finished."\r
+> >              (goto-char (point-min))\r
+> >              (forward-line (1- notmuch-search-target-line))))))))\r
+> >  \r
+> > -(defcustom notmuch-search-line-faces nil\r
+> > +(defcustom notmuch-search-line-faces '(("unread" :weight bold)\r
+> \r
+> As much as I would like this, many terminals don't visually\r
+> distinguish between the default face and the default face in bold.\r
+\r
+Would it really matter, though? People on such terminals are probably\r
+aware they are missing out on a bunch of visual stuff. But please do\r
+suggest an alternative that could be "composited" with the :foreground\r
+setting below for "unread" and "flagged" threads.\r
+\r
+> > +                                 ("flagged" :foreground "red"))\r
+> \r
+> Red is pretty universally used to indicate danger or a serious\r
+> condition, while "flagged" is simply supposed to draw attention.  I\r
+> would say blue as a neutral and distinct indicator, but it also has\r
+> poor visibility (I used to use blue, but found that when scanning my\r
+> mail, I would habitually skip over flagged messages because they were\r
+> dark, which was the opposite of what I wanted).  Personally I've\r
+> settled on yellow; it's visually distinct enough to be easily\r
+> noticeable and bright enough that I don't skip over it, though it\r
+> obviously wouldn't work on a light background.\r
+\r
+Can be yellow, I don't really care that much. But isn't a light\r
+background the default for emacs in most distros? If that matters,\r
+*shrug*.\r
+\r
+\r
+BR,\r
+Jani.\r