webmail: when moving out all messages in a thread (none remaining in …

…view), don't cause js error but select next message removing an item from the selected list should be done regardless of focus, i.e. the code snippet shouldn't have been behind the "if (focus...)" condition.
mjl- · Sep 22, 2023 · 3353062 · 3353062
1 parent be5f804
commit 3353062
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 8 deletions.
diff --git a/webmail/webmail.js b/webmail/webmail.js
@@ -3606,10 +3606,10 @@ const newMsglistView = (msgElem, listMailboxes, setLocationHash, otherMailbox, p
 					else if (i > 0) {
 						focus = mivs[i - 1];
 					}
-					const si = selected.indexOf(miv);
-					if (si >= 0) {
-						selected.splice(si, 1);
-					}
+				}
+				const si = selected.indexOf(miv);
+				if (si >= 0) {
+					selected.splice(si, 1);
 				}
 				// Strict cleanup.
 				miv.parent = null;

diff --git a/webmail/webmail.ts b/webmail/webmail.ts
@@ -3394,10 +3394,10 @@ const newMsglistView = (msgElem: HTMLElement, listMailboxes: listMailboxes, setL
 					} else if (i > 0) {
 						focus = mivs[i-1]
 					}
-					const si = selected.indexOf(miv)
-					if (si >= 0) {
-						selected.splice(si, 1)
-					}
+				}
+				const si = selected.indexOf(miv)
+				if (si >= 0) {
+					selected.splice(si, 1)
 				}
 				// Strict cleanup.
 				miv.parent = null