a7/4150a998c74085accf34754416e9c4fadb3a06

   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 95760431FAF\r
   6         for <notmuch@notmuchmail.org>; Wed,  4 Dec 2013 01:01:44 -0800 (PST)\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 FCSty5ggHqUt for <notmuch@notmuchmail.org>;\r
  16         Wed,  4 Dec 2013 01:01:39 -0800 (PST)\r
  17 Received: from mail-qc0-f178.google.com (mail-qc0-f178.google.com\r
  18         [209.85.216.178]) (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 C5F9E431FAE\r
  21         for <notmuch@notmuchmail.org>; Wed,  4 Dec 2013 01:01:39 -0800 (PST)\r
  22 Received: by mail-qc0-f178.google.com with SMTP id i17so3268342qcy.9\r
  23         for <notmuch@notmuchmail.org>; Wed, 04 Dec 2013 01:01:38 -0800 (PST)\r
  24 X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;\r
  25         d=1e100.net; s=20130820;\r
  26         h=x-gm-message-state:from:to:subject:in-reply-to:references\r
  27         :user-agent:date:message-id:mime-version:content-type;\r
  28         bh=CpAhOFE6pk8oVkUU2EoeHeschw9EP0Pt+vGdA7dmZ60=;\r
  29         b=G/cbwPnX8tEV0QjAhhNNj8o8r9p5WCt/0fdluQ36FjlL3wk/rPP1biuRrerydBJbWR\r
  30         Jfg4Z8uSABKMnjg6lHAxwysqpMz5VufH0g/FfHmQbz4tr5joCofNJBAA5LSoDftNNtRf\r
  31         c1ijefOBlgLdu5a22aPgORDvLXgZC+SLGczItNe8zIfZQPa8zCmDMFUEf+z07GmjSFK2\r
  32         a0CGOnhsZ1y34onmvOfTwO7gYYFYcj9XKjOD9gtIBK/qBYrOG4MGfvp7q4n83kPctyEr\r
  33         HiWTU2QGi7/Yxy+m01f0tqAcmeYLAVU6x6OWxY/EUTwDliKq02hzjEujqP8H5Ha4L6XD\r
  34         Ozrw==\r
  35 X-Gm-Message-State:\r
  36  ALoCoQl9WbyB+tzXhBYz5snfTGpSkGwezFQqzlcE2tyJ2oM2dWuLhXifYlFgheP45aqDOYJkJlGt\r
  37 X-Received: by 10.224.127.74 with SMTP id f10mr98428359qas.56.1386147698116;\r
  38         Wed, 04 Dec 2013 01:01:38 -0800 (PST)\r
  39 Received: from localhost ([2001:4b98:dc0:43:216:3eff:fe1b:25f3])\r
  40         by mx.google.com with ESMTPSA id hb2sm15867812qeb.6.2013.12.04.01.01.37\r
  41         for <multiple recipients>\r
  42         (version=TLSv1.1 cipher=RC4-SHA bits=128/128);\r
  43         Wed, 04 Dec 2013 01:01:37 -0800 (PST)\r
  44 From: Jani Nikula <jani@nikula.org>\r
  45 To: Mark Walters <markwalters1009@gmail.com>, notmuch@notmuchmail.org\r
  46 Subject: Re: [PATCH WIP v2 0/5] emacs: show: redesign unread/read logic\r
  47 In-Reply-To: <1385892147-16994-1-git-send-email-markwalters1009@gmail.com>\r
  48 References: <1385892147-16994-1-git-send-email-markwalters1009@gmail.com>\r
  49 User-Agent: Notmuch/0.17~rc2+4~gd7b0a0a (http://notmuchmail.org) Emacs/23.2.1\r
  50         (x86_64-pc-linux-gnu)\r
  51 Date: Wed, 04 Dec 2013 10:01:19 +0100\r
  52 Message-ID: <8761r5at28.fsf@nikula.org>\r
  53 MIME-Version: 1.0\r
  54 Content-Type: text/plain; charset=us-ascii\r
  55 X-BeenThere: notmuch@notmuchmail.org\r
  56 X-Mailman-Version: 2.1.13\r
  57 Precedence: list\r
  58 List-Id: "Use and development of the notmuch mail system."\r
  59         <notmuch.notmuchmail.org>\r
  60 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  61         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  62 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  63 List-Post: <mailto:notmuch@notmuchmail.org>\r
  64 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  65 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  66         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  67 X-List-Received-Date: Wed, 04 Dec 2013 09:01:44 -0000\r
  68 \r
  69 On Sun, 01 Dec 2013, Mark Walters <markwalters1009@gmail.com> wrote:\r
  70 > This is further wip of the message at\r
  71 > id:1385285551-5158-1-git-send-email-markwalters1009@gmail.com\r
  72 > see there for some discussion of the design.\r
  73 >\r
  74 > This series is still definitely wip: one reason for posting is so\r
  75 > people can play with different strategies for marking read\r
  76 > easily. (As WIP tree's unread handling is broken and the tests need updating.)\r
  77 >\r
  78 > The series consists of three parts: the first 4 patches add the notion\r
  79 > of seen: this means the user has seen the message but the message has\r
  80 > typically not been marked read yet. The seen messages are marked read\r
  81 > when the user quits the show buffer unless the user quits with\r
  82 > prefix-arg quit. In all cases an informative message is shown.\r
  83 >\r
  84 > The fifth patch adds a psot-command-hook stub for updating the seen\r
  85 > status. This seems a natural place to do the update as it means\r
  86 > however the user navugates around the buffer (eg next-message or\r
  87 > page-down etc) the update gets done.\r
  88 >\r
  89 > This is intended to be an easy place for other people to try out their\r
  90 > own mark read strategies.\r
  91 >\r
  92 > The final patch implements something pretty close to what I would like\r
  93 > for marking seen/read. A message is deemed seen provided the user has\r
  94 > seen the top of the message, and has seen either the bottom of the\r
  95 > message or a point at least some customisable number of lines into the\r
  96 > message. The customisable number of lines can either be a fixed number\r
  97 > e.g. 20, or a number depending on the height of the current window\r
  98 > e.g. the default is 3/4 of the window height.\r
  99 >\r
 100 > The idea is a message seen if the user has seen the entire message, or\r
 101 > enough of it they have to have noticed it. The figure of 3/4 also\r
 102 > means that the notmuch commands like next-message which place the top\r
 103 > of the message at the top of the window automatically mark the message\r
 104 > seen as either the whole message or at least one window full must be\r
 105 > visible.\r
 106 >\r
 107 > I would be very grateful for any comments on whether this behaves as\r
 108 > people would expect, what they would want instead etc\r
 109 \r
 110 Hi Mark, thanks for working on this.\r
 111 \r
 112 I had tons of mail reading to catch up, so this was a good opportunity\r
 113 to try the patches. I'll try to be objective and constructive next, but\r
 114 up front, just so there's no doubt: I don't like it.\r
 115 \r
 116 I think my issues boil down to the series containing two pretty\r
 117 significant changes at once: how to decide if a message was read and\r
 118 when to apply the tag changes to reflect that.\r
 119 \r
 120 I found it confusing that messages were not being tagged -unread while I\r
 121 was viewing the thread. I found it even more confusing to get a message\r
 122 "Marked N messages read" on quitting show view with no feedback on\r
 123 *which* messages were read, and often the N didn't feel right\r
 124 either. And I think that's the problem: I wanted to see the new\r
 125 heuristics on deciding whether a message was read in action, but I got\r
 126 zero immediate feedback on it!\r
 127 \r
 128 My suggestion is to drop the delay in tag changes for now, and focus on\r
 129 the part that decides whether a message was read or not. Do the tag\r
 130 changes immediately when you consider a message "seen". I think this way\r
 131 we get a better feel of how well the heuristics really work, and we can\r
 132 make it just right. I think that's the bug in we currently have in\r
 133 notmuch, and delaying the tag changes doesn't contribute to fixing\r
 134 it. Indeed I think the delay makes it *harder* to fix.\r
 135 \r
 136 Afterwards, we could add the delayed tag changes (although hopefully as\r
 137 an option) if desired. And keeping that in mind, AFAICT you wouldn't\r
 138 need to rework your patches all that much.\r
 139 \r
 140 How does that sound?\r
 141 \r
 142 \r
 143 BR,\r
 144 Jani.\r