23191 - linux/win: large hierarchical menus: child obsures parent menu

Status: VERIFIED FIXED

(Core :: XUL, defect, P4)

Description

[sairuh (rarely reading bugmail)]

noticed that that the submenus draw right on top of the parent menu, obscuring
much of the parent menu. in 4.x, such submenus would be shifted either to the
left or right of the parent menu (depending on where the browser win is located
on the screen).

here's a sample recipe of what i'm seeing:

1. move the
mozilla browser window so that it's flush with the right side of the screen.
(eg, have it opened at www.mozilla.org)

2. go to Bookmarks (from main menubar)
> Sample SmartFind queries: you'll notice that this submenu obscures 5 lines
worth of the parent menu (only the beginning of the parent's items are visible).


3. for an even more extreme example, check out the following subemenu:
Bookmarks> Sample SmartFind queries> Find: Bookmark names starting with 'A' >
AAA URLs with no redirection. in this case there are no borders btwn the
submenus (difficult to determine which items are in which menu) and the submenus
often flicker...

occurs on both windows and linux (2000010508 comm.). not an issue for mac since
native menubar is used.

Comment 1

[saari (gone)]

Anyone could take this bug, all the positioning code is in nsMenuPopupFrame::SyncViewWithFrame

Comment 2

[saari (gone)]

Peter, can we give this to someone less doomed?

Comment 3

[Peter Trudelle]

sure, removing help wanted from status whiteboard & reassigning to sdagley as p4
for m16. BTW, this should not be 'major' severity, which is supposed to indicate
major loss of function. This should be either minor or trivial.

Comment 4

[Dan Erikson]

Is this a dup of #21155?

Comment 5

[leger]

BULK MOVE: Changing component from XP Menus to XP Toolkit/Widgets: Menus.  XP 
Menus component will be deleted.

Comment 6

[Steve Dagley]

Passing the menu bug torch to pinkerton

Comment 7

[leger]

Setting the keyword all open [4.xp] bugs to 4xp.

Comment 8

[Mike Pinkerton (not reading bugmail)]

bulk accept

Comment 9

[Mike Pinkerton (not reading bugmail)]

fixed.

Comment 10

[sairuh (rarely reading bugmail)]

oh yeah much bettah. verif on winNT and linux, opt comm bits 2000.03.03.08.

Comment 11

[Pulsebot]

Pushed by ffxbld-merge:
https://hg.mozilla.org/releases/mozilla-beta/rev/87ce1383c2b1
[fenix] For https://github.com/mozilla-mobile/fenix/issues/23191 - Replace @color/preference_section_header_normal_theme with @color/fx_mobile_text_color_accent