135103 - Context Menus: Missing separator in image menu

Status: VERIFIED FIXED

(SeaMonkey :: UI Design, defect)

Description

[Hixie (not reading bugmail)]

STEPS TO REPRODUCE
   1. http://www.damowmow.com/
   2. Right click on astrophy (the cat).

ACTUAL RESULTS
   Save and Send Image are adjacent to the three page-related items.   

EXPECTED RESULTS
   They should be separated.

Using a Linux build from CVS tip with the menu changes on Mandrake 8.1.

Comment 1

[Paul Wyskoczka]

qa --> [e-mail]

Comment 2

[:deanis74]

I don't see this on Windows.  There's a Set As Wallpaper item below Send Image,
and then a separator.  Is the separator tied to Set As Wallpaper instead of
Bookmark This Page?

Comment 3

[Blake Ross]

Attachment: patch

Comment 4

[:deanis74]

r=me

Comment 5

[Jonathan Abbey]

I see this also, on Linux 2002040314.

However, that's not the main thing wrong with the context menu on images in this
build.. right-clicking on an image no longer includes the 'back' and 'forward'
menu items anymore.  This means that I can't just casually right-click anywhere
on the page and make a slight flick of the wrist to go back or forward, but
would have to move all the way up to the top of the window to the buttons, or to
carefully mouse around until I find an area of the window not displaying an img tag.

I don't see any bug reported for this in today's bugzilla.. was this misfeature
on purpose?

Comment 6

[:deanis74]

Jonathan: That's bug 135331

Comment 7

[Blake Ross]

fixed.

Comment 8

[sairuh (rarely reading bugmail)]

vrfy'd fixed on all platforms, 2002.09.19.08 comm trunk.