60776 - Cross-folder navigation should not land on Trash folder.

Status: VERIFIED FIXED

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

Description

[laurel]

Using nov 17th commercial trunk build

This is one of the issues carried forward from main feature bug #17801...

Trash folders should be ignored by cross-folder navigation. Currently it seems
if the trash folder has unread messages, it is not landed on immediately, but
rather takes a couple rounds through the folders... I haven't figured out the
details, but you will be promptec for navigation to the trash folder (I was
using IMAP).

Comment 1

[R.K.Aa.]

*** Bug 60777 has been marked as a duplicate of this bug. ***

Comment 2

[laurel]

*** Bug 61214 has been marked as a duplicate of this bug. ***

Comment 3

[laurel]

*** Bug 61214 has been marked as a duplicate of this bug. ***

Comment 4

[laurel]

nominating for 6.5

Comment 5

[scottputterman]

marking nsbeta1+ and moving to mozilla0.8 milestone

Comment 6

[scottputterman]

moving to mozilla0.9

Comment 7

[scottputterman]

I think I have a fix for this. I will attach it shortly.

Comment 8

[scottputterman]

Attachment: patch for bug.

Comment 9

[scottputterman]

reassigning to me.  I might wait a bit to go through the review and checking in
process because I think I know how to fix a bunch of the other folder navigation
bugs and this might have to change because of that.

Comment 10

[scottputterman]

reassigning to chuang. Candice, if this patch still works, can you check this
in? I'm not sure if this will cause a merge problem with the perf branch landing.

Comment 11

[(not reading, please use [e-mail] instead)]

taking from chuang.  this will cause a conflict on the branch.  I'm about to get
"cross folder" navigation to work on the branch, so I'll apply this patch there.

Comment 12

[scottputterman]

moving to mozilla0.9.1

Comment 13

[(not reading, please use [e-mail] instead)]

fixed.

thanks to putterman for the initial patch.  (I've since changed
IsSpecialFolder() to use bit flags instead of strings.)

Comment 14

[fenella]

Linux (2001-03-27-05 mtrunk)
Win32 (2001-03-27-06 mtrunk)
Mac (2001-03-26-12 mtrunk)
This works fine now.