Opened 3 months ago

Last modified 7 weeks ago

#15930 portneeded defect (port to stable)

GTK: dnd return code is not set for Drop event

Reported by: R.U.10 Owned by:
Priority: normal Milestone:
Component: wxGTK Version: dev-latest
Keywords: Cc: nerijus@…
Blocked By: Blocking:
Patch: yes


Under GTK, the return code is defined during the Drag Over operation and not for the Drop event. It is currently impossible for the Drag source to know if the Drop operation on the target has failed or succeeded.
If some data are dragged over a suitable destination but if the Drop operation fails, the Drop Source gets back a return code that indicates that the Drop operation has succeeded.

Attachments (1)

GTK_Drop_status.patch download (1.8 KB) - added by R.U.10 3 months ago.

Download all attachments as: .zip

Change History (7)

Changed 3 months ago by R.U.10

comment:1 Changed 3 months ago by vadz

  • Resolution set to port to stable
  • Status changed from new to portneeded

Thanks! I'm not so sure of my understanding of GTK+ dnd to apply this to 3.0 immediately, but if no problems are found on the trunk with this patch, it should be backported there too later.

comment:2 Changed 3 months ago by VZ

  • Resolution changed from port to stable to fixed
  • Status changed from portneeded to closed

(In [75745]) Correctly detect cancelled drag-and-drop operations in wxGTK.

The status of the drop operation wasn't propagated back to the initiator of
drag and drop, so failing to drop data over a possibly accepting recipient
could result in wrong behaviour and even data loss.

Closes #15930.

comment:3 Changed 3 months ago by PC

(In [75749]) build fix for GTK3, see #15930

comment:4 Changed 3 months ago by pcor

  • Resolution changed from fixed to port to stable
  • Status changed from closed to portneeded

comment:5 Changed 7 weeks ago by nerijus

This patch breaks message moving in Mahogany and wxGTK. See for more info.

comment:6 Changed 7 weeks ago by nerijus

  • Cc nerijus@… added
Note: See TracTickets for help on using tickets.