improvements/simplifications for MozAttachmentlist
Categories
(Thunderbird :: Mail Window Front End, defect)
Tracking
(thunderbird68 fixed, thunderbird69 fixed)
People
(Reporter: mkmelin, Assigned: arshad)
References
Details
Attachments
(1 file)
(deleted),
patch
|
mkmelin
:
review+
jorgk-bmo
:
approval-comm-beta+
|
Details | Diff | Splinter Review |
+++ This bug was initially created as a clone of Bug #1523607 +++
This is the followup to bug 1523607 comment 53
Comment 1•5 years ago
|
||
... and bug 1523607 comment 63 and bug 1523607 comment 64.
Assignee | ||
Comment 2•5 years ago
|
||
ALso fix Bug 1554943 and Bug 1552965.
Assignee | ||
Comment 3•5 years ago
|
||
Assignee | ||
Comment 4•5 years ago
|
||
target.matches error is not fixed yet.
Reporter | ||
Comment 5•5 years ago
|
||
Reporter | ||
Updated•5 years ago
|
Comment 6•5 years ago
|
||
Comment 7•5 years ago
|
||
(In reply to Arshad Khan [:arshad] from comment #3)
Fixes Bug 1554943, Bug 1552965 and https://bugzilla.mozilla.org/show_bug.cgi?id=1547699#c1
Yes, I verified that the two bugs are fixed. The issue from bug 1523607 comment #64 is not fixed, I still get:
JavaScript error: chrome://messenger/content/messengercompose/MsgComposeCommands.js, line 5825: TypeError: target.matches is not a function
and an attachment still can't be dragged to after the last item to it becomes the last item.
Reporter | ||
Comment 8•5 years ago
|
||
Yes, but better to do that in a separate bug.
Assignee | ||
Comment 9•5 years ago
|
||
This bug needs a rebase if Bug 1556122 is pushed first.
Comment 10•5 years ago
|
||
Sure, already lined up for landing.
Comment 11•5 years ago
|
||
Pushed by mozilla@jorgk.com:
https://hg.mozilla.org/comm-central/rev/5d14d6540a92
Improvements/Simplifications for MozAttachmentlist. r=mkmelin
Updated•5 years ago
|
Comment 12•5 years ago
|
||
Description
•