Closed Bug 1257087 Opened 9 years ago Closed 9 years ago

Middle mouse click on history item would not open

Categories

(Firefox :: Bookmarks & History, defect, P2)

48 Branch
Unspecified
Windows 7
defect

Tracking

()

VERIFIED FIXED
Firefox 49
Tracking Status
firefox46 --- unaffected
firefox47 + verified
firefox48 + verified
firefox49 --- verified

People

(Reporter: alice0775, Assigned: dao)

References

Details

(Keywords: regression)

Attachments

(2 files)

[Tracking Requested - why for this release]: regression Reproducible: always Steps to reproduce: 1. Open hamburger menu 2. Click History icon 3. Middle mouse click on history item Actual results: Nothing happens Expected results: The history item should be opened in new tab
Flags: needinfo?(mak77)
Flags: needinfo?(dao)
well, we also need middle click working for recent bookmarks, so let's use this for both.
Flags: needinfo?(mak77)
Assignee: nobody → dao+bmo
Flags: needinfo?(dao+bmo)
Attachment #8756263 - Flags: review?(mak77) → review+
Attached patch patchSplinter Review
I believe bug 1248267 fixed this for recent bookmarks.
Attachment #8756263 - Flags: review?(mak77)
Blocks: 1275793
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 49
Attached patch patch for upliftSplinter Review
Approval Request Comment [Feature/regressing bug #]: bug 1249473 [User impact if declined]: see comment 0 (only history widget items are affected, the recent bookmarks list isn't on beta / aurora) [Describe test coverage new/current, TreeHerder]: no coverage [Risks and why]: trivial fix, low risk [String/UUID change made/needed]: none
Attachment #8757254 - Flags: approval-mozilla-beta?
Attachment #8757254 - Flags: approval-mozilla-aurora?
Comment on attachment 8757254 [details] [diff] [review] patch for uplift Recent regression, taking it
Attachment #8757254 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
This is marked as a regression in 47; can we uplift to beta?
Flags: needinfo?(rkothari)
(In reply to Andrew Overholt [:overholt] from comment #12) > This is marked as a regression in 47; can we uplift to beta? Yes, this seems like a low-risk fix and recent regression. I will take it in RC1.
Flags: needinfo?(rkothari)
Comment on attachment 8757254 [details] [diff] [review] patch for uplift This is a recent and very visible end-user regression, let's uplift to Beta47
Attachment #8757254 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
Flags: qe-verify+
Reproduced on Nightly 2016-03-23. Verified fixed FX 47 RC build 3, 48.0a2 (2016-06-05), 49.0a1 (2016-06-05) Win 7
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: