| Summary: | [DND] ViewerDropAdapter is not supporting new URLTransfer | ||
|---|---|---|---|
| Product: | [Eclipse Project] Platform | Reporter: | Benjamin Pasero <bpasero> |
| Component: | UI | Assignee: | Platform UI Triaged <platform-ui-triaged> |
| Status: | CLOSED WONTFIX | QA Contact: | |
| Severity: | major | ||
| Priority: | P3 | CC: | duongn, snorthov |
| Version: | 3.4 | Keywords: | helpwanted |
| Target Milestone: | --- | ||
| Hardware: | PC | ||
| OS: | Windows XP | ||
| Whiteboard: | stalebug | ||
|
Description
Benjamin Pasero
Not sure if this should be SWT or not. Ben, do you know where the ViewerDropAdapter is broken ? A quick look indicates that 'doDropValidation' will only show 'lastValidOperation' but then only if it's -not- DROP_NONE....this seems suspicious. I'm going to mark this one 'helpwanted' so if you track it down and attach a patch I'd be glad to review it...the current implementation seems a bit flakey to me, I think we're trying to 'abstract away' too much. For my money 'validateDrop' shouldn't return a boolean, it should return the event.detail... Hi Eric, hmm looking more into this, this does not seem to be a bug in Eclipse at all. It seems the detail field of the DropTargetEvent is DND.DROP_NONE when dragging a link from IE to the Snippet. However, dragging a link from Firefox to the Snippet correctly shows the feedback. Maybe this should be moved to SWT for a comment. Steve, do you know of any differences between the drag source as generated by IE vs FireFox ?? Duong, do you know the answer? When an item is dragged over the drop target, SWT checks to see if the allowed operations (copy, move, link) is available in the drop target (intersection of the operations). If there is no common operation, the operation will be DND.DROP_NONE. So, IE and Firefox can specify different operations that will affect the drop target. I forgot to mention that the allowed operations are specified by the source. Doung, I just ran the code with a breakpoint on the 'dragEnter' (Line 133). When I drag a file from FFox the BP fires, from IE...not. I also put a BP at line 59 of DNDListener (DBD.DragEner handler) and it doesn't fire. Any idea why? I don't think it's a mis-match in the operations, the adapter registers to use all three 'standard' ops (MOVE, COPY & LINK). My guess is that IE doesn't actually generate the transferType that URLTransfer needs. This bug hasn't had any activity in quite some time. Maybe the problem got resolved, was a duplicate of something else, or became less pressing for some reason - or maybe it's still relevant but just hasn't been looked at yet. If you have further information on the current state of the bug, please add it. The information can be, for example, that the problem still occurs, that you still want the feature, that more information is needed, or that the bug is (for whatever reason) no longer relevant. This bug hasn't had any activity in quite some time. Maybe the problem got resolved, was a duplicate of something else, or became less pressing for some reason - or maybe it's still relevant but just hasn't been looked at yet. As such, we're closing this bug. If you have further information on the current state of the bug, please add it and reopen this bug. The information can be, for example, that the problem still occurs, that you still want the feature, that more information is needed, or that the bug is (for whatever reason) no longer relevant. -- The automated Eclipse Genie. |