Closed Bug 110320 Opened 23 years ago Closed 23 years ago

QuickSearch: results don't sort correctly until re-sort (Date, Subj, possible others)

Categories

(SeaMonkey :: MailNews: Message Display, defect, P2)

defect

Tracking

(Not tracked)

VERIFIED FIXED
mozilla0.9.9

People

(Reporter: laurel, Assigned: naving)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

Using nov15 commercial trunk build

The search toolbar (a.k.a. QuickSearch) in the 3-pane window is supposed to show
search results in the same sort as the thread pane was in prior to sort. After
results are displayed, the use can change sort.

In the current build the type of sort is preserved when initially displaying
results, but the results are not always sorted appropriately.  I noticed this
consistently with both Subject and Date sorts. Haven't yet tried all sorts.

1.  Select a large mail folder or newsgroup and sort by date with the newest
messages at the top of the pane.
2.  Type a short string in the QuickSearch text field which will yield many
results over various dates.
    Result:  messages are displayed in date sort, but the newest messages are at
the bottom of the results thread pane.  Still the ascending/descending arrow on
the column heading is in the same direction as was in thread pane prior to sort.
3.  Select a large mail folder or newsgroup and sort by Subject (either
ascending or descending).
2.  Type a short string in the QuickSearch text field which will yield many
results over various subjects.
    Result:  messages are displayed in subject sort, but many messages are out
of alphabetical order.  Not all are misplaced, but many within the list. Once
you re-sort the results by subject the messages will sort correctly.
Keywords: nsbeta1
Status sort behaves similarly to Subj -- status sort before search, search
results are displayed in a pseudo status sort where some results are not in order.
I am not sorting the QS results.  I just add it to the view as we 
are notified of search hits. I can make it to sort after search is over.

QA Contact: esther → laurel
Blocks: 106943
*** Bug 112186 has been marked as a duplicate of this bug. ***
I've noticed that when a sort is performed, the sort order is actually
incorrect. Not sure whether to open a separate bug or just report here.

1. Sort msgs by date
2. Perform a quick search

(For me the results appear sorted but in the reverse direction. This bug seems
to indicate sorting doesn't happen at all. Perhaps it's just because I'm sorting
by date)

3. Click on date column to sort in reverse direction.

Expected results:
Sorting should be in the same direction indicated by the sort indicator.

Actual results:
Sorting is in the opposite direction indicated by the sort indicator.
Keywords: nsbeta1nsbeta1+
Priority: -- → P2
Status: NEW → ASSIGNED
*** Bug 116402 has been marked as a duplicate of this bug. ***
*** Bug 118021 has been marked as a duplicate of this bug. ***
*** Bug 119784 has been marked as a duplicate of this bug. ***
Target Milestone: --- → mozilla0.9.9
Attached patch proposed fixSplinter Review
sort the results after search is over.
cc bienvenu for review.
Comment on attachment 64911 [details] [diff] [review]
proposed fix

r=bienvenu
Attachment #64911 - Flags: review+
fix checked in.
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
OK using jan29 commercial trunk: win98, linux rh6.2
OK using jan28 commercial trunk: Mac OS X.2

Varied sorts checked. Basically working -- results are sorted according to the
previous/folder view's sort as soon as the search is completed.  When search is
still in progress,  results are added to view but not sorted.

Note:  as according to spec, search results not threaded.  So if folder view
threaded, when search completes it is a default date sort.

Any further issues with search sort will be logged separately.
Status: RESOLVED → VERIFIED
*** Bug 122474 has been marked as a duplicate of this bug. ***
*** Bug 122852 has been marked as a duplicate of this bug. ***
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: