29100 - Menus don't adjust to screen size changes

Status: RESOLVED WORKSFORME

(Core :: XUL, defect, P3)

Description

[Eli Goldberg]

* TITLE/SUMMARY
Pop-up menus don't adjust to screen size changes

* STEPS TO REPRODUCE
0) Set your screen resolution to one that can be increased later e.g. 800x600.
Launch Apprunner
1) Fill up a "Bookmarks" menu with lot items (just so that it's tall enough to
cover the height of your screen)
2) Click on it.
3) Increase your screen resolution significantly, e.g. to 1024x768
4) Click on the pop-up menu again, or your Bookmarks menu.

* RESULT
 - What happened

The bottom edge of the menu is cut off according to the screen resolution when
Mozilla was launched, not the current screen size.

 - What was expected

Menus should be drawn relative to the current screen size; users may not restart
their computer or applications when they make configuration changes as they did
when Win95 first came out.

* REGRESSION

 - Occurs On
        Win32 Apprunner (2.24.00 AM optimized build [NT 4, Service Pack 5])

 - Can't Tell
        Mac OS Apprunner (2.24.00 AM optimized build --- menus are native,
pop-up menu is too messed up to realistically tell)
        Linux --- Not checked; not obvious how to do dynamic resolution changes)

* CONFIGURATIONS TESTED

- [Mac] Beige Power Mac G3 (266 MHz PowerPC 750), 96 MB RAM (VM on; 1 MB of VM
used), 1024x768 (Thousands of Colors), Mac OS 8.6

- [Win32] Vectra VL (233 MHz P2), 96 MB RAM, 800x600 (True Color), NT 4.0 SP5.

- [Linux] Vectra VL (266 MHz P2), 96 MB RAM. Red Hat Linux 6.0 (GNOME).

Comment 1

[Eli Goldberg]

(It may be more obvious if you add a step 5, of adding more bookmarks to your
Bookmarks menu, and noting that they don't appear.)

Comment 2

[Mike Pinkerton (not reading bugmail)]

danm owns screen object....i suspect this would happen on macOS as well, but i 
don't know for sure.

Comment 3

[sairuh (rarely reading bugmail)]

eli, if this does *not* refer to Bookmarks off the toolbar, feel free to bounce
it back to me.

Comment 4

[Eli Goldberg]

Actually, it also involves the Bookmarks menu (from the menu bar). QA Assigning 
back to Sarah, per Claudius's okay.

Comment 5

[Eli Goldberg]

Corrected bug summary.

Comment 6

[Peter Trudelle]

Mass moving M17 bugs to M18

Comment 7

[Peter Trudelle]

mass-moving all bugs to m21 that are not dofood+, or nsbeta2+

Comment 8

[sairuh (rarely reading bugmail)]

*spam*: transferring current XP Menu bugs over to jrgm, the new component owner.
feel free to add me to the cc list (unless am the Reporter) of any of these, if
you have any questions/etc.

Comment 9

[Tuukka Tolvanen (sp3000)]

is this dup/dep bug 62395?

Comment 10

[Wayne Mery (:wsmwk)]

Tuukka, do you think WFM?

I am unsure comment 0 is a sufficent test, but this WFM Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9b4pre) Gecko/2008021504 

I can't speak to w98

Comment 11

[David Fraser]

I get this on Linux (Ubuntu 8.04) when adding an extra monitor using xrandr.
As to bug 62395, in this case window.screen still shows the resolution of the original screen, not the combined resolution.
Changing OS to All as a result.
There's an upstream Ubuntu bug at https://bugs.launchpad.net/ubuntu/+source/firefox-3.0/+bug/229298

See also bug 239043 which had some comments relating to this.

Comment 12

[Chris Bainbridge]

I ran in to bug #61000 which is possibly a dupe of this one. With the latest firefox sources from Mercurial (apparently 3.2a1pre) I no longer see either this bug (bookmark window size now adapts to change in resolution), or bug #61000 (popup windows now appears on correct monitor). 

It would be useful if someone else could check to make sure 3.2a1pre does indeed fix this issue. Hey, almost 9 years for a fix...

Comment 13

[Bourdieu]

I can confirm that the problem is fixed with firefox 3.1 beta 2 on linux.

Comment 14

[Wayne Mery (:wsmwk)]

WFM based on comment 12 and comment 13