154089 - "My Sidebar" in Address Book can't show at the first time

Status: VERIFIED FIXED

(SeaMonkey :: Sidebar, defect)

Description

[Cindy Fan]

Steps to Reproduce:
1.Open the browser.
2.Go to Address Book window by clicking "Tools"->"Address Book".
3.Click the little bar in the middle of left pane to minimize the "My sidebar".
4.Press F9 to hide "My sidebar".
5.Press F9 to show "My sidebar".
 
Expected Result:
"My sidebar" is showed correctly or is minimized.

Actual Result:
"My sidebar" is showed but no content at all.

Comment 1

[Gilles Durys]

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

Comment 2

[Gilles Durys]

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

Comment 3

[Gilles Durys]

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

Comment 4

[Gilles Durys]

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

Comment 5

[Gilles Durys]

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

Comment 6

[Gilles Durys]

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

Comment 7

[Gilles Durys]

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

Comment 8

[Silvia Zhao]

Reassign to Silvia

Comment 9

[Robin Lu]

Attachment: patch

Comment 10

[Robin Lu]

reassign to Robin Lu

Comment 11

[Robin Lu]

seek r=

Comment 12

[Samir Gehani]

Could you please list the test matrix this has been put through?  Have you
tested against old profiles, i.e., profiles created with Netscape 6.0, 6.1, 6.2,
7.0PR1?  Have you toggled the sidebar between collapsed, hidden, visible etc.? 
Has this been tested in the browser, mail, composer, and standalone IM?  Thanks.

Patch looks good (and innocuous).

Comment 13

[Robin Lu]

Samir Gehani,

Thanks for your comments!

Untill now, the test status is:

Test against old profile
Netscape 6.2.3 profile    Passed
The other old profile     NA      lack of old profiles


Toggle scenario:
1.Toggle between collapsed and visible with mouse
2.Toggle between hidden and visible with F9 or menu
3.Toggle from collapsed to hidden with F9 or menu
4.Toggle from collapsed to hidden then between hidden and visible with F9 or menu
Test in browser with scenario 1 to 4  passed on Redhat Linux 7.2 and Solaris 8
Test in mailnews with scenario 1 to 4 passed on Redhat Linux 7.2 and Solaris 8
Test in addressbook with scenario 1 to 4 passed on Redhat Linux 7.2 and Solaris 8
Test in composer with scenario 1 to 4  passed on Redhat Linux 7.2 and Solaris 8
Test with standalone IM 1 passed, 2 to 4 passed with menu, F9 is captured by IM.
Passed on Solaris 8 (Chinese Simplified)

Since the patch does not change the treatment of profile, can we consider the
test as pass?

Comment 14

[Stephen P. Morse]

Comment on attachment 90293 [details] [diff] [review]
patch

r=morse since patch looks harmless

Comment 15

[Stephen P. Morse]

Comment on attachment 90293 [details] [diff] [review]
patch

r=morse since patch looks harmless

Comment 16

[Robin Lu]

seek sr=?

Comment 17

[Kyle Yuan]

robin, make sure you have mailed the people you want to sr= this patch, add
cc-ed [e-mail]. bugmail is not going to be sufficient to get their
attention.

Comment 18

[Alec Flett]

Comment on attachment 90293 [details] [diff] [review]
patch

sr=alecf
(I'm one of the few super-reviewers who actually read my bugmail, so you don't
need to e-mail me :))

Comment 19

[Asa Dotzler [:asa]]

Comment on attachment 90293 [details] [diff] [review]
patch

a=asa (on behalf of drivers) for checkin to 1.1

Comment 20

[Kyle Yuan]

checked into trunk!

Comment 21

[Bolian Yin]

checked in NETSCAPE_7_0_OEM_BRANCH.

Comment 22

[Robin Lu]

verified at Mozilla 1.1b
Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.1b) Gecko/20020722