Closed
Bug 253757
Opened 20 years ago
Closed 20 years ago
Sub-menus on context menu don't open automatically
Categories
(Firefox :: Menus, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: bugzilla, Assigned: vlad)
Details
(Keywords: fixed-aviary1.0, regression, Whiteboard: [have patch])
Attachments
(1 file)
1.73 KB,
patch
|
bugs
:
approval-aviary+
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.7) Gecko/20040730 Firefox/0.9.1+ Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.7) Gecko/20040730 Firefox/0.9.1+ Several extensions add sub-menus to the context menu. In previous builds, the sub-menu would pop open on mouseover. On 20040730, a left-click is now required to open the extenson's sub-menu. Reproducible: Always Steps to Reproduce: 1. Install an extension that uses a context menu (for example, MakeLink http://www.netsoc.ucd.ie/~finuvir/makelink_1.2.xpi ) and restart Firefox. 2. Right-click anywhere on the page and mouse over the context menu item "Make Link >" Actual Results: The context menu did not open, and a left-click was required to open it. Expected Results: The context menu should open automatically on mouseover. I saw this bug with three of the extensions I use (Make Link, Linky, and BBCode). Another user in the forums verified the bug with two other extensions (ReloadEvery and ImageZoom). Looking through yesterday's checkins, I thought it might be associated with either Bug 243720 [bookmark can't move into sub-folder in personal toolbar (drag and drop)] or Bug 203183 [Bookmarks context menu stays when clicking other menus].
Comment 1•20 years ago
|
||
confirmed for latest FF branch build (Gecko/20040730 Firefox/0.9.1+)
Comment 2•20 years ago
|
||
This broke recently. Blake, can you look at this?
Severity: minor → major
Status: UNCONFIRMED → NEW
Ever confirmed: true
Flags: blocking-aviary1.0PR+
Keywords: regression
Comment 3•20 years ago
|
||
It's not just extensions. Firefox has sub-menus on the context menu for frames and iframes.
Comment 4•20 years ago
|
||
Vlad, could this be your changes at bug 203183 *** This bug has been marked as a duplicate of 203183 ***
Status: NEW → RESOLVED
Closed: 20 years ago
Resolution: --- → DUPLICATE
Updated•20 years ago
|
Status: RESOLVED → REOPENED
Resolution: DUPLICATE → ---
Comment 5•20 years ago
|
||
Oops, need more coffee. Meant to reassign, not dupe.
Assignee: firefox → vladimir
Status: REOPENED → NEW
Assignee | ||
Comment 6•20 years ago
|
||
Uggh, I know why this is happening -- the menu open timer is just checking whether any popup is open and refusing to open, it's not checking whether it's about to open a submenu of the popup itself. Will fix ASAP.
Status: NEW → ASSIGNED
Assignee | ||
Comment 7•20 years ago
|
||
patch; just does the check (also removes unnecessary addref).
Assignee | ||
Updated•20 years ago
|
Attachment #154877 -
Flags: approval-aviary?
Updated•20 years ago
|
Whiteboard: [have patch]
Comment 8•20 years ago
|
||
Comment on attachment 154877 [details] [diff] [review] context-menu-popup-fix-0.patch a=ben@mozilla.org
Attachment #154877 -
Flags: approval-aviary? → approval-aviary+
Assignee | ||
Comment 9•20 years ago
|
||
in, fixed.
Status: ASSIGNED → RESOLVED
Closed: 20 years ago → 20 years ago
Resolution: --- → FIXED
Updated•20 years ago
|
Keywords: fixed-aviary1.0
Updated•18 years ago
|
QA Contact: bugzilla → menus
You need to log in
before you can comment on or make changes to this bug.
Description
•