29813 - Collapse "My Sidebar" and its state is not remembered

Status: VERIFIED FIXED

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

Description

[Ninoschka Baca]

Build 2000-02-29-08M15: NT4, Linux 6.0, Mac 8.5.1

Overview: After collapsing the "My Sidebar" area, it's not remembered after 
closing/quiting and then restarting.

Steps to reproduce:
1. From Browser open Mail
2. In the 3-pane, resize the "My Sidebar" area down as far as it will go.
3. Select File|Close
4. Open Mail 
5. In the 3-pane, resize the "My Sidebar" area down as far as it will go again.
6. Select File|Quit
7. Restart Mail

Actual Results: Notice after steps 4  and 7 that the collapsed state is not 
remembered.
Expected Results: After a Close or a Quit and restarting Mail, the "My Sidebar" 
collapsed state should be remembered. It seems to remember the state if it's 
sized larger or smaller but not when it's collapsed.

Comment 1

[Ninoschka Baca]

This is important with multiple accounts. I can see that many mail users will 
want to maximize the mail folder view. 

Comment 2

[Paul MacQuiddy]

moving beta1 to keyword field. adding slamm, who said this is probably easy

This is pretty annoying and apparently one of daver's pet peeves :-)

Comment 3

[Phil Peterson]

[PDT+] w/b minus on 3/7

Comment 4

[scottputterman]

OK, then reassigning to slamm.  I'm not sure if this is a sidebar problem or a 
splitter problem.  The sidebar js for persisting height seems to be ok.  What 
happens is that even when I bring the Main Sidebar splitter to a collapsed 
state, it seems to always think that its height is 82 pixels (at least that's 
what keeps getting saved in my localstore.rdf). I've tried adding "collapse" to 
the persist tag for title-box in sidebaroverlay.xul but that doesn't work even 
though it does save the collapse state correctly.  I'm wondering if that's 
because we're setting the height to 82.  I guess we could also try setting the 
box height to 0 in this case though I'm not sure why that isn't the case anyway.

Comment 5

[Steve Lamm]

This is splitter bug #20546, but I have a work around. I already use the work
around for the main sidebar splitter. Applying the work around to the sub
splitter should be straight forward. I will try to have the work done today and
check it in no later than tomorrow.

Comment 6

[Steve Lamm]

That's weird, evaughan marked 20546 fixed a month ago. Either he didn't fix it,
or something else is causing this bug.

Comment 7

[don]

slamm and evaughan are working on a fix now ...

Comment 8

[Steve Lamm]

The fix is to add collased="true" to the splitters sibling that will be
collapsed. A style rule in global.css applies, "visibility: collapse".

Comment 9

[Steve Lamm]

Checked in the fix.

I fixed the grey horizontal splitter in the three pane window. If there is
another splitter that needs this fix, just add persist="collapsed" to the
sibling of the splitter whose collapsed state needs to be remembered.

Comment 10

[Ninoschka Baca]

Build 2000-03-08-08M15: Linux 6.0
Fixed. The collapse state is remembered after Closing and Quitting.

Will check the commercial Win and Mac builds when they pass the smoke test.

Comment 11

[Ninoschka Baca]

Build 2000-03-08-13M15: NT4, Mac 8.5.1
Verified Fixed.