Carl Worth
2009-Aug-26 22:40 UTC
[sup-talk] [PATCH] Add ''a'' and ''d'' keybindings to thread-view-mode to archive/delete current thread
These behave identically to the existing ",a" and ",d"
commands, (that
is they archive or delete the current thread and then view the next).
---
I appreciate that within thread-view-mode, sup already supports
explicitly selecting one of three different actions for what do after
the user is done with a thread. And the choice is determined by one of
three different prefix characters. For example:
. -> return to previous buffer (likely something like
inbox-mode/search-result-mode)
, -> advance to next thread (into thread-view-mode)
] -> advance to previous thread (into thread-view-mode)
I''m lazy enough to want a single-character command for efficient
processing of mail in thread-view-mode, (just like I already have
while in inbox-mode and search-results-mode). And for two of the
different actions, (archive and delete), the obvious/consistent
keybinding is available. So this patch assigns those, (''a'' as
a
shortcut for '',a'' and ''d'' as a shortcut for
'',d'').
If I seem too lazy here, and it doesn''t seem sane to add these
keybindingd by default, just let me know. In that case I''ll consign
myself to just using a hook to setup these keybindings for myself
personally.
The perhaps-not-as-obvious piece is what the exit action should be for
these shortcut commands. For my own use, I''ve found it best to advance
to the next thread. (This supports a workflow where there''s one pass
through the inbox to archive as much as possible without reading, and
then another pass through thread-view-mode for everything that''s
left.)
One thing this patch doesn''t provide is shortcuts for the other three
exit actions, ('',s'' for mark-as-spam-then-next,
'',N'' for
mark-as-unread-then-next, and '',n'' for simply advancing to the
next
thread without changing the current thread).
Of those, the only one I really care about personally is '',n''.
It''s
obviously not a simple matter of just making ''n'' a shortcut
for '',n''
since the ''n'' keybinding is already used. But actually, I
think I
would like the ''n'' keybinding to advance to the next thread
once I''m
viewing the last open message in the current thread. This would make
for very efficient sessions of just reading mail, where I''d only need
to keep hitting ''n'', (and I could hit ''a''
whenever bored with any
particular thread). So perhaps I''ll look into coding that up next.
lib/sup/modes/thread-view-mode.rb | 3 +++
1 files changed, 3 insertions(+), 0 deletions(-)
diff --git a/lib/sup/modes/thread-view-mode.rb
b/lib/sup/modes/thread-view-mode.rb
index 678b841..0706757 100644
--- a/lib/sup/modes/thread-view-mode.rb
+++ b/lib/sup/modes/thread-view-mode.rb
@@ -64,6 +64,9 @@ EOS
k.add :unsubscribe_from_list, "Subscribe to/unsubscribe from mailing
list", ")"
k.add :pipe_message, "Pipe message or attachment to a shell
command", ''|''
+ k.add :archive_and_next, "Archive this thread, kill buffer, and view
next", ''a''
+ k.add :delete_and_next, "Delete this thread, kill buffer, and view
next", ''d''
+
k.add_multi "(a)rchive/(d)elete/mark as (s)pam/mark as
u(N)read:", ''.'' do |kk|
kk.add :archive_and_kill, "Archive this thread and kill
buffer", ''a''
kk.add :delete_and_kill, "Delete this thread and kill buffer",
''d''
--
1.6.3.3
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL:
<http://rubyforge.org/pipermail/sup-talk/attachments/20090826/ac261bb3/attachment.bin>
William Morgan
2009-Sep-03 18:06 UTC
[sup-talk] [PATCH] Add ''a'' and ''d'' keybindings to thread-view-mode to archive/delete current thread
Reformatted excerpts from Carl Worth''s message of 2009-08-26:> These behave identically to the existing ",a" and ",d" commands, (that > is they archive or delete the current thread and then view the next).Sorry it''s taken me so long to look at this. Thanks for the patch! I''m curious what other people think about this. On the one hand, it''s only additional keybindings, so it''s not going to adversely affect anyone. On the other hand, there is a nice balance with the ".", "," and "]" commands which this disrupts, and it can''t be extended to the other commands from thread-index-mode. And on the third hand, I''m happy to have keybinding hooks. I guess if most people primarily closes their thread-view-modes using ",a" and ",d", then I''m fine with this. -- William <wmorgan-sup at masanjin.net>
Mark Alexander
2009-Sep-03 18:37 UTC
[sup-talk] [PATCH] Add ''a'' and ''d'' keybindings to thread-view-mode to archive/delete current thread
Excerpts from William Morgan''s message of Thu Sep 03 14:06:55 -0400 2009:> I guess if most people primarily closes their thread-view-modes using > ",a" and ",d", then I''m fine with this.Those are perhaps my most-commonly used commands in sup, so having them shortened is a very nice improvement.
Nicholas Bergson-Shilcock
2009-Sep-03 18:58 UTC
[sup-talk] [PATCH] Add ''a'' and ''d'' keybindings to thread-view-mode to archive/delete current thread
Excerpts from Mark Alexander''s message of Thu Sep 03 14:37:34 -0400 2009:> Excerpts from William Morgan''s message of Thu Sep 03 14:06:55 -0400 2009: > > I guess if most people primarily closes their thread-view-modes using > > ",a" and ",d", then I''m fine with this. > > Those are perhaps my most-commonly used commands in sup, so having > them shortened is a very nice improvement.+1. These would be quite helpful for me.
Nicolas Pouillard
2009-Sep-06 19:43 UTC
[sup-talk] [PATCH] Add ''a'' and ''d'' keybindings to thread-view-mode to archive/delete current thread
Excerpts from William Morgan''s message of Thu Sep 03 20:06:55 +0200 2009:> Reformatted excerpts from Carl Worth''s message of 2009-08-26: > > These behave identically to the existing ",a" and ",d" commands, (that > > is they archive or delete the current thread and then view the next). > > Sorry it''s taken me so long to look at this. Thanks for the patch! > > I''m curious what other people think about this. On the one hand, it''s > only additional keybindings, so it''s not going to adversely affect > anyone. On the other hand, there is a nice balance with the ".", "," > and "]" commands which this disrupts, and it can''t be extended to the > other commands from thread-index-mode. And on the third hand, I''m happy > to have keybinding hooks. > > I guess if most people primarily closes their thread-view-modes using > ",a" and ",d", then I''m fine with this.I personally do prefer ]a and ]d, to read mail in the chronological order. -- Nicolas Pouillard http://nicolaspouillard.fr
Wirt Wolff
2009-Sep-08 15:17 UTC
[sup-talk] [PATCH] Add ''a'' and ''d'' keybindings to thread-view-mode to archive/delete current thread
Excerpts from William Morgan''s message of Thu Sep 03 12:06:55 -0600 2009:> Reformatted excerpts from Carl Worth''s message of 2009-08-26: > > These behave identically to the existing ",a" and ",d" commands, (that > > is they archive or delete the current thread and then view the next). > > Sorry it''s taken me so long to look at this. Thanks for the patch! > > I''m curious what other people think about this. On the one hand, it''s > only additional keybindings, so it''s not going to adversely affect > anyone. On the other hand, there is a nice balance with the ".", "," > and "]" commands which this disrupts, and it can''t be extended to the > other commands from thread-index-mode. And on the third hand, I''m happy > to have keybinding hooks. > > I guess if most people primarily closes their thread-view-modes using > ",a" and ",d", then I''m fine with this.Don''t mind having ''a'' and ''d'' around, but likely won''t use them. Generally I pass down index with &, A, S, etc. occaisionally reading high priority thread, then ]n, ]a etc. back up in thread view. -- wmw
William Morgan
2009-Sep-08 20:08 UTC
[sup-talk] [PATCH] Add ''a'' and ''d'' keybindings to thread-view-mode to archive/delete current thread
Reformatted excerpts from Wirt Wolff''s message of 2009-09-08:> Don''t mind having ''a'' and ''d'' around, but likely won''t use them.So far I''m counting 3 yes votes (including patch submitter), 3 "won''t use them" votes (including me), and no one being offended. I think I''m going to apply this. -- William <wmorgan-sup at masanjin.net>
Carl Worth
2009-Sep-08 20:26 UTC
[sup-talk] [PATCH] Add ''a'' and ''d'' keybindings to thread-view-mode to archive/delete current thread
Excerpts from Nicolas Pouillard''s message of Sun Sep 06 12:43:06 -0700 2009:> I personally do prefer ]a and ]d, to read mail in the chronological order.What if there were a way to reverse the sort order for the index view? That would then change the meaning of "next" and "previous" and then you could use the single letter ''a'' and ''d'' commands to read your mail in chronological order. Would that make sense? Because that is something I would like to do on occasion. -Carl -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 189 bytes Desc: not available URL: <http://rubyforge.org/pipermail/sup-talk/attachments/20090908/01fd44fe/attachment.bin>
Nicolas Pouillard
2009-Sep-08 20:33 UTC
[sup-talk] [PATCH] Add ''a'' and ''d'' keybindings to thread-view-mode to archive/delete current thread
Excerpts from Carl Worth''s message of Tue Sep 08 22:26:45 +0200 2009:> Excerpts from Nicolas Pouillard''s message of Sun Sep 06 12:43:06 -0700 2009: > > I personally do prefer ]a and ]d, to read mail in the chronological order. > > What if there were a way to reverse the sort order for the index view? > That would then change the meaning of "next" and "previous" and then > you could use the single letter ''a'' and ''d'' commands to read your mail > in chronological order. > > Would that make sense? Because that is something I would like to do on > occasion.Yes it does make sense. This change would both improve this feature and will reduce the default enforcement. Regards, -- Nicolas Pouillard http://nicolaspouillard.fr
William Morgan
2009-Sep-09 14:37 UTC
[sup-talk] [PATCH] Add ''a'' and ''d'' keybindings to thread-view-mode to archive/delete current thread
Reformatted excerpts from William Morgan''s message of 2009-09-08:> So far I''m counting 3 yes votes (including patch submitter), 3 "won''t > use them" votes (including me), and no one being offended. I think I''m > going to apply this.Applied to master. Thanks! -- William <wmorgan-sup at masanjin.net>