From f1e7bbd1bda7796fed1971d3c296926d8547c311 Mon Sep 17 00:00:00 2001
From: Aleksander Machniak <alec@alec.pl>
Date: Wed, 14 Oct 2015 03:44:55 -0400
Subject: [PATCH] Fix bug where message preview was unintentionally reset on check-recent action (#1490563)

---
 CHANGELOG         |    1 +
 program/js/app.js |    9 +++++----
 2 files changed, 6 insertions(+), 4 deletions(-)

diff --git a/CHANGELOG b/CHANGELOG
index e19b976..fc14da5 100644
--- a/CHANGELOG
+++ b/CHANGELOG
@@ -41,6 +41,7 @@
 - Fix so unrecognized TNEF attachments are displayed on the list of attachments (#1490351)
 - Fix so database_attachments::cleanup() does not remove attachments from other sessions (#1490542)
 - Fix responses list update issue after response name change (#1490555)
+- Fix bug where message preview was unintentionally reset on check-recent action (#1490563)
 
 RELEASE 1.1.3
 -------------
diff --git a/program/js/app.js b/program/js/app.js
index e346356..bb1dd32 100644
--- a/program/js/app.js
+++ b/program/js/app.js
@@ -2514,22 +2514,23 @@
   // removes messages that doesn't exists from list selection array
   this.update_selection = function()
   {
-    var selected = this.message_list.selection,
-      rows = this.message_list.rows,
+    var list = this.message_list,
+      selected = list.selection,
+      rows = list.rows,
       i, selection = [];
 
     for (i in selected)
       if (rows[selected[i]])
         selection.push(selected[i]);
 
-    this.message_list.selection = selection;
+    list.selection = selection;
 
     // reset preview frame, if currently previewed message is not selected (has been removed)
     try {
       var win = this.get_frame_window(this.env.contentframe),
         id = win.rcmail.env.uid;
 
-      if (id && $.inArray(id, selection) < 0)
+      if (id && !list.in_selection(id))
         this.show_contentframe(false);
     }
     catch (e) {};

--
Gitblit v1.9.1