Closed Bug 48838 Opened 24 years ago Closed 24 years ago

Context menus need to first set focus to the widget you right-click on

Categories

(Core :: XUL, defect, P4)

Product:
Core
Component:
XUL
Platform:
x86
Windows 98
Type:
defect

Tracking

()

Status:
VERIFIED FIXED

People

(Reporter: bugzilla, Assigned: saari)

Details

(Keywords: dataloss, Whiteboard: [nsbeta3+]) 

Standard behavior on windows is that when you right click on a widget, focus is 
first set to that widget, and then the items on the context menu that appears 
all affect that widget.  The current menu system is flawed, and doesn't work 
this way.  Instead, if an element has the focus and you click elsewhere, the 
context menu that appears still has items that affect the element that has the 
focus.  This is incorrect, and has repercussions in many areas of the product: 
selection and editor to name a few.

Try this: go to a website with a textbox, type a few characters into the 
textbox, set focus to the URL bar, then right click on the textbox in the 
site.  Then click Select All in the context menu that appears.  Note that while 
Select All actually selects the contents of the URL bar, other items on the 
same context menu (like Bookmark this Page and Save Page As...) actually apply 
to the website itself.

There are other examples like this throughout the product, and I'm willing to 
give more if need be.  I don't think this is a quick FUTURE because it's a 
serious flaw in our event handling (er, should this be under event handling?) 
and is inconsistent with Windows behavior (and possible that of other 
platforms...)
Nominating for beta 3. This is bad.
Keywords: correctness, dataloss, nsbeta3
reassigning to saari, who agreed to take a look. says context clicks just need 
to fire a focus event first, which should be easy...
Assignee: pinkerton → saari
nsbeta3+ to take (partial) in-hand fix. p4 for M18
Priority: P3 → P4
Whiteboard: [nsbeta3+]
Target Milestone: --- → M18
*** Bug 40796 has been marked as a duplicate of this bug. ***
Status: NEW → ASSIGNED
Please note that we are not committing to fix P4/P5 bugs for N6, although we 
could still work on a fix in priority order, or take a patch.
Fixed
marking fixed :-P
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
verified on windows build 2000091208m18,
Status: RESOLVED → VERIFIED
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.