* [sup-talk] [PATCH] Add 'a' and 'd' keybindings to thread-view-mode to archive/delete current thread
@ 2009-08-26 22:40 Carl Worth
2009-09-03 18:06 ` William Morgan
0 siblings, 1 reply; 10+ messages in thread
From: Carl Worth @ 2009-08-26 22:40 UTC (permalink / raw)
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>
^ permalink raw reply [flat|nested] 10+ messages in thread
* [sup-talk] [PATCH] Add 'a' and 'd' keybindings to thread-view-mode to archive/delete current thread
2009-08-26 22:40 [sup-talk] [PATCH] Add 'a' and 'd' keybindings to thread-view-mode to archive/delete current thread Carl Worth
@ 2009-09-03 18:06 ` William Morgan
2009-09-03 18:37 ` Mark Alexander
` (2 more replies)
0 siblings, 3 replies; 10+ messages in thread
From: William Morgan @ 2009-09-03 18:06 UTC (permalink / raw)
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>
^ permalink raw reply [flat|nested] 10+ messages in thread
* [sup-talk] [PATCH] Add 'a' and 'd' keybindings to thread-view-mode to archive/delete current thread
2009-09-03 18:06 ` William Morgan
@ 2009-09-03 18:37 ` Mark Alexander
2009-09-03 18:58 ` Nicholas Bergson-Shilcock
2009-09-06 19:43 ` Nicolas Pouillard
2009-09-08 15:17 ` Wirt Wolff
2 siblings, 1 reply; 10+ messages in thread
From: Mark Alexander @ 2009-09-03 18:37 UTC (permalink / raw)
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.
^ permalink raw reply [flat|nested] 10+ messages in thread
* [sup-talk] [PATCH] Add 'a' and 'd' keybindings to thread-view-mode to archive/delete current thread
2009-09-03 18:37 ` Mark Alexander
@ 2009-09-03 18:58 ` Nicholas Bergson-Shilcock
0 siblings, 0 replies; 10+ messages in thread
From: Nicholas Bergson-Shilcock @ 2009-09-03 18:58 UTC (permalink / raw)
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.
^ permalink raw reply [flat|nested] 10+ messages in thread
* [sup-talk] [PATCH] Add 'a' and 'd' keybindings to thread-view-mode to archive/delete current thread
2009-09-03 18:06 ` William Morgan
2009-09-03 18:37 ` Mark Alexander
@ 2009-09-06 19:43 ` Nicolas Pouillard
2009-09-08 20:26 ` Carl Worth
2009-09-08 15:17 ` Wirt Wolff
2 siblings, 1 reply; 10+ messages in thread
From: Nicolas Pouillard @ 2009-09-06 19:43 UTC (permalink / raw)
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
^ permalink raw reply [flat|nested] 10+ messages in thread
* [sup-talk] [PATCH] Add 'a' and 'd' keybindings to thread-view-mode to archive/delete current thread
2009-09-03 18:06 ` William Morgan
2009-09-03 18:37 ` Mark Alexander
2009-09-06 19:43 ` Nicolas Pouillard
@ 2009-09-08 15:17 ` Wirt Wolff
2009-09-08 20:08 ` William Morgan
2 siblings, 1 reply; 10+ messages in thread
From: Wirt Wolff @ 2009-09-08 15:17 UTC (permalink / raw)
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
^ permalink raw reply [flat|nested] 10+ messages in thread
* [sup-talk] [PATCH] Add 'a' and 'd' keybindings to thread-view-mode to archive/delete current thread
2009-09-08 15:17 ` Wirt Wolff
@ 2009-09-08 20:08 ` William Morgan
2009-09-09 14:37 ` William Morgan
0 siblings, 1 reply; 10+ messages in thread
From: William Morgan @ 2009-09-08 20:08 UTC (permalink / raw)
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>
^ permalink raw reply [flat|nested] 10+ messages in thread
* [sup-talk] [PATCH] Add 'a' and 'd' keybindings to thread-view-mode to archive/delete current thread
2009-09-06 19:43 ` Nicolas Pouillard
@ 2009-09-08 20:26 ` Carl Worth
2009-09-08 20:33 ` Nicolas Pouillard
0 siblings, 1 reply; 10+ messages in thread
From: Carl Worth @ 2009-09-08 20:26 UTC (permalink / raw)
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>
^ permalink raw reply [flat|nested] 10+ messages in thread
* [sup-talk] [PATCH] Add 'a' and 'd' keybindings to thread-view-mode to archive/delete current thread
2009-09-08 20:26 ` Carl Worth
@ 2009-09-08 20:33 ` Nicolas Pouillard
0 siblings, 0 replies; 10+ messages in thread
From: Nicolas Pouillard @ 2009-09-08 20:33 UTC (permalink / raw)
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
^ permalink raw reply [flat|nested] 10+ messages in thread
* [sup-talk] [PATCH] Add 'a' and 'd' keybindings to thread-view-mode to archive/delete current thread
2009-09-08 20:08 ` William Morgan
@ 2009-09-09 14:37 ` William Morgan
0 siblings, 0 replies; 10+ messages in thread
From: William Morgan @ 2009-09-09 14:37 UTC (permalink / raw)
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>
^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2009-09-09 14:37 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2009-08-26 22:40 [sup-talk] [PATCH] Add 'a' and 'd' keybindings to thread-view-mode to archive/delete current thread Carl Worth
2009-09-03 18:06 ` William Morgan
2009-09-03 18:37 ` Mark Alexander
2009-09-03 18:58 ` Nicholas Bergson-Shilcock
2009-09-06 19:43 ` Nicolas Pouillard
2009-09-08 20:26 ` Carl Worth
2009-09-08 20:33 ` Nicolas Pouillard
2009-09-08 15:17 ` Wirt Wolff
2009-09-08 20:08 ` William Morgan
2009-09-09 14:37 ` William Morgan
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox