Closed Bug 343801 Opened 18 years ago Closed 18 years ago

History menu jumps around (disappears) since addition of new Recently Closed Tabs menu item

Categories

(Firefox :: Menus, defect, P1)

2.0 Branch
defect

Tracking

()

VERIFIED FIXED
Firefox 2 beta1

People

(Reporter: ispiked, Assigned: asaf)

References

Details

(Keywords: regression, verified1.8.1)

Attachments

(4 files)

Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.8.1a3) Gecko/20060706 BonEcho/2.0a3 Steps to reproduce: 1. Open at least two tabs and close one of them. 2. Go to History and move your mouse down till you reach the "Recently Closed Tabs" menu item. 3. Move your mouse back up over the listed History items. Results: The list of History items disappears and the menu "jumps" around. Definitely a regression from bug 342700.
Assignee: dietrich → nobody
Component: Tabbed Browser → Menus
QA Contact: tabbed.browser → menus
Assignee: nobody → dietrich
Just a note that this occurs on WinXP also, but *not* on Mac.
Status: NEW → ASSIGNED
*** Bug 343949 has been marked as a duplicate of this bug. ***
I see the same thing in Mozilla/5.0 (rv:1.8.1a3) Gecko/20060707 BonEcho/2.0a3 on XP and 2003. I described it as follows in bug 343949. The History menu shrinks and loses the 10 item recent history list, after the user touches and expands the Recently Closed Tabs option. If the user quickly moves over the Recently Closed Tabs option (without it expanding), then no shrinking occurs. (The shrinking is caused by the loss of the 10 item recent history list. Navigating to another menu and then back to the History menu causes the 10 item list to be repopulated.) The menu shouldn't shrink and disappear out from under the mouse cursor. Reproducible: Always Steps to Reproduce: 1. Click on the History menu 2. Click on the Recently Closed Tabs entry to expand it 3. Put the cursor on another History menu entry (like 'Show in Sidebar') and see it shrink in < 1 second right before your eyes. Expected Results: menu shouldn't shrink and disappear out from under the mouse cursor
Attachment #228515 - Attachment description: viewing Recently Closed tabs menu entry → 1 - viewing Recently Closed tabs menu entry
Assignee: dietrich → bugs.mano
Severity: normal → major
Status: ASSIGNED → NEW
Flags: blocking-firefox2?
OS: Linux → All
Priority: -- → P1
Hardware: PC → All
Target Milestone: --- → Firefox 2 beta2
Status: NEW → ASSIGNED
Attached patch patch (deleted) — Splinter Review
Attachment #228589 - Flags: review?(gavin.sharp)
Attachment #228589 - Flags: review?(gavin.sharp) → review+
Attachment #228589 - Flags: approval1.8.1?
Target Milestone: Firefox 2 beta2 → Firefox 2 beta1
mozilla/browser/base/content/browser.js 1.658 mozilla/browser/base/content/browser-menubar.inc 1.97 Will get post-facto review on this once mconnor is back.
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
Attachment #228589 - Flags: superreview?(mconnor)
Tested patch on Mac. Definitely blocks Firefox 2, and based on others' assertions of safety (dietrich?) we should probably take this for today's respins. Mano: mconnor's not going to be back until tonight; can you think of another appropriate sr? ben? vlad? bryner?
Flags: blocking-firefox2? → blocking-firefox2+
(In reply to comment #9) As i noted on IRC, I'm confident enough in this patch (and gavin's review) to only look for a post-facto peer review on it. This cannot bake on trunk as the bug doesn't affect places-enabled builds.
(In reply to comment #9) > Tested patch on Mac. Did you see the bug on Mac? I was not able to reproduce it on Mac, only Windows (didn't try Linux). I tested the patch on Windows; it works, and seems safe, as those functions don't have any other callers (that I could find).
Attachment #228589 - Flags: approval1.8.1? → approval1.8.1+
I asked beltzner to test it on Mac to make sure it does not regress the go menu behavior (onpopupshowing/onpopuphiding are dispatched differently on mac).
1.8 branch: mozilla/browser/base/content/browser-menubar.inc 1.57.2.37 mozilla/browser/base/content/browser.js 1.479.2.163
Keywords: fixed1.8.1
Comment on attachment 228589 [details] [diff] [review] patch (clearing sr)
Attachment #228589 - Flags: superreview?(mconnor)
Verified FIXED using Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.8.1b1) Gecko/20060710 Firefox/2.0b1.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: