114908 - Sidebar Tabs menu stops working after close/reopen

Status: VERIFIED FIXED

(SeaMonkey :: Sidebar, defect, P2)

Description

[Luke Koleszar]

From Bugzilla Helper:
User-Agent: Mozilla/5.0 (Windows; U; Win 9x 4.90; en-US; rv:0.9.6+) Gecko/20011212
BuildID:    2001121203

The tabs pulldown menu stops working after closing the sidebar and reopening it

Reproducible: Always
Steps to Reproduce:
1. Open sidebar
2. Click Tabs menu, menu pops up
3. Close sidebar (F9)
4. Reopen sidebar (F9)
5. Click Tabs menu, no menu pops up

Actual Results:  Tabs menu no longer works

Expected Results:  Tabs menu continues to work normally

Works as expected in Mozilla 0.9.6
If you open a new window then the Tabs menu works in the new window until you
close and reopen the sidebar in that window.

Comment 1

[Tracy Walker [:tracy]]

currently can't test this do to sidebar not being present at all (commercial 
builds)

Comment 2

[R.K.Aa.]

seeing this on linux too, current CVS.

Comment 3

[Samir Gehani]

This regressed before 2001-12-11.  Narrowing down.  

Comment 4

[Samir Gehani]

Attachment: Hide menu (rather than collapse it) so it must rebuild next time and can display.

Comment 5

[Samir Gehani]

morse, please r.
dveditz, please sr.

Thanks.

Comment 6

[Stephen P. Morse]

Comment on attachment 61547 [details] [diff] [review]
Hide menu (rather than collapse it) so it must rebuild next time and can display.

r=morse

Comment 7

[Daniel Veditz [:dveditz]]

Comment on attachment 61547 [details] [diff] [review]
Hide menu (rather than collapse it) so it must rebuild next time and can display.

sr=dveditz

Please be kinder to reviewers unfamiliar with your code in your choice of
context. If you had chosen -8 instead of -6 the entire function, including
name, would have been in one contiguous chunk and I could have reviewed it
without having to fire up my editor on the source.

Comment 8

[Samir Gehani]

Sure, will keep in mind more context when deemed appropriate.  Thanks for the
reviews, folks.

Comment 9

[Asa Dotzler [:asa]]

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

Comment 10

[Samir Gehani]

Checked in.

Comment 11

[sujay]

verified...REOPEN if this is still a problem .

looks fixed to me...