Mail/News: Right-click of Move To in context menu leaves sub-menu stranded

VERIFIED FIXED in mozilla0.9

Status

()

VERIFIED FIXED
18 years ago
10 years ago

People

(Reporter: dean_tessman, Assigned: mikepinkerton)

Tracking

Trunk
mozilla0.9
x86
Windows 2000
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

18 years ago
Found in testing for bug 45108...

1. Right-click a message in Mail/News.
2. Select Move To
3. Allow the sub-menu to open
4. Right-click "Move To"

Expected Results: Nothing happens

Actual Results: The top-level context menu dismisses and the sub-menu remains.  
I wonder if this is specific to the mail/news context menu, or if it happens for 
all multi-level context menus?  My guess is the latter.

Build: 2001031204 on Win2000 sp1

Can someone check this out on Mac/Linux to see if it's xp?

Note: This is _not_ related to the patch I was testing in bug 45108.  The build 
I was using was fresh from mozilla.org.
(Assignee)

Comment 1

18 years ago
i've noticed problems with selecting items from submenus of context menus that 
pose a dialog dismisses only one of the context menus not the whole chain (bug 
71219). might be related.
(Assignee)

Updated

18 years ago
Status: NEW → ASSIGNED
Target Milestone: --- → mozilla0.9

Comment 2

18 years ago
I'd dup this to bug 71219 (or vice versa) if I were you!
Bug 7219 describes the same problem with Composer context menu and its
Table submenus.
(Reporter)

Comment 3

18 years ago
No, don't dupe it.  The bugs as they currently are describe different enough
circumstances that I suspect they're not the exact same issue.

This is a general multi-level context menu bug.  I just duplicated this with the
Table context menu in composer.
(Reporter)

Comment 4

18 years ago
Mike, do you have another example of the sub-menu opening a dialog problem?  I'd 
like to have another place to test this, other than just the insert table 
context menu mentioned in bug 71219.
(Assignee)

Comment 5

18 years ago
that's the only place i know of, i think. why aren't we discussing this in that 
bug? ;) 
(Assignee)

Comment 6

18 years ago
i fixed this with my context menu changes last week. probably was the
"contextmenu" handler on the popup.

fixed.
Status: ASSIGNED → RESOLVED
Last Resolved: 18 years ago
Resolution: --- → FIXED
(Assignee)

Comment 7

18 years ago
never mind, this is win32 only.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
(Assignee)

Comment 8

18 years ago
notes to self:
- we're not rolling up the context menu
- DealWithPopups correctly finds the click in the parent and does nothing
- someone else is causing this window to hide but i can't figure out where
that's happening
Status: REOPENED → ASSIGNED
(Assignee)

Comment 9

18 years ago
here's the fix:

Index: mozilla/layout/xul/base/src/nsMenuFrame.cpp
===================================================================
RCS file: /m/pub/mozilla/layout/xul/base/src/nsMenuFrame.cpp,v
retrieving revision 1.172
diff -c -2 -r1.172 nsMenuFrame.cpp
*** nsMenuFrame.cpp	2001/03/29 03:32:34	1.172
--- nsMenuFrame.cpp	2001/04/03 18:32:27
***************
*** 359,363 ****
              aEvent->message == NS_CONTEXTMENU &&
  #endif
!             mMenuParent && !IsDisabled()) {
      // if this menu is a context menu it accepts right-clicks...fire away!
      // Make sure we cancel default processing of the context menu event so
--- 359,363 ----
              aEvent->message == NS_CONTEXTMENU &&
  #endif
!             mMenuParent && !IsMenu() && !IsDisabled()) {
      // if this menu is a context menu it accepts right-clicks...fire away!
      // Make sure we cancel default processing of the context menu event so

can someone r/sr?
(Assignee)

Comment 10

18 years ago
fixed.
Status: ASSIGNED → RESOLVED
Last Resolved: 18 years ago18 years ago
Resolution: --- → FIXED
(Reporter)

Comment 11

18 years ago
Whoops, my bad.  Thanks for catching that, Mike.  Verified using 2001040520

Also, bug 71219 is not related to this.  That behavior persists.
Status: RESOLVED → VERIFIED

Updated

10 years ago
Component: XP Toolkit/Widgets: Menus → XUL
QA Contact: jrgmorrison → xptoolkit.widgets
You need to log in before you can comment on or make changes to this bug.