Closed
Bug 1601573
Opened 5 years ago
Closed 5 years ago
Context menu for chat-contact-richlistitem is not shown
Categories
(Thunderbird :: Instant Messaging, defect)
Thunderbird
Instant Messaging
Tracking
(thunderbird72 fixed, thunderbird73 fixed)
RESOLVED
FIXED
Thunderbird 73.0
People
(Reporter: khushil324, Assigned: khushil324)
Details
Attachments
(1 file, 1 obsolete file)
1.20 KB,
patch
|
khushil324
:
review+
mkmelin
:
approval-comm-beta+
|
Details | Diff | Splinter Review |
No description provided.
Assignee | ||
Updated•5 years ago
|
Assignee: nobody → khushil324
Assignee | ||
Comment 1•5 years ago
|
||
Attachment #9113734 -
Flags: review?(mkmelin+mozilla)
Assignee | ||
Updated•5 years ago
|
Status: NEW → ASSIGNED
Comment 2•5 years ago
|
||
Comment on attachment 9113734 [details] [diff] [review]
Bug-1601573_context-menu-chat-contact-richlistitem-fixed.patch
Review of attachment 9113734 [details] [diff] [review]:
-----------------------------------------------------------------
Looks good. Please set up the proper commit message
Attachment #9113734 -
Flags: review?(mkmelin+mozilla) → review+
Comment 3•5 years ago
|
||
Do we know what caused this regression? (Does this need to be uplifted?)
Assignee | ||
Comment 4•5 years ago
|
||
Attachment #9113734 -
Attachment is obsolete: true
Attachment #9113769 -
Flags: review+
Assignee | ||
Comment 5•5 years ago
|
||
(In reply to Patrick Cloke [:clokep] from comment #3)
Do we know what caused this regression? (Does this need to be uplifted?)
I couldn't find any regression as aXulMenu.triggerNode was working previously. Some changes from MC must have caused this.
Looking at the code, I think we need it for beta uplift.
Assignee | ||
Updated•5 years ago
|
Keywords: checkin-needed-tb
Pushed by mkmelin@iki.fi:
https://hg.mozilla.org/comm-central/rev/73aab51bd41d
Resolved context menu for chat-contact and chat-imconv richlistitems. r=mkmelin
Updated•5 years ago
|
Target Milestone: --- → Thunderbird 72.0
Updated•5 years ago
|
Attachment #9113769 -
Flags: approval-comm-beta+
Comment 7•5 years ago
|
||
Looks like you set the wrong Target Milestone. Maybe that's the reason the beta uplift was missed.
Target Milestone: Thunderbird 72.0 → Thunderbird 73.0
Comment 8•5 years ago
|
||
Target Milestone: Thunderbird 73.0 → Thunderbird 72.0
Updated•5 years ago
|
status-thunderbird72:
--- → fixed
Target Milestone: Thunderbird 72.0 → Thunderbird 73.0
Updated•5 years ago
|
status-thunderbird73:
--- → fixed
You need to log in
before you can comment on or make changes to this bug.
Description
•