61305 - in mailnews, sidebar context menu has too many items

Status: RESOLVED INVALID

(SeaMonkey :: Sidebar, defect, P4)

Description

[Jesse Ruderman]

The context menu for the CNN sidebar shows too many items, but only in 
MailNews.  It works fine in the browser except for the "frame" problem (bug 
61304).  I didn't test with the stand-alone sidebar (the thing that Netscape 
Instant Messenger uses).

Comment 1

[shrirang khanzode]

mail news FE qa :nbaca. Pls send back to me if I reassigned incorrectly.

Comment 2

[german]

FYI: on mac I do not get any context menu at all in the CNN panel? Is this
related? I am using N6 RTM on MacOS 9.

Comment 3

[Ninoschka Baca]

Build 2000-11-28-09: Mac 9.04
I can see a context menu using the CNN panel using todays build.

Comment 4

[Blake Ross]

Yeah, I saw this too.  Weird.

German: N6 didn't have sidebar context menus, they're only in the trunk.

Comment 5

[Ninoschka Baca]

According to johng, no context menu should appear in the Sidebar.

Shrirang: I found the following so I'm wondering if this bug should be assigned
to you now since Navigator also has the problem.

My results:
Build 2000-11-28-09: Win95
- Navigator - No context menu appears
- Mail - No context menu appears

Build 2000-11-28-08: Linux, Mac
- Navigator - A context menu appears
- Mail - A context menu appears

Comment 6

[Blake Ross]

johng?  Why shouldn't the sidebar have a context menu?  Sidebar panels are 
webpages too :)

Comment 7

[shrirang khanzode]

yeah..why ?

Comment 8

[Blake Ross]

I think this happens because Mail/News has a different idea of window._content 
than Navigator.  Futuring this for now until <browser/> starts coming 
together...

Comment 9

[Jesse Ruderman]

See also bug 66683, text for "save image" is incorrect in mailnews sidebar 
context menus.

Comment 10

[Doron Rosenberg (IBM)]

<browser/> is long in...

Comment 11

[Blake Ross]

usability/polish 0.9.4.

Comment 12

[Blake Ross]

reassigning to default owner.

Comment 13

[Samir Gehani]

Sidebar triage team: over to Blake for a look (with context menu rework).

Comment 14

[[e-mail]]

This is caused because the sidebar normally uses the navigator context menu.
Unfortunately MailNews uses its own context menu which duplicates the sidebar
ids (in the case of the cookie context overlay the ids are triplicated!).
This means that the show/hide code affects the wrong context menu.
It might be possible to fix this using commands and commandupdaters.

Comment 15

[R.K.Aa.]

Is this still a valid issue?
Sidebar was perm removed from mailnews in trunk builds on Sept. 14th, 2002.

Comment 16

[Vedran Miletic]

retargeting

Comment 17

[timeless]

killing since it's not present anymore. it'll have to be fixed if it's ever
reintroduced.