Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Drag and drop issues #97

Open
GoogleCodeExporter opened this issue Aug 28, 2015 · 6 comments
Open

Drag and drop issues #97

GoogleCodeExporter opened this issue Aug 28, 2015 · 6 comments
Labels

Comments

@GoogleCodeExporter
Copy link

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
Copy link
Author

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
Copy link
Author

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
Copy link
Author

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

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

@GoogleCodeExporter
Copy link
Author

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

  • Added labels: Bug

@GoogleCodeExporter
Copy link
Author

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

  • Added labels: Restrict-AddIssueComment-Commit

@GoogleCodeExporter
Copy link
Author

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants