On Thu, 05 Dec 2013, Mark Walters <markwalters1009@gmail.com> wrote: > This is the next iteration of this WIP patch series. v2 was at > id:1385892147-16994-1-git-send-email-markwalters1009@gmail.com > > I would just like to emphasise that *all* feedback is appreciated > including "I HATE IT" etc. I LIKE IT. :) It's not perfect, but IMHO it's a nice, incremental improvement over what we have, and it's discoverable: you can see what it does and when. If anything is surprising, the user has a better chance of explaining what happened, and what he liked and didn't. Good work, Mark! BR, Jani. PS. I didn't look at the code, I only tried it. > I have followed Jani's suggestion and removed the delayed updates to > unread status: it makes it harder to see if the heuristics are doing > what you want. > > The first patch is completely new: it makes all tag changes in a > buffer (in particular the unread tag) clearly visible to the user: a > deleted tag is displayed with red strike-through and an added tag is > displayed underlined in green. (This may have strange interactions with > notmuch-tag-formats in some cases.) > > This makes the visual feedback when a message is marked unread very > clear: the unread tag is struck-through. > > As always with the previous series lots of tests will fail for the > obvious reason that marking read is done differently and displayed > differently. > > Best wishes > > Mark > > > > > Mark Walters (3): > emacs: show: mark tags changed since buffer loaded > emacs: show: add an update seen function to post-command-hook > emacs: show: make `seen' mean user viewed whole message > > emacs/notmuch-show.el | 136 +++++++++++++++++++++++++++++++++++++++++------- > emacs/notmuch-tag.el | 30 +++++++---- > 2 files changed, 136 insertions(+), 30 deletions(-) > > -- > 1.7.9.1 > > _______________________________________________ > notmuch mailing list > notmuch@notmuchmail.org > http://notmuchmail.org/mailman/listinfo/notmuch