Bloc icon indicating copy to clipboard operation
Bloc copied to clipboard

Dead code regarding drag and drop

Open tinchodias opened this issue 1 year ago • 1 comments

We have these classes, only "cancel" is referenced: Screenshot 2024-09-30 at 13 25 06

they are the only sends of these spaceDraw* selectors, and the methods have "red" (undefined) methods: Screenshot 2024-09-30 at 13 25 45

@rvillemeur and @Enzo-Demeulenaere do you know about this? do we remove this code?

tinchodias avatar Sep 30 '24 16:09 tinchodias

@tinchodias, these event are used in the GToolkit version of Drag&Drop, where dragged element are managed at the space level. They may be removed in the future, once we clarify exactly which way we want to handle drag&drop in Pharo. Stay with low level space, or provide higher level API like GToolkit, or a mix between both. There are no clear answers for now.

rvillemeur avatar Sep 30 '24 17:09 rvillemeur

Let's close this issue. I recently merged a PR from Enzo regarding this topic, and I see it's on development.

tinchodias avatar Nov 15 '24 14:11 tinchodias