Closed Bug 97859 Opened 23 years ago Closed 23 years ago

Twisty doesn't display after creating subfolders until collapse/expand folders or relaunch mail again

Categories

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

x86
Windows NT
defect

Tracking

(Not tracked)

VERIFIED FIXED
mozilla0.9.5

People

(Reporter: huang, Assigned: waterson)

References

Details

(Keywords: regression, Whiteboard: [folderpane], [PDT+], [ETA ?])

Attachments

(2 files)

Used 08-31-09-trunk build Twisty doesn't display after creating subfolders until collapse/expand folders or relaunch mail again 1)Login to an IMAP mail account 2)After create 1st level fodler & few levels subfolders 3)Actual Results: Twisty doesn't display for those subfolders that I just created. After I collapse & expand or relaunch the mail again, the twisty will display again. Expected results: twisty should display right away after creating subfolders.
QA Contact: esther → huang
Whiteboard: [folderpane]
Should this nominate nsBranch? Since users will misunderstand that the twisty disappear.....
Keywords: nsbranch
I've seen a similar problem with news. create a new news accounts, and subscribe to a group. the twisty doesn't appear for the account until you restart.
Status: NEW → ASSIGNED
There aren't any comments on this bug since the 21st of Sept. Can QA regess against the Netscape commercial builds and determine if this is still a valid bug? Please mark as nsbranch+ which will get this on the PDT radar if you think this is critical and can give us an ETA for the fix. Else, mark is as nsbranch-. Also, can someone comment in the bug how serious you think this is? PDT is only accepting "stop ship" bugs at this point such as data loss, loss of major functionality, regressions and bugs to the eMojo "stop ship" features.
I am sure this is still a valid bug - this bug was filed and commented on very recently, however, we should find out if this is a regression from 6.1. Karen - is this a regression from Netscape 6.1?
Yes. This is a regression from Netscape 6.1. Adding regression for the keywords. Users obviously will see Netscape 6.2 is worse than Netscape 6.1 for such UI problem...... :-(
Keywords: regression
after some debugging, I think this is going to be an outliner builder bug. waterson, can you help me out?
Although this is a regression, I don't see that it's a stop-ship bug. I would only consider a patch that is trivial in nature.
Whiteboard: [folderpane] → [folderpane], [PDT], [ETA ?]
Share the love!
Assignee: sspitzer → waterson
Status: ASSIGNED → NEW
Status: NEW → ASSIGNED
Priority: -- → P3
Target Milestone: --- → mozilla0.9.5
So am I correct in understanding that this happens when you: 1. Have an ``open'' parent folder that contains a subfolder. 2. You remove the subfolder. 3. You re-start. 4. You now attempt to add a subfolder to the original parent folder. In which case, the new subfolder appears, but the twisty won't show up until you restart. Based on the test case I've been playing with, it looks like we need to do steps (1) and (2) to get some state into the localstore (persisting the ``open''-ness of the parent folder).
If (1) and (2) are required for this bug, I would suggest nsbrach-
The fix is pretty trivial: replace ``else if'' with ``if''. OTOH, it seems like it will probably be pretty hard for a real user to get into this state, and even if they do, it's really only a minor visual glitch that should fix itself the next time the user opens the mail window. Therefore, I'm going to mark nsbranch-. If I've misunderstood the severity of the bug, please feel free to slap me.
Keywords: nsbranchnsbranch-, patch
cc'ing some more rdfliner cronies.
makes sense to me r=varga
sr=hyatt
works great! do you consider this too risky for the branch? I'd like to have it on the branch (I can land it for you), since it will fix several mailnews folderpane UI bugs. I'll go find those dups now.
*** Bug 99915 has been marked as a duplicate of this bug. ***
Chris, if you believe the risk for this change is low then I would really like to take it for the branch. I'll do all the dirty work at PDT I just need 2 things from you: 1) risk assessment from you (it looks pretty low risk to me) 2) can you check it into the trunk today so we can get testing on it before Monday
Blocks: 99508
Keywords: nsbranch-nsbranch+
*** Bug 101047 has been marked as a duplicate of this bug. ***
this bug also fixes #101047, which is that on folder discovery folders don't get twisties.
Fixed on the trunk.
Pls check it in, and update Patch Status - PDT+
Whiteboard: [folderpane], [PDT], [ETA ?] → [folderpane], [PDT+], [ETA ?]
Checked in to MOZILLA_0_9_4_BRANCH.
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
Verified on all the platforms: WinNT 10-01-05-0.9.4 Linux 10-01-04-0.9.4 Mac 10-01-03-0.9.4 Twisty is displaying after creating subfolders now. Marking as verified.
Status: RESOLVED → VERIFIED
Verified on WinNT 10-01-09-trunk build as well.
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: