From da16f55d18b61d3d9be0bc0446df8f315097a67d Mon Sep 17 00:00:00 2001 From: Martin Rotter Date: Wed, 10 Aug 2016 13:09:02 +0200 Subject: [PATCH] Tweak multi-selection messages behavior. --- resources/text/CHANGELOG | 3 ++- src/gui/messagesview.cpp | 4 ++-- 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/resources/text/CHANGELOG b/resources/text/CHANGELOG index 34b16c14d..8e2382810 100755 --- a/resources/text/CHANGELOG +++ b/resources/text/CHANGELOG @@ -7,10 +7,11 @@ Added: ▪ Application settings now offers "Apply" button which saves settings without closing the dialog. (issue #16) Fixed: -▪ Correct message is now selected after some feed gets updated. (bug #46) +▪ Correct message is now selected after some feed gets updated. Multi-selection is not maintained, only focused ("current") message selection is. (bug #46) ▪ RSS Guard correctly saves windows size/position when maximized on exit. (bug #45) Changed: +▪ Focused message is now displayed in previewer when multiple messages are selected. This is related to bug #46. ▪ MariaDB binary DLL file updated to 10.1.16 (Windows only). 3.3.2 diff --git a/src/gui/messagesview.cpp b/src/gui/messagesview.cpp index afcbda794..c8783634e 100755 --- a/src/gui/messagesview.cpp +++ b/src/gui/messagesview.cpp @@ -232,8 +232,8 @@ void MessagesView::selectionChanged(const QItemSelection &selected, const QItemS current_index.row(), current_index.column(), mapped_current_index.row(), mapped_current_index.column()); - if (mapped_current_index.isValid() && selected_rows.count() == 1) { - Message message = m_sourceModel->messageAt(m_proxyModel->mapToSource(selected_rows.at(0)).row()); + if (mapped_current_index.isValid() && selected_rows.count() > 0) { + Message message = m_sourceModel->messageAt(m_proxyModel->mapToSource(current_index).row()); if (!m_batchUnreadSwitch) { // Set this message as read only if current item