168607 - Quick Search View not getting invalidated after the sort

Status: RESOLVED FIXED

(SeaMonkey :: MailNews: Message Display, defect)

Description

[Scott MacGregor]

This minor issue arose from the new quick search DB view class we recently
created. In short, after we do a quick search, we sort the results to match the
sort order for the folder. However we seem to be missing an invalidate after the
sort. The quick search results are not matching the sort order my folder is in.
But if I force a row to repaint itself, I see the right result underneath. 

Here's my aim excerpt with Navin:

BigMacNscp: change your sort order so your most recent mail is at the top
BigMacNscp: scroll to the top
BigMacNscp: type something into the QS bar like "Scott"
BigMacNscp: what I see is mail coming up in the wrong sort order
BigMacNscp: so the oldest scott message is first
BigMacNscp: however
navin1975: yeah, we sort after search is done
BigMacNscp: it turns out it is just a painting problem
BigMacNscp: if I move a window on top of the mail window we repaint and the most
recent scott is on top
BigMacNscp: so it looks like when we moved code into the new quick search DB
view class, we somehow lost an Invalidate() call after the search is done
BigMacNscp: the rows are definetly in the right order in the quick search view
class...cause if i force a row to repaint itself I see the correct row

Comment 1

[Navin Gupta]

Attachment: proposed fix

Invalidate the tree after sort is done.

Comment 2

[Scott MacGregor]

Comment on attachment 99174 [details] [diff] [review]
proposed fix

sr=mscott

Comment 3

[Navin Gupta]

fixed.