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

RESOLVED FIXED in Instantbird 42

Status

--
major
RESOLVED FIXED
4 years ago
3 years ago

People

(Reporter: clokep, Assigned: aleth)

Tracking

({addon-compat})

trunk
Instantbird 42
addon-compat
Dependency tree / graph

Details

(Whiteboard: [1.6-blocking])

Attachments

(1 attachment)

(Reporter)

Description

4 years ago
+++ 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.
(Assignee)

Updated

4 years ago
Whiteboard: [1.6-blocking]
(Assignee)

Comment 1

4 years ago
Created attachment 8645458 [details] [diff] [review]
dragevents.diff

IB relies on nsDragAndDrop in a number of places.
Attachment #8645458 - Flags: review?(clokep)
(Reporter)

Updated

4 years ago
Attachment #8645458 - Flags: review?(clokep) → review+
(Assignee)

Updated

4 years ago
Keywords: checkin-needed
(Assignee)

Comment 2

4 years ago
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)

Updated

4 years ago
Assignee: nobody → aleth
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Instantbird 42
Depends on: 1286902
(Assignee)

Comment 3

3 years ago
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.