18017 - Sidebar panels not showing up in My Panels of Mail 3 pane

Status: VERIFIED FIXED

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

Description

[daver]

In today's build (11/4/99) and in past builds, when you slide my panels up, none
of the default sidebar panels are there like what useless, tinderbox, etc.

Comment 1

[Phil Peterson]

Reassign to hangas

Comment 2

[Steve Lamm]

I will check this out when my morning build finishes.

Looking messenger.xul and msgMail3PaneWindow.js, I see a few things things that
need tweeking, but not anything that should cause this bug.

Comment 3

[Steve Lamm]

Chris, I do not see any obvious reason why the panels are not showing up in the
Mail window. They sidebar template is getting applied since the "My Panels"
splitter is create. For some reason the individual panels are not created. Is
the brutal sharing messing this up?

Comment 4

[Chris Waterson]

it's possible, but I kinda doubt it. sounds similar to the bug where sidebar
doesn't show up after "View Sidebar"

Comment 5

[Steve Lamm]

I figured it out. The sidebarOverlayInit() function tries to set the value of
"menu_sidebar" which is the "View / Sidebar" menu. I have a change that checks
if the menu is there. We can also decide if we want to add the menu item to
messenger. Paul, will you review the patch? I will attach it.

Comment 6

[Steve Lamm]

Attachment: Patch for sidebarOverlay.js (Test for menu before setting it)

Comment 7

[Steve Lamm]

I checked in the fix.

Comment 8

[Steve Lamm]

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

Comment 9

[lchiang]

verified 1999-11-09-11-m11 commercial win32 build.