Opened 11 years ago

Closed 4 years ago

#7164 closed enhancement (patchwelcome)

[CLA][Patch] DnD extension: 2D (i.e. non-linear) Container using absolute positioning

Reported by: chucky Owned by: Eugene Lazutkin
Priority: low Milestone: 1.13
Component: DnD Version: 1.1.1
Keywords: needsreview Cc:
Blocked By: Blocking:

Description

Following description is copied from the forum (http://dojotoolkit.org/forum/dojo-core-dojo-0-9/dojo-core-development-discussion/patch-dnd-extension-2d-i-e-non-linear-cont)

I extended the DnD module to support inserting items at the position where they are dropped. It is done by absolute positioning within the Container. I created a simple demo at http://mikula.ic.cz/dnddemo/ Basically, when Manager publishes the "/dnd/drop" topic, it passes an additional argument -- mouse event that caused the drop. Container constructor accepts an additional Boolean parameter absolutePositioning -- when true, the items are positioned absolutely within the container at the position calculated from the passed mouse event.

There is still an odd behavior when dragging more items at once -- they are all placed at the same position. Still I would rather have 2D positioning work fine for single items only than not have 2D positioning at all.

I attach the patch.

Attachments (1)

dnd_absolutePositioning.diff (11.2 KB) - added by chucky 11 years ago.
patch against revision 14394

Download all attachments as: .zip

Change History (9)

Changed 11 years ago by chucky

patch against revision 14394

comment:1 Changed 11 years ago by Adam Peller

Owner: changed from anonymous to Eugene Lazutkin

comment:2 Changed 11 years ago by bill

Component: GeneralDnD

comment:3 Changed 11 years ago by Eugene Lazutkin

Milestone: tbdfuture
Status: newassigned

comment:4 Changed 11 years ago by Eugene Lazutkin

Moving all open ticketd to the future.

comment:5 Changed 8 years ago by ben hockey

Keywords: needsreview added
Priority: highlow

i'm trying to identify stale tickets. if there is a need to keep this ticket open, please replace the "needsreview" keyword with "reviewed". if there is no need to keep this ticket open then please close it.

comment:6 Changed 8 years ago by bill

Sounds like MDND, but simpler in that there's nothing to avoid the items overlapping each other.

comment:7 Changed 8 years ago by Eugene Lazutkin

Need more time to study the solution. But it does look like it overlaps with dojox.mdnd somewhat.

comment:8 Changed 4 years ago by dylan

Milestone: future1.12
Resolution: patchwelcome
Status: assignedclosed

Sorry, I have no idea why this was never renewed. It's very stale now, so I'm closing as patchwelcome. I promise a faster review if there's interest in updating against master.

Note: See TracTickets for help on using tickets.