[PATCH 00/11] contrib: pick: keybindings
Mark Walters
markwalters1009 at gmail.com
Fri Jul 5 11:11:07 PDT 2013
This series adds lots of keybindings and funtionality to pick: in
particular it adds the stash keymap, the ability to tab between and
activate buttons in the message pane, and it reduces a lot of code
duplication between pick and show.
It is a large series: but most of it is a lot of small changes. These
small changes are mostly logically independent but as they add
keybindings their contexts all clash. I have made most of the
keybindings as single separate patches to make discussion of them
individually easier.
The key patches for review/discussion (apart from bike-shedding on
key-bindings!) are patches 1, 5 and 8.
Patch 1 is the most "controversial": it over-rides
notmuch-show-get-prop so that whether it uses
notmuch-show-get-message-properties or
notmuch-pick-get-message-properties depends on the major-mode (ie
whether it is called from pick or show).
This means that functions from show which just use message properties
(most often just the message id) "just work" when called from pick. In
particular it gives us access to lots of functions without having to
duplicate the code.
In the longer term it would be better to have some show/pick common
file and migrate the common functions there.
Patch 5 and 8 add in functions for creating fucntions ready to be used
in keybindings. The one in patch 5 takes a show fucntion and creates a
function which switches from pick to the message pane, applies the
function and then switches back to pick. The one in patch 8 takes a
function and creates a function which closes the message pane and the
calls this function.
Both of these make the keybinding section clearer. They also have the
advantage that the user can use them easily to create their own
keybindings which do this.
This completes all the keybindings I use and I think means that pick
doesn't have any glaring omissions.
Finally, this will clash with the thread archive patches
id:1371195472-441-1-git-send-email-markwalters1009 at gmail.com
Best wishes
Mark
Mark Walters (11):
contrib: pick: override notmuch-show-get-prop
contrib: pick: Link in notmuch-show-pipe-message
contrib: pick: Link in attachment functions straight from
notmuch-show
contrib: pick: Link in stash map straight from notmuch-show
contrib: pick: add in to-message-window function
contrib: pick: add button press helper
contrib: pick: pass tab through to the message pane
contrib: pick: close window function
contrib: pick: make help close the message pane first
contrib: pick: add in binding to view raw message
contrib: pick: use close-message-pane for reply etc
contrib/notmuch-pick/notmuch-pick.el | 139 +++++++++++++++++-----------------
1 files changed, 70 insertions(+), 69 deletions(-)
--
1.7.9.1
More information about the notmuch
mailing list