From 31c69e8e253c6b359e1bd05eebe4168bff74ffdc Mon Sep 17 00:00:00 2001 From: "bienvenu%nventure.com" Date: Thu, 24 Jul 2003 20:15:09 +0000 Subject: [PATCH] fix 213768 reverse sort by threads broken, r/sr=mscott --- mailnews/base/resources/content/threadPane.js | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/mailnews/base/resources/content/threadPane.js b/mailnews/base/resources/content/threadPane.js index 0ead6411a180..a3e3b1eb408a 100644 --- a/mailnews/base/resources/content/threadPane.js +++ b/mailnews/base/resources/content/threadPane.js @@ -111,11 +111,13 @@ function HandleColumnClick(columnID) // if sortType is 0, this is an unsupported sort type // return, since we can't sort by that column. - if (sortType == 0) { + if (sortType == 0) return; - } var dbview = GetDBView(); + // if we're already threaded, clicking the thread icon does a reverse sort by id. + if (sortType == nsMsgViewSortType.byThread && (dbview.viewFlags & nsMsgViewFlagsType.kThreadedDisplay)) + sortType = nsMsgViewSortType.byId; if (sortType == nsMsgViewSortType.byThread && !dbview.supportsThreading) return; if (dbview.sortType == sortType) {