notmuch.el: Override next-line and previous-line to make them reliable.
authorCarl Worth <cworth@cworth.org>
Wed, 4 Nov 2009 21:39:26 +0000 (13:39 -0800)
committerCarl Worth <cworth@cworth.org>
Wed, 4 Nov 2009 21:39:26 +0000 (13:39 -0800)
commiteacd1ac41eb6f50bdae8121606859a3a673e7e7b
tree2641d18299e3dbb782d3d6624b2c195c7dbb3682
parent1d68e75c21e8e7873277c15bfa896cb3b9f726e9
notmuch.el: Override next-line and previous-line to make them reliable.

I noticed that these functions would sometimes leave point on an
invisible character[*]. The problem would be that point would appear
to be on a particular message, but adding or removing a tag would
actually add/remove a tag from the *previous* message.

Fix the C-n and C-p keybindings at least to call the underlying
command and then advance to a visible character. We set this-command
in our overrides so that the temporary-goal-column feature still
works.

[*] The documentation says that command loop is supposed to move point
outside of any invisible region when a command exits. But apparently
not.
notmuch.el