Move thread code to threadPane.js and remove messengerdnd.js
Categories
(SeaMonkey :: MailNews: Message Display, defect)
Tracking
(seamonkey2.53+ fixed)
People
(Reporter: sgautherie, Assigned: iannbugzilla)
References
Details
(Whiteboard: SM2.53.8.1)
Attachments
(1 file)
(deleted),
patch
|
frg
:
review+
frg
:
approval-comm-release+
frg
:
approval-comm-esr60+
|
Details | Diff | Splinter Review |
[Approval Request Comment]
Regression caused by (bug #): 510793
User impact if declined: none
Testing completed (on m-c, etc.): 2.53.8
Risk to taking this patch (and alternatives if risky): low
String changes made by this patch: none
Three functions (CanDropOnFolderTree, DropOnFolderTree and BeginDragFolderTree) are no longer used and the rest are related to the threadPane so should be move to threadPane.js
Comment 2•3 years ago
|
||
Comment on attachment 9220963 [details] [diff] [review]
515675-remove-messengerdnd-2539.patch
LGTM
I think the max file limit could probably be lifted but NS_MAX_FILEDESCRIPTOR is still at 128 +1 in :
https://searchfox.org/mozilla-central/source/widget/windows/nsDataObj.cpp#471
For a future bug I see Re: not being translated which is the same for TB but does not sound right when creating a file name
FRG
Pushed by frgrahl@gmx.net:
https://hg.mozilla.org/comm-central/rev/98c19646a9fe
Move thread code to threadPane.js and remove messengerdnd.js. r=frg
Updated•3 years ago
|
Comment 4•3 years ago
|
||
https://gitlab.com/seamonkey-project/seamonkey-2.53-comm/-/commit/ff8238867a6e8ff1de276ddc7fa3c8d06046a34b
Move thread code to threadPane.js and remove messengerdnd.js. r=frg a=frg
Description
•