Closed
Bug 1498839
Opened 6 years ago
Closed 6 years ago
copyUrlPopup context menu not shown for link in RSS feed message header (normal copyPopup context menu shown)
Categories
(Thunderbird :: Mail Window Front End, defect)
Thunderbird
Mail Window Front End
Tracking
(Not tracked)
RESOLVED
FIXED
Thunderbird 64.0
People
(Reporter: joachim.herb, Assigned: mkmelin)
References
Details
(Keywords: regression)
Attachments
(1 file)
|
(deleted),
patch
|
arshad
:
review+
|
Details | Diff | Splinter Review |
In the current turn (64.0) the special context menu for URL is not working anymore in the header of RSS feed messages. Instead the "normal" context menu for copying, e.g. for copying the subject, is shown. In 63.0 (beta) it still works as before.
Steps to reproduce:
Select a message of an RSS feed.
Right click on the article URL in the header.
Normal context menu for copying is shown instead of the special copyUrlPopup
defined at https://dxr.mozilla.org/comm-central/source/comm/mail/base/content/msgHdrPopup.inc#130
|
Reporter | |
Updated•6 years ago
|
Status: NEW → UNCONFIRMED
Ever confirmed: false
|
||
Comment 1•6 years ago
|
||
Looks like a regression from the de-XBL work.
So correct would be to show: Copy link location, and not the "normal" context menu.
Flags: needinfo?(mkmelin+mozilla)
Flags: needinfo?(arshdkhn1)
|
Assignee | |
Updated•6 years ago
|
Assignee: nobody → mkmelin+mozilla
Blocks: 1489748
Flags: needinfo?(mkmelin+mozilla)
Keywords: regression
|
|
Assignee | |
Updated•6 years ago
|
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Flags: needinfo?(arshdkhn1)
|
|
Assignee | |
Comment 2•6 years ago
|
||
Attachment #9017031 -
Flags: review?(arshdkhn1)
|
||
Comment 3•6 years ago
|
||
Comment on attachment 9017031 [details] [diff] [review]
bug1498839_mailurlfield_context.patch
Review of attachment 9017031 [details] [diff] [review]:
-----------------------------------------------------------------
https://searchfox.org/comm-central/source/mail/base/content/mailWidgets.js#10 , why adding again?
|
|
Assignee | |
Comment 4•6 years ago
|
||
I'm not. Different context.
|
|
||
Updated•6 years ago
|
Attachment #9017031 -
Flags: review?(arshdkhn1) → review+
|
|
Assignee | |
Updated•6 years ago
|
Keywords: checkin-needed
Pushed by mozilla@jorgk.com:
https://hg.mozilla.org/comm-central/rev/8539fb6b4605
fix context menu for link in RSS feed message header (normal copyPopup context menu instead of copyUrlPopup). r=arshad
|
|
||
Updated•6 years ago
|
Target Milestone: --- → Thunderbird 64.0
You need to log in
before you can comment on or make changes to this bug.
Description
•