171965 - Thread pane not getting invalidated when changing sort order

Status: RESOLVED FIXED

(SeaMonkey :: MailNews: Message Display, defect)

Description

[Scott MacGregor]

I noticed this while testing mail views, but the problem is really in the new
quick search view class I think.

1) Enter some criteria in the quick search bar that will generate multiple
search results

2) Try changing a sort on the results. Change ascending vs. descending, or
choose a different sort criteria. 

Notice that the results don't change to match the new sort order.

However, if you just make the window repaint, the correct list is shown. We just
aren't invalidating the view after a sort is performed when inside of a quick
search view. Anything that forces a repaint makes the results show up in the
correct order.

Comment 1

[Navin Gupta]

Attachment: proposed fix

The fix is to override Sort for quickSearch so that we can invalidate the tree
when the sort is done.

Comment 2

[Navin Gupta]

Cavin, Scott, can you do the reviews ? thx. 

Comment 3

[Scott MacGregor]

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

sr=mscott

Thanks for the quick turn around on this fix.

Comment 4

[Cavin Song]

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

r=cavin.

Comment 5

[Navin Gupta]

fix checked in.