From 83a71b089cdb6ff16b0a54f93bca3f2bb23434ff Mon Sep 17 00:00:00 2001 From: the-djmaze <> Date: Mon, 12 Aug 2024 15:24:30 +0200 Subject: [PATCH] Don't use focusedMessage for active menu items, it could confuse --- dev/Stores/User/Messagelist.js | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/dev/Stores/User/Messagelist.js b/dev/Stores/User/Messagelist.js index c77318410..8afda5478 100644 --- a/dev/Stores/User/Messagelist.js +++ b/dev/Stores/User/Messagelist.js @@ -108,9 +108,8 @@ addComputablesTo(MessagelistUserStore, { listCheckedOrSelected: () => { const selectedMessage = MessagelistUserStore.selectedMessage(), - focusedMessage = MessagelistUserStore.focusedMessage(), checked = MessagelistUserStore.filter(item => isChecked(item)); - return checked.length ? checked : (selectedMessage || focusedMessage ? [selectedMessage || focusedMessage] : []); + return checked.length ? checked : (selectedMessage ? [selectedMessage] : []); }, listCheckedOrSelectedUidsWithSubMails: () => { @@ -138,7 +137,6 @@ MessagelistUserStore.hasChecked = koComputable( MessagelistUserStore.hasCheckedOrSelected = koComputable(() => !!MessagelistUserStore.selectedMessage() - | !!MessagelistUserStore.focusedMessage() // Issue: not all are observed? | !!MessagelistUserStore.find(isChecked) ).extend({ rateLimit: 50 });