Closed Bug 26509 Opened 25 years ago Closed 24 years ago

Context menus don't grab the mouse on linux

Categories

(Core :: XUL, defect, P4)

x86
Linux
defect

Tracking

()

VERIFIED FIXED

People

(Reporter: rzach, Assigned: blizzard)

Details

(Keywords: platform-parity, Whiteboard: [nsbeta3+])

Attachments

(2 files)

I didn't want to resurrect bug 11266, so I'm filing a new bug on a related issue. When the focus switches to another window, xpmenus (menus from menu bar, drop-downs from select fields, etc) stay on top until you click in the new window. It is even possible to even change the selection in the "disembodied" menu as long as the mouse doesn't leave it. To reproduce: 1. Open menu, eg by clicking on Bookmarks menu tab 2. Position cursor in menu 3. Alt-Tab to another window Actual result: the menu stays in front even though the browser window is now obscured by another app. You can change the selection by moving the mouse up and down. The menu will stay on top even if you move the cursor outside the menu. It will close only if you click in the new window. Expected result: the menu should close/drop-downs collapse as soon as you click Alt-Tab. Linux build 2000.02.02.09
the actual expected result would be for nothing to happen. There is a bug right now causing the menus to not always grab the mouse. I think I know how to fix this. normally this does work though.
Yes, most recently having a window or a select pull-down open and clicking Alt-TAB does nothing. I'd rather have it collapse the menu/pulldown and then switch, but Gnome apps and NS 4.7 don't, so I'll resolve this WFM.
Status: NEW → RESOLVED
Closed: 25 years ago
Resolution: --- → WORKSFORME
Verif.
Status: RESOLVED → VERIFIED
After reading bug 28319: this still happens with context menus. Reopen, change summary.
Status: VERIFIED → REOPENED
Resolution: WORKSFORME → ---
Summary: Menus stay on top after parent window loses focus → Context menus stay on top after parent window loses focus
changing summary, accepting
Status: REOPENED → ASSIGNED
Summary: Context menus stay on top after parent window loses focus → Context menus don't grab the mouse on linux
Target Milestone: M15
bill, pink -- the only reason this would happen is if the mouse is trying to be grabbed before the popup window is visible... are we doing any tricks here with changing visibilty of the popup?
Target Milestone: M15 → M17
I dunno. From my perspective, it's just xul magic (we define a context menu and connect it to the content area in xul; the xptoolkit/widget code handles detecting context menu events, opening the popup, and dismissing it when necessary). pink/saari/hyatt would know more.
Mass moving M17 bugs to M18
Target Milestone: M17 → M18
mass-moving all bugs to m21 that are not dofood+, or nsbeta2+
Target Milestone: M18 → M21
*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.
QA Contact: sairuh → jrgm
Keywords: 4xp, nsbeta3, pp
this is really annoying and needs to be fixed.
nsbeta3+ because this is driving pavlov insane, and we can't have that. Lower priority, fix other bugs first.
Priority: P3 → P4
Whiteboard: [nsbeta3+]
I have something that seems to fix this, but i'm not sure I like it.. i'm getting blizzard to take a look at it to get his input.
I didn't like it either. I'm uploading another patch in a bit.
Attached patch patch me harder (deleted) — Splinter Review
Uhh...looking for review/comment on this. Hello, anyone there?
Attached patch patch 2 (deleted) — Splinter Review
New patch that fixes some of the midleading variable naming and also is less agressive about trying to grab the window on an nsIWindow::Show()
I have an r=pavlov on this.
Keywords: approval, patch
reassigning
Assignee: pavlov → blizzard
Status: ASSIGNED → NEW
I have an a=waterson
Status: NEW → ASSIGNED
Keywords: approval
Checked in.
Status: ASSIGNED → RESOLVED
Closed: 25 years ago24 years ago
Resolution: --- → FIXED
verified fixed (2000091308m18 linux)
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.

Attachment

General

Created:
Updated:
Size: