Closed Bug 1186516 Opened 5 years ago Closed 5 years ago

Instantbird: Change usage of draggesture to dragstart, dragdrop to drop

Categories

(Instantbird :: Other, defect)

defect
Not set
major

Tracking

(Not tracked)

RESOLVED FIXED
Instantbird 42

People

(Reporter: clokep, Assigned: aleth)

References

Details

(Keywords: addon-compat, Whiteboard: [1.6-blocking])

Attachments

(1 file)

+++ This bug was initially created as a clone of Bug #1171979 +++

In bug 1162050, we'd like to remove support for the draggesture and dragdrop events in favour of the standard dragstart and drop events. They should work the same, so it should be a search and replace.

In addition, I'd like to remove the old nsDragAndDrop.js library and instead convert to using the standard drag and drop api. This is a more significant amount of work, so Thunderbird could include the existing version in its own code in the meantime.
Whiteboard: [1.6-blocking]
Attached patch dragevents.diffSplinter Review
IB relies on nsDragAndDrop in a number of places.
Attachment #8645458 - Flags: review?(clokep)
Attachment #8645458 - Flags: review?(clokep) → review+
Keywords: checkin-needed
url:        https://hg.mozilla.org/comm-central/rev/a5e14ae6598a1dcc614a2840865c41896c76820b
changeset:  a5e14ae6598a1dcc614a2840865c41896c76820b
user:       aleth <aleth@instantbird.org>
date:       Mon Aug 10 00:34:52 2015 +0200
description:
Bug 1186516 - Change usage of draggesture to dragstart, dragdrop to drop. r=clokep
Assignee: nobody → aleth
Status: NEW → RESOLVED
Closed: 5 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Instantbird 42
This bug only renamed the events, but now nsDragAndDrop.js itself has been removed as well (announced in the description, but at the time did not happen).
You need to log in before you can comment on or make changes to this bug.