phpdesktop icon indicating copy to clipboard operation
phpdesktop copied to clipboard

Drag and drop issues

Open GoogleCodeExporter opened this issue 10 years ago • 6 comments

To allow drag and drop in phpdesktop you need to set the "external_drag" 
setting to true, see the ChromeSettings wiki page. The name is unfortunate as 
it controls both external dragging (from OS/other applications) and internal 
dragging on webpages. Internal dragging should be enabled by default without 
the need of any setting.

Original issue reported on code.google.com by [email protected] on 24 Mar 2014 at 7:03

GoogleCodeExporter avatar Aug 28 '15 11:08 GoogleCodeExporter

Seems like a bug in CEF. The CefDragHandler::OnDragEnter callback is supposed 
to affect only external drag events, but looks like it also affects internal 
drag events when returning true (to cancel drag event). Problem reported on CEF 
Forum:

http://www.magpcss.org/ceforum/viewtopic.php?f=6&t=11592

Original comment by [email protected] on 24 Mar 2014 at 11:30

GoogleCodeExporter avatar Aug 28 '15 11:08 GoogleCodeExporter

The solution to file dropping onto phpdesktop window is to detect it local file 
url in OnBeforeNavigation. The url dropping is a rare case and I think we can 
ignore it.

So by default OnDragEnter would return false (allow drag events). The 
external_drag setting would be removed. And OnBeforeNavigation would add a case 
for a local file being dropped into phpdesktop browser and that would be 
disallowed.

Original comment by [email protected] on 24 Mar 2014 at 9:05

GoogleCodeExporter avatar Aug 28 '15 11:08 GoogleCodeExporter

Original comment by [email protected] on 9 Aug 2014 at 6:28

  • Added labels: NextRelease, Priority-High
  • Removed labels: Priority-Medium

GoogleCodeExporter avatar Aug 28 '15 11:08 GoogleCodeExporter

Original comment by [email protected] on 13 Oct 2014 at 6:51

  • Added labels: Bug

GoogleCodeExporter avatar Aug 28 '15 11:08 GoogleCodeExporter

Original comment by [email protected] on 21 Oct 2014 at 10:38

  • Added labels: Restrict-AddIssueComment-Commit

GoogleCodeExporter avatar Aug 28 '15 11:08 GoogleCodeExporter

Project will move to Github. Find this issue at the new address (soon): 
https://github.com/cztomczak/phpdesktop/issues/97

Original comment by [email protected] on 24 Aug 2015 at 3:28

GoogleCodeExporter avatar Aug 28 '15 11:08 GoogleCodeExporter