Closed
Bug 1556839
Opened 5 years ago
Closed 5 years ago
Searchbox in Addressbook has wrong context menu
Categories
(Thunderbird :: Address Book, defect)
Thunderbird
Address Book
Tracking
(thunderbird68+ fixed, thunderbird69 fixed)
RESOLVED
FIXED
Thunderbird 69.0
People
(Reporter: Paenglab, Assigned: mkmelin)
References
Details
(Keywords: regression)
Attachments
(1 file)
1.32 KB,
patch
|
jorgk-bmo
:
review+
jorgk-bmo
:
approval-comm-beta+
|
Details | Diff | Splinter Review |
The Addressbook searchbox shows the toolbar context menu instead of the edit menu like all other boxes show.
Comment 1•5 years ago
|
||
Also broken in TB 68 beta. Most likely a de-XBL issue. Magnus, can you please get a de-XBL person to have a look.
tracking-thunderbird68:
--- → +
Flags: needinfo?(mkmelin+mozilla)
Assignee | ||
Comment 2•5 years ago
|
||
I think this is the c-c fix, and a e.preventDefault(); should be added at https://searchfox.org/comm-central/rev/4a11fb532256bbd0901758d25ec6a79fb2d87afd/mozilla/toolkit/content/editMenuOverlay.js#94 (or both context menus show up). I'll file a bug for m-c.
Assignee: nobody → mkmelin+mozilla
Status: NEW → ASSIGNED
Flags: needinfo?(mkmelin+mozilla)
Attachment #9070217 -
Flags: review?(jorgk)
Comment 3•5 years ago
|
||
Comment on attachment 9070217 [details] [diff] [review] bug1556839_absearch_context.patch Richard suggested this (via a PM) for bug 1556582, but he wanted to avoid a double menu. Maybe we should wait until bug 1557286 is fixed.
Attachment #9070217 -
Flags: review?(jorgk) → review+
Updated•5 years ago
|
Attachment #9070217 -
Flags: approval-comm-beta+
Pushed by mozilla@jorgk.com:
https://hg.mozilla.org/comm-central/rev/62dc416dcabf
Restore context menu in searchbox in address book that got lost in bug 1532595. r=jorgk DONTBUILD
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Updated•5 years ago
|
Target Milestone: --- → Thunderbird 69.0
Comment 5•5 years ago
•
|
||
Updated•5 years ago
|
status-thunderbird68:
--- → fixed
status-thunderbird69:
--- → fixed
You need to log in
before you can comment on or make changes to this bug.
Description
•