STEPS TO REPRODUCE: 1) Apply the patch from https://bugzilla.mozilla.org/attachment.cgi?id=251032 2) Try to select an item from a submenu (either the main menus or the context menu; simple thing is to try to do "View frame source" on the attachment in https://bugzilla.mozilla.org/attachment.cgi?id=251032&action=edit). ACTUAL RESULTS: No response from submenu for hover or click. EXPECTED RESULTS: Submenus work.
Oh yeah. Menus just position their views and don't update their frame coordinates. Bah. Something to fix when I remove views. Anyway, as I said in that other bug, we're not ready for that patch yet anyway because if view geometry is wrong, other things will get messed up.
Yeah. I'm just making sure we have tracking bugs on the issues that need to be resolved to remove views... I should note that there _is_ the code at http://bonsai.mozilla.org/cvsblame.cgi?file=mozilla/layout/xul/base/src/nsMenuPopupFrame.cpp&rev=1.297&mark=1012-1017#1010 that looks like it's trying to deal with this. I assume it fails because GetOriginToViewOffset() is a bloody mess?
11 years ago
Is this still a problem?