36248 - Should reflect correct change right away on "\Noselect" folder on UI for IMAP servers

Status: VERIFIED FIXED

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

Description

[Karen Huang]

Used 04-16-10-M16 commercial build

Not greyout right away for the folder with "/" folder name

When I create the "abc/def" folder, it should grey out right away, but it 
didn't -- it will display as normal at first until I select & expand the "abc" 
folder in order to see "def" subfolder...then abc folder will turn to grey....

1) Login to IMAP mail account.
2) Create abc/def folder.
3) Actual Results: "abc" folder should grey out right away, but it 
didn't -- it will display as normal at first until I select & expand the "abc" 
folder.

Expected Results: "abc" folder should grey out right away.

Comment 1

[Ben Bucksch (:BenB)]

I think, Karen is speaking about folder-only folders.

Comment 2

[selmer (gone)]

Target M17.

Comment 3

[selmer (gone)]

Scott, is this really the front end or are you being given the wrong state
information?

Comment 4

[scottputterman]

Jeff, it sounds like we aren't being told about the noselect flag right away 
when a folder is created.  Does this sound like it could be yours?  Or do we 
need to hook up notifications for a change in the noselect flag?

Comment 5

[jefft]

I'll take this one...

Comment 6

[Karen Huang]

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

Comment 7

[Karen Huang]

Updating this summary since this bug not only need to verify on NMS 4.1 "a/b" 
folder, it need to verify on other IMAP servers which with "\Noselect" flag 
folder.

Comment 8

[scottputterman]

Jeff, is this something we need for beta3?

Comment 9

[jefft]

No, future should be fine.

Comment 10

[scottputterman]

Reassigning to naving

Comment 11

[Karen Huang]

Since UI will base on the IMAP server implementation (with \Noselect flag) to 
reflect the correct change on UI. The fix for this bug at least should include:
1) NMS 4.1 "a/b" folder 
2) UW IMAP "non dual-use" & "w/slash" folder.

Comment 12

[Navin Gupta]

Attachment: Proposed fix

Comment 13

[Navin Gupta]

CC'ing mscott for review + super-review

Comment 14

[Navin Gupta]

fix checked in 

Comment 15

[Karen Huang]

Verified on 02-16-09-Mtrunk build:
1) NMS 4.1 "a/b" folder 
2) UW IMAP "non dual-use" & "w/slash" folder.
All above "\Noselect" folders grey out right away now.
Good work! Naving. Marking as verified now.