284782 - Context menu sometimes partially offscreen

Status: VERIFIED DUPLICATE of bug 245163

(Firefox :: Menus, defect)

Description

[Geoff Kuenning]

User-Agent:       Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.7.6) Gecko/20050225 Firefox/1.0.1
Build Identifier: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.7.6) Gecko/20050225 Firefox/1.0.1

I have a fair number of extensions installed, many of which add items to the
context menu, especially the context menu for images.  In some circumstances,
the context menu pops up partly offscreen, even though it's small enough to fit
on the screen.  In all the cases I've been able to reproduce, the menu is off
the top.

My theory, not based on looking at the source code, is that the popup code uses
an algorithm like this:

    if (mouse.y + menu.height <= display.height)
        pop menu up below mouse
    else
        pop menu up above mouse

If the menu height is greater than half the display height, this will produce
an offscreen menu for mouse positions near the screen.
 

Reproducible: Always

Steps to Reproduce:
1.Install sufficient extensions to make the context menu larger than half the
screen height.  It "helps" to use extensions that relate to images, and a small
screen (I'm on a 1024x768 laptop).  Relevant extensions that I have installed
are AniDisable, ColorZilla, Image Zoom, Linkification, MapIt!, Nuke Anything,
TinyURL Creator, ViewSourceWith, and WayBack.
2.Visit any Web page that has lots of smallish images, such as the one
referenced above.
3.Size the browser window to less than the display size (I'm not sure if this is
necessary) and scroll the Web page so that a chosen image is near the middle y
position.
4.Right-click on the image to bring up the context menu.

Actual Results:  
Context menu is off the screen.

Expected Results:  
If context menu fits on screen, it should be positioned entirely on-screen.  If
it is too big for the screen, it should include scrolling capability like the
bookmarks menu (the latter may already be implemented; my context menu isn't
_that_ big yet!).


I'm using the default theme.  My window manager is sawfish 2.0, in case the WM
is relevant.

Comment 1

[Simon Paquet [:sipaq]]

*** This bug has been marked as a duplicate of 245163 ***