Skip to content

Commit 69087f0

Browse files
joaompnevesjoaompneves
authored
Merge 7a6c364 into c3bbd37 
2 parents c3bbd37 + 7a6c364 commit 69087f0

File tree

11 files changed

+174
-64
lines changed

11 files changed

+174
-64
lines changed

‎CefSharp.Example/CefExample.cs‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,7 @@ public static class CefExample
2424
public const string BasicSchemeTestUrl = BaseUrl + "/SchemeTest.html";
2525
public const string ResponseFilterTestUrl = BaseUrl + "/ResponseFilterTest.html";
2626
public const string DraggableRegionTestUrl = BaseUrl + "/DraggableRegionTest.html";
27+
public const string DragDropCursorsTestUrl = BaseUrl + "/DragDropCursorsTest.html";
2728
public const string CssAnimationTestUrl = BaseUrl + "/CssAnimationTest.html";
2829
public const string CdmSupportTestUrl = BaseUrl + "/CdmSupportTest.html";
2930
public const string TestResourceUrl = "http://test/resource/load";

‎CefSharp.Example/CefSharp.Example.csproj‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -152,6 +152,7 @@
152152
<None Include="Resources\assets\js\shBrushJScript.js" />
153153
<Content Include="Resources\assets\js\shCore.js" />
154154
<Content Include="Resources\BindingTestSingle.html" />
155+
<Content Include="Resources\DragDropCursorsTest.html" />
155156
<Content Include="Resources\LegacyBindingTest.html" />
156157
<Content Include="Resources\CdmSupportTest.html" />
157158
<Content Include="Resources\CssAnimation.html" />

‎CefSharp.Example/CefSharpSchemeHandlerFactory.cs‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -47,6 +47,7 @@ static CefSharpSchemeHandlerFactory()
4747
{ "/ScriptedMethodsTest.html", Resources.ScriptedMethodsTest },
4848
{ "/ResponseFilterTest.html", Resources.ResponseFilterTest },
4949
{ "/DraggableRegionTest.html", Resources.DraggableRegionTest },
50+
{ "/DragDropCursorsTest.html", Resources.DragDropCursorsTest },
5051
{ "/CssAnimationTest.html", Resources.CssAnimation },
5152
{ "/CdmSupportTest.html", Resources.CdmSupportTest },
5253
{ "/Recaptcha.html", Resources.Recaptcha },

‎CefSharp.Example/InMemorySchemeAndResourceHandlerFactory.cs‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -45,6 +45,7 @@ static InMemorySchemeAndResourceHandlerFactory()
4545
{ "/PopupTest.html", Resources.PopupTest },
4646
{ "/SchemeTest.html", Resources.SchemeTest },
4747
{ "/TooltipTest.html", Resources.TooltipTest },
48+
{ "/DragDropCursorsTest.html", Resources.DragDropCursorsTest },
4849
{ "/FramedWebGLTest.html", Resources.FramedWebGLTest },
4950
{ "/MultiBindingTest.html", Resources.MultiBindingTest },
5051
{ "/ScriptedMethodsTest.html", Resources.ScriptedMethodsTest },

‎CefSharp.Example/Properties/Resources.Designer.cs‎

Lines changed: 31 additions & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

‎CefSharp.Example/Properties/Resources.resx‎

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -133,6 +133,9 @@
133133
<data name="TooltipTest" type="System.Resources.ResXFileRef, System.Windows.Forms">
134134
<value>..\Resources\TooltipTest.html;System.String, mscorlib, Version=2.0.0.0, Culture=neutral, PublicKeyToken=b77a5c561934e089;utf-8</value>
135135
</data>
136+
<data name="DragDropCursorsTest" type="System.Resources.ResXFileRef, System.Windows.Forms">
137+
<value>..\Resources\DragDropCursorsTest.html;System.String, mscorlib, Version=2.0.0.0, Culture=neutral, PublicKeyToken=b77a5c561934e089;utf-8</value>
138+
</data>
136139
<data name="bootstrap_min_css" type="System.Resources.ResXFileRef, System.Windows.Forms">
137140
<value>..\Resources\bootstrap\bootstrap.min.css;System.String, mscorlib, Version=4.0.0.0, Culture=neutral, PublicKeyToken=b77a5c561934e089;Windows-1252</value>
138141
</data>

‎CefSharp.Example/Resources/DragDropCursorsTest.html‎

Lines changed: 54 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,54 @@
1+
<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.0 Transitional//EN">
2+
<html>
3+
<head>
4+
<title>Drag&Drop Cursors Test</title>
5+
<style>
6+
.dropzone {
7+
width: 80px;
8+
height: 70px;
9+
padding: 10px;
10+
display: inline-block;
11+
border: 1px solid #aaaaaa;
12+
background: white;
13+
}
14+
15+
#drag {
16+
width: 200px;
17+
height: 50px;
18+
padding: 10px;
19+
border: 1px solid #aaaaaa;
20+
background: wheat;
21+
}
22+
</style>
23+
<script>
24+
function allowDrop(ev, effect) {
25+
ev.dataTransfer.dropEffect = effect;
26+
ev.preventDefault();
27+
}
28+
29+
function drag(ev) {
30+
ev.dataTransfer.effectAllowed = "all";
31+
ev.dataTransfer.setData("text/plain", ev.target.id);
32+
}
33+
34+
function drop(ev) {
35+
ev.preventDefault();
36+
var data = ev.dataTransfer.getData("text/plain");
37+
var element = document.getElementById(data);
38+
if (element) {
39+
ev.target.appendChild(element);
40+
}
41+
alert(ev.dataTransfer.dropEffect + " | " + data);
42+
}
43+
</script>
44+
</head>
45+
<body>
46+
<div class="dropzone" ondrop="drop(event)" ondragover="allowDrop(event, 'none')">None</div>
47+
<div class="dropzone" ondrop="drop(event)" ondragover="allowDrop(event, 'copy')">Copy</div>
48+
<div class="dropzone" ondrop="drop(event)" ondragover="allowDrop(event, 'move')">Move</div>
49+
<div class="dropzone" ondrop="drop(event)" ondragover="allowDrop(event, 'link')">Link</div>
50+
<br>
51+
<br>
52+
<div id="drag" draggable="true" ondragstart="drag(event)">Drag me over the drop zones</div>
53+
</body>
54+
</html>

‎CefSharp.Wpf.Example/MainWindow.xaml‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,7 @@
2929
<MenuItem Header="_LegacyBinding Test" Command="controls:CefSharpCommands.OpenTabCommand" CommandParameter="{Binding Source={x:Static ex:CefExample.LegacyBindingTestUrl}}"/>
3030
<MenuItem Header="_List Plugins" Command="controls:CefSharpCommands.OpenTabCommand" CommandParameter="{Binding Source={x:Static ex:CefExample.PluginsTestUrl}}"/>
3131
<MenuItem Header="_Tooltip Test" Command="controls:CefSharpCommands.OpenTabCommand" CommandParameter="{Binding Source={x:Static ex:CefExample.TooltipTestUrl}}"/>
32+
<MenuItem Header="D_rag&amp;Drop Cursors Test" Command="controls:CefSharpCommands.OpenTabCommand" CommandParameter="{Binding Source={x:Static ex:CefExample.DragDropCursorsTestUrl}}"/>
3233
<MenuItem Header="_Popup Test" Command="controls:CefSharpCommands.OpenTabCommand" CommandParameter="{Binding Source={x:Static ex:CefExample.PopupParentUrl}}"/>
3334
<MenuItem Header="_Popup Test Custom Scheme" Command="controls:CefSharpCommands.OpenTabCommand" CommandParameter="{Binding Source={x:Static ex:CefExample.PopupTestUrl}}"/>
3435
<MenuItem Header="_Basic Scheme Test" Command="controls:CefSharpCommands.OpenTabCommand" CommandParameter="{Binding Source={x:Static ex:CefExample.BasicSchemeTestUrl}}"/>

‎CefSharp.Wpf/CefSharp.Wpf.csproj‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -79,6 +79,7 @@
7979
</ItemGroup>
8080
<ItemGroup>
8181
<Compile Include="CefSettings.cs" />
82+
<Compile Include="Internals\DragOperationMaskExtensions.cs" />
8283
<Compile Include="Internals\MonitorInfo.cs" />
8384
<Compile Include="Internals\MonitorInfoEx.cs" />
8485
<Compile Include="Internals\RectStruct.cs" />

‎CefSharp.Wpf/ChromiumWebBrowser.cs‎

Lines changed: 12 additions & 63 deletions
Original file line numberDiff line numberDiff line change
@@ -104,7 +104,10 @@ public class ChromiumWebBrowser : Control, IRenderWebBrowser, IWpfWebBrowser
104104
/// we can reuse the drag data provided from CEF
105105
/// </summary>
106106
private IDragData currentDragData;
107-
107+
/// <summary>
108+
/// Keep the current drag&drop effects to return the appropriate effects on drag over.
109+
/// </summary>
110+
private DragDropEffects currentDragDropEffects;
108111
/// <summary>
109112
/// A flag that indicates whether or not the designer is active
110113
/// NOTE: Needs to be static for OnApplicationExit
@@ -787,14 +790,14 @@ bool IRenderWebBrowser.StartDragging(IDragData dragData, DragOperationsMask allo
787790
if (browser != null)
788791
{
789792
//DoDragDrop will fire DragEnter event
790-
var result = DragDrop.DoDragDrop(this, dataObject, GetDragEffects(allowedOps));
793+
var result = DragDrop.DoDragDrop(this, dataObject, allowedOps.GetDragEffects());
791794

792795
//DragData was stored so when DoDragDrop fires DragEnter we reuse a clone of the IDragData provided here
793796
currentDragData = null;
794797

795798
//If result == DragDropEffects.None then we'll send DragOperationsMask.None
796799
//effectively cancelling the drag operation
797-
browser.GetHost().DragSourceEndedAt(x, y, GetDragOperationsMask(result));
800+
browser.GetHost().DragSourceEndedAt(x, y, result.GetDragOperationsMask());
798801
browser.GetHost().DragSourceSystemDragEnded();
799802
}
800803
});
@@ -813,7 +816,7 @@ void IRenderWebBrowser.UpdateDragCursor(DragOperationsMask operation)
813816
/// <param name="operation">describes the allowed operation (none, move, copy, link). </param>
814817
protected virtual void UpdateDragCursor(DragOperationsMask operation)
815818
{
816-
//TODO: Someone should implement this
819+
currentDragDropEffects = operation.GetDragEffects();
817820
}
818821

819822
/// <summary>
@@ -1524,7 +1527,7 @@ private void OnDrop(object sender, DragEventArgs e)
15241527
if (browser != null)
15251528
{
15261529
var mouseEvent = GetMouseEvent(e);
1527-
var effect = GetDragOperationsMask(e.AllowedEffects);
1530+
var effect = e.AllowedEffects.GetDragOperationsMask();
15281531

15291532
browser.GetHost().DragTargetDragOver(mouseEvent, effect);
15301533
browser.GetHost().DragTargetDragDrop(mouseEvent);
@@ -1553,8 +1556,10 @@ private void OnDragOver(object sender, DragEventArgs e)
15531556
{
15541557
if (browser != null)
15551558
{
1556-
browser.GetHost().DragTargetDragOver(GetMouseEvent(e), GetDragOperationsMask(e.AllowedEffects));
1559+
browser.GetHost().DragTargetDragOver(GetMouseEvent(e), e.AllowedEffects.GetDragOperationsMask());
15571560
}
1561+
e.Effects = currentDragDropEffects;
1562+
e.Handled = true;
15581563
}
15591564

15601565
/// <summary>
@@ -1567,7 +1572,7 @@ private void OnDragEnter(object sender, DragEventArgs e)
15671572
if (browser != null)
15681573
{
15691574
var mouseEvent = GetMouseEvent(e);
1570-
var effect = GetDragOperationsMask(e.AllowedEffects);
1575+
var effect = e.AllowedEffects.GetDragOperationsMask();
15711576

15721577
//DoDragDrop will fire this handler for internally sourced Drag/Drop operations
15731578
//we use the existing IDragData (cloned copy)
@@ -1578,62 +1583,6 @@ private void OnDragEnter(object sender, DragEventArgs e)
15781583
}
15791584
}
15801585

1581-
/// <summary>
1582-
/// Converts .NET drag drop effects to CEF Drag Operations
1583-
/// </summary>
1584-
/// <param name="dragDropEffects">The drag drop effects.</param>
1585-
/// <returns>DragOperationsMask.</returns>
1586-
/// s
1587-
private static DragOperationsMask GetDragOperationsMask(DragDropEffects dragDropEffects)
1588-
{
1589-
var operations = DragOperationsMask.None;
1590-
1591-
if (dragDropEffects.HasFlag(DragDropEffects.All))
1592-
{
1593-
operations |= DragOperationsMask.Every;
1594-
}
1595-
if (dragDropEffects.HasFlag(DragDropEffects.Copy))
1596-
{
1597-
operations |= DragOperationsMask.Copy;
1598-
}
1599-
if (dragDropEffects.HasFlag(DragDropEffects.Move))
1600-
{
1601-
operations |= DragOperationsMask.Move;
1602-
}
1603-
if (dragDropEffects.HasFlag(DragDropEffects.Link))
1604-
{
1605-
operations |= DragOperationsMask.Link;
1606-
}
1607-
1608-
return operations;
1609-
}
1610-
1611-
/// <summary>
1612-
/// Gets the drag effects.
1613-
/// </summary>
1614-
/// <param name="mask">The mask.</param>
1615-
/// <returns>DragDropEffects.</returns>
1616-
private static DragDropEffects GetDragEffects(DragOperationsMask mask)
1617-
{
1618-
if ((mask & DragOperationsMask.Every) == DragOperationsMask.Every)
1619-
{
1620-
return DragDropEffects.All;
1621-
}
1622-
if ((mask & DragOperationsMask.Copy) == DragOperationsMask.Copy)
1623-
{
1624-
return DragDropEffects.Copy;
1625-
}
1626-
if ((mask & DragOperationsMask.Move) == DragOperationsMask.Move)
1627-
{
1628-
return DragDropEffects.Move;
1629-
}
1630-
if ((mask & DragOperationsMask.Link) == DragOperationsMask.Link)
1631-
{
1632-
return DragDropEffects.Link;
1633-
}
1634-
return DragDropEffects.None;
1635-
}
1636-
16371586
/// <summary>
16381587
/// PresentationSource changed handler.
16391588
/// </summary>

0 commit comments

Comments
 (0)