Ticket #2532291 (closed defect)
ReporterJesse Foster |
Opened: 05/8/12 Last modified: 08/1/12 Status: closed Type: defect Resolution: fixed |
Owner Dav Glass |
Target Release: 3.6.0 Priority: P3 (normal) |
|---|---|---|---|
| Summary: | DD Delegate ignores dragNode configuration option | ||
| Description: | DD Delegate allows you to pass in a configuration object, dragConfig, which is later used when creating the Drag instance. However during mousedown event handling the 'dragNode' configuration option
instead it should be something like
|
||
| Type: | defect | Observed in Version: | 3.5.1 |
| Component: | DragDrop | Severity: | S3 (normal) |
| Assigned To: | Dav Glass | Target Release: | 3.6.0 |
| Location: | Library Code | Priority: | P3 (normal) |
| Tags: | Relates To: | ||
| Browsers: | All | ||
| URL: | |||
| Test Information: | |||
Change History
|
Posted: 05/8/12
|
|
Posted: 05/11/12
|
|
Posted: 05/14/12
|
|
Posted: 05/14/12
Sure, here's a quick example in JSFiddle, http://jsfiddle.net/GQkXH/. All the yellow boxes are draggable while the blue one is configured as the dragNode. If you drag the yellow box with just DD the dragNode(blue box) is physically dragged instead, which is supposed to happen. Under DD Delegate the yellow boxes are incorrectly dragged around since it's ignoring the 'dragNode' config option. I did a quick patch in our copy of YUI 3.5 by editing _delMouseDown in dd-delegate
|
|
Posted: 05/15/12
I changed up the logic a little bit. I made it check config.dragNode before checking for proxy. That way if they want a custom proxy they can. |
|
Posted: 05/15/12
|
|
Posted: 05/15/12
Fixes #2532291 - Added config check for dragNode test on mousedown |
|
Posted: 08/1/12
Shipped in 3.6.0. Marking closed/fixed. |
Jenny Donnelly
Can you provide a simple repro case?
I created a test for this and it passes with no modification to DD:
https://gist.github.com/2696768