msgstore: leave visual-mark mode after certain commands

Commit "4753cfd visual-mode: deselect messages after performing command"
introduced the behavior of leaving visual mark mode after performing
certain commands. Add this behavior to additional commands:

- Delete
- Archive
- Move

Remark the selected mail files if an error occurred during the
operation.

Signed-off-by: Tim Culverhouse <tim@timculverhouse.com>
Acked-by: Robin Jarry <robin@jarry.cc>

1 files changed, 3 insertions, 0 deletions

diff --git a/commands/msg/delete.go b/commands/msg/delete.go
index ce5c4ca5..d90132aa 100644
--- a/commands/msg/delete.go
+++ b/commands/msg/delete.go
@@ -44,6 +44,7 @@ func (Delete) Execute(aerc *widgets.Aerc, args []string) error {
 	}
 	//caution, can be nil
 	next := findNextNonDeleted(uids, store)
+	store.ClearVisualMark()
 	store.Delete(uids, func(msg types.WorkerMessage) {
 		switch msg := msg.(type) {
 		case *types.Done:
@@ -79,8 +80,10 @@ func (Delete) Execute(aerc *widgets.Aerc, args []string) error {
 			}
 			acct.Messages().Invalidate()
 		case *types.Error:
+			store.Remark()
 			aerc.PushError(msg.Error.Error())
 		case *types.Unsupported:
+			store.Remark()
 			// notmuch doesn't support it, we want the user to know
 			aerc.PushError(" error, unsupported for this worker")
 		}