search

pharo-graphics / Bloc

Low-level UI infrastructure & framework for Pharo
MIT License
80 stars 40 forks source link

Dead code regarding drag and drop #610

Open tinchodias opened 1 day ago

tinchodias commented 1 day ago

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?

rvillemeur commented 1 day ago

@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.