delete current message does not move to next unread message

[nobody]

Every mail client moves to the next unread message
after deleting or filing the message currently being
viewed. Roundcube does not. This is a major impedement
of workflow.

Great program, elsewise. Thanks!

[cmcnulty]

Logged In: YES 
user_id=275835

checked in fix for this, but it still needs a tiny bit more
work.  It doesn't work right now for the case where deleting
a messager permanantly deletes it (only works for moving
message, in other words).  Also doesn't work for deleting
last message in the list (where it needs to go up instead of
down).  Should have fixes for those last two issues soon.

[cmcnulty]

Logged In: YES 
user_id=275835

These two issues are now fixed (as are the bugs that the
previous patch introduced.  Only remaining "issue" is that
when you delete the last message of the list, RC goes out
and fetches the next message.  This next message isn't the
one that is highlighted, nor should it be, IMO.  

[micah]

Hi,

I hate to re-open old bugs, but it doesn't appear as if this works. I'm using the latest svn checkout ([89406f36]) and if I delete a message (while viewing it), or move it to Trash, I am returned to the message index, rather than brought to the next message.

In other mail clients there are actually three buttons: delete, delete+next, delete+prev, because some people prefer to go one way rather than the other.

[zyzzyvas]

I can confirm that this is NOT FIXED for me using 0.1beta2 on Firefox 1.5.0.7. I just get dumped back to the inbox after a delete or move.

This seriously cramps efficiency. Looking forward to a fix.

[robin]

Please try this and see if it does not break anything else:

Index: program/js/app.js
===================================================================
--- program/js/app.js   (revision 385)
+++ program/js/app.js   (working copy)
@@ -3150,7 +3150,7 @@
       case 'delete':
       case 'moveto':
         if (this.env.action=='show')
-          this.command('list');
+          this.command('nextmessage','',this);
         break;

       case 'list':

[jpingle]

That change does fix the problem for me when viewing the messages full screen, however when using the preview pane on SVN [733c78f4] it does not load the next message into the preview pane.

It can be very confusing to a user if they delete a message in the preview pane and not have the body of the deleted message cleared out or replaced with the next message.

[seansan]

review in 0.1.1

[lancey]

In SVN 1102, deleting/moving an opened message gets us back to the inbox.

Deleting/moving a message from the messages list with preview pane enabled correctly refreshes the preview pane, or clears it out if the message was the last one.

[till]

I can confirm that this works (Firefox, Safari, IE)