108595 - Context Menu Items Missing/Showing Up

Status: VERIFIED FIXED

(SeaMonkey :: UI Design, defect)

Description

[Ben Goodger (use ben at mozilla dot org for email)]

Send Link/Send Page are appearing for all items, link or page or not...
"Block images from this server" is not showing up
Wallet items are being inserted between "Copy" and "Copy Link Address" rather 
than after them.

Comment 1

[Ben Goodger (use ben at mozilla dot org for email)]

Attachment: patch

Comment 2

[Ben Goodger (use ben at mozilla dot org for email)]

This patch: 
- removes an extra load of contentAreaContextOverlay.xul from navigator.xul... 
sidebarOverlay.xul was loading it, causing it to be loaded twice into navigator 
and causing problems for cookie. 
- remove hacks from cookieContextOverlay that compensated for the former bug 
but were also causing the block image items not to show up since the way 
context menus are hidden has changed
- change the insert position of wallet items to be after copy link address
- hack the addition of a load listener in mailNavigatorOverlay to ensure the 
mail items are hidden/

Comment 3

[Blake Ross]

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

sr=blake

Comment 4

[Ben Goodger (use ben at mozilla dot org for email)]

kerz sez 'r=kerz'

Comment 5

[Ben Goodger (use ben at mozilla dot org for email)]

Fix checked in. 

Comment 6

[sairuh (rarely reading bugmail)]

vrfy fixed using these commercial builds:

linux rh6.2 - 2001.11.06.12
winnt - 2001.11.06.12
mac os 10.1 - 2001.11.06.12