Sorting by order received doesn't clear sort direction indicator

RESOLVED FIXED

Status

--
minor
RESOLVED FIXED
17 years ago
14 years ago

People

(Reporter: neil, Assigned: sspitzer)

Tracking

({polish})

Trunk
x86
Windows 95
polish

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

17 years ago
Using Build ID: 2002012203
Steps to reproduce problem:
1. Sort by e.g. subject
2. Sort by order received (via menu)

Expected results: sort direction indicator on subject column is cleared

Actual results: sort direction indicator persists on subject column

Updated

17 years ago
QA Contact: esther → laurel
(Reporter)

Comment 1

17 years ago
Created attachment 71299 [details] [diff] [review]
Proposed patch; see also diff -uw below

Index: threadPane.js
===================================================================
RCS file: /cvsroot/mozilla/mailnews/base/resources/content/threadPane.js,v
retrieving revision 1.43
diff -u -w -r1.43 threadPane.js
--- threadPane.js	2002/01/16 03:01:08	1.43
+++ threadPane.js	2002/02/25 13:27:03
@@ -267,16 +267,16 @@
       else {
	 sortedColumn.setAttribute("sortDirection","descending");
       }
+    }
+  }

       // remove the sort indicator from all the columns
       // except the one we are sorted by
-      var currCol = sortedColumn.parentNode.firstChild;
+  var currCol = document.getElementById("threadCol");
       while (currCol) {
	 if (currCol != sortedColumn && currCol.localName == "outlinercol")
	   currCol.removeAttribute("sortDirection");
	 currCol = currCol.nextSibling;
-      }
-    }
   }
 }
(Reporter)

Updated

17 years ago
Keywords: patch, polish, review
(Reporter)

Comment 2

16 years ago
This was fixed by the patch to bug 44823.
Status: NEW → RESOLVED
Last Resolved: 16 years ago
Resolution: --- → FIXED
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.