Fix crash when deleting last mail and then pressing delete againmail_delete_crash

.. wonder if this should be handled at ListBox level, instead.

i.e. calling setModel again, but that might resize the listbox? Plus,
would still want to select last mail, instead of invalidating index.

1 files changed, 6 insertions, 0 deletions

diff --git a/src/gui/windows/mailwindow.cpp b/src/gui/windows/mailwindow.cpp
index 4f5f9cac5..64dbb53e4 100644
--- a/src/gui/windows/mailwindow.cpp
+++ b/src/gui/windows/mailwindow.cpp
@@ -255,6 +255,12 @@ void MailWindow::removeMail(const int64_t id)
         if (message != nullptr)
             mMailModel->add(getMailHeader(message));
     }
+
+    const auto lastIndex = mMailModel->getNumberOfElements() - 1;
+    if (mListBox->getSelected() > lastIndex)
+    {
+        mListBox->setSelected(lastIndex);
+    }
 }
 
 void MailWindow::showMessage(MailMessage *const mail,