59/19c161a81358d47f81e94be0da7ed541ce6781

   1 Return-Path: <jani@nikula.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 55F84431FB6\r
   6         for <notmuch@notmuchmail.org>; Fri, 13 Apr 2012 01:57:22 -0700 (PDT)\r
   7 X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
   8 X-Spam-Flag: NO\r
   9 X-Spam-Score: -0.7\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-0.7 tagged_above=-999 required=5\r
  12         tests=[RCVD_IN_DNSWL_LOW=-0.7] autolearn=disabled\r
  13 Received: from olra.theworths.org ([127.0.0.1])\r
  14         by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
  15         with ESMTP id kdt5FEiEyCpu for <notmuch@notmuchmail.org>;\r
  16         Fri, 13 Apr 2012 01:57:21 -0700 (PDT)\r
  17 Received: from mail-qc0-f181.google.com (mail-qc0-f181.google.com\r
  18         [209.85.216.181]) (using TLSv1 with cipher RC4-SHA (128/128 bits))\r
  19         (No client certificate requested)\r
  20         by olra.theworths.org (Postfix) with ESMTPS id 61099431FAF\r
  21         for <notmuch@notmuchmail.org>; Fri, 13 Apr 2012 01:57:21 -0700 (PDT)\r
  22 Received: by qcsk26 with SMTP id k26so2175663qcs.26\r
  23         for <notmuch@notmuchmail.org>; Fri, 13 Apr 2012 01:57:19 -0700 (PDT)\r
  24 X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;\r
  25         d=google.com; s=20120113;\r
  26         h=from:to:subject:in-reply-to:references:user-agent:date:message-id\r
  27         :mime-version:content-type:x-gm-message-state;\r
  28         bh=EPCqCdRwv6ONOLnxfrOlgQ/S8pln126MvQWsBjWvc9Q=;\r
  29         b=mkTzfqFtMF2WCC5ceiNiO8Brd4uSORocmsMlMaNIG7jGKmTtwlXLIuc8PhDCvANxUb\r
  30         Gf2nOxQwn1obuDc/IJly+SIH7JjRjqJ73m+JDs3nqEhgfppfPtMhi+HwbbUwIBBcKdob\r
  31         G6+AhFcosqAsD0HoItdbhln03y4SRE+DO1xvOcwTHtDFMrPsGIP2E+YHlJP6mQKBTc6D\r
  32         jbM6OsAjeJ93T1qVgxG9ZolFRcihmEDRby2CAzMFE6QYM03J1uaqtZiK7LjQIL/f8s+/\r
  33         iGSXWbEFum68ikDpmbT977CMykjnK0SynK7gtI2xf+OexbR19SpGkpihZS3ahypUNKpD\r
  34         Aapw==\r
  35 Received: by 10.224.173.194 with SMTP id q2mr1417250qaz.15.1334307439702;\r
  36         Fri, 13 Apr 2012 01:57:19 -0700 (PDT)\r
  37 Received: from localhost ([92.243.24.172])\r
  38         by mx.google.com with ESMTPS id dx8sm7609556qab.5.2012.04.13.01.57.17\r
  39         (version=SSLv3 cipher=OTHER); Fri, 13 Apr 2012 01:57:18 -0700 (PDT)\r
  40 From: Jani Nikula <jani@nikula.org>\r
  41 To: Tomi Ollila <tomi.ollila@iki.fi>,\r
  42         Jameson Graef Rollins <jrollins@finestructure.net>,\r
  43         Mark Walters <markwalters1009@gmail.com>, notmuch@notmuchmail.org\r
  44 Subject: Re: [PATCH] emacs-show: open excluded matches if no other matches\r
  45 In-Reply-To: <m2ty0odaej.fsf@guru.guru-group.fi>\r
  46 References: <1331836090-30560-1-git-send-email-markwalters1009@gmail.com>\r
  47         <8762d44vkv.fsf@servo.finestructure.net>\r
  48         <m2ty0odaej.fsf@guru.guru-group.fi>\r
  49 User-Agent: Notmuch/0.11.1+222~ga47a98c (http://notmuchmail.org) Emacs/23.1.1\r
  50         (i686-pc-linux-gnu)\r
  51 Date: Fri, 13 Apr 2012 08:57:13 +0000\r
  52 Message-ID: <87k41kj9g6.fsf@nikula.org>\r
  53 MIME-Version: 1.0\r
  54 Content-Type: text/plain; charset=us-ascii\r
  55 X-Gm-Message-State:\r
  56  ALoCoQlVg+oxqlKs58VOkXs1uSll2xlmqNuQ8QW0dqluGDElhagBnVb/mhIYGnGq70N00rThP+yT\r
  57 X-BeenThere: notmuch@notmuchmail.org\r
  58 X-Mailman-Version: 2.1.13\r
  59 Precedence: list\r
  60 List-Id: "Use and development of the notmuch mail system."\r
  61         <notmuch.notmuchmail.org>\r
  62 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  63         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  64 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  65 List-Post: <mailto:notmuch@notmuchmail.org>\r
  66 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  67 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  68         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  69 X-List-Received-Date: Fri, 13 Apr 2012 08:57:22 -0000\r
  70 \r
  71 On Thu, 12 Apr 2012 22:20:52 +0300, Tomi Ollila <tomi.ollila@iki.fi> wrote:\r
  72 > On Thu, Apr 12 2012, Jameson Graef Rollins wrote:\r
  73 > \r
  74 > > On Thu, Mar 15 2012, Mark Walters <markwalters1009@gmail.com> wrote:\r
  75 > >> Currently emacs show does not open matching but excluded\r
  76 > >> messages. This is normally the desired behaviour but is probably not\r
  77 > >> ideal if only excluded messages match. This patch opens all the\r
  78 > >> matching (necessarily excluded) messages in this case and goes to the\r
  79 > >> first one.\r
  80 > >\r
  81 > > Hi, Mark.  I have tested this patch and it LGTM.  I think this is the\r
  82 > > behavior we want, since it's no fun to open up a show buffer and have\r
  83 > > all messages in the thread closed.  That doesn't make much sense.\r
  84 > \r
  85 > I disagree: If I open up a show buffer I'd like to have all messages\r
  86 > in the thread closed if search terms do not match. But that is my\r
  87 > opinion and inconsistent with current behaviour.\r
  88 > \r
  89 > (Therefore I already +1:d this patch)\r
  90 > \r
  91 > An example: I have thread of 100 messages and 1 of those is unread.\r
  92 > Search terms is tag:unread. I enter the thread: 99 messages closed\r
  93 > and this one open. Now unread is automatically removed. If I go back\r
  94 > to search view (by pressing q) and re-choose the same thread,\r
  95 > current behaviour is to open all 100 messages. I'd like to see all\r
  96 > 100 messages closed.\r
  97 \r
  98 IIUC this would still show all messages closed. Messages that don't\r
  99 match will be closed. The change is that if there are messages that\r
 100 match, but all of them are excluded, then open the messages anyway.\r
 101 \r
 102 In your example, only if you have *another* message that is both\r
 103 tag:unread and tag:deleted (or some other excluded tag) it would show up\r
 104 the second time you enter show view.\r
 105 \r
 106 I actually think I'd still want the above to show all messages closed,\r
 107 unless the excluded tag was explicitly specified in the search. The idea\r
 108 of exclusions is to reduce "noise" in the view, why open them? If you\r
 109 have those 100 messages in a thread, and you want to see the interesting\r
 110 (i.e. non-excluded) messages, I'd rather see 0 open if 0 are\r
 111 interesting, instead of 100 non-interesting if they are all excluded.\r
 112 \r
 113 Perhaps a function to toggle the open/close of excluded messages would\r
 114 be a better idea. But then I don't use exclusions ATM, so maybe I just\r
 115 don't understand, and won't be affected either way. *shrug*.\r
 116 \r
 117 \r
 118 BR,\r
 119 Jani.\r