Review Request 124472: WIP: Implement drag and drop support for icon widget

[Bhushan Shah]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124472/#review84994
-----------------------------------------------------------

Ship it!


Ship It!

- Bhushan Shah


On Sept. 8, 2015, 12:25 a.m., Kai Uwe Broulik wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/124472/
> -----------------------------------------------------------
> 
> (Updated Sept. 8, 2015, 12:25 a.m.)
> 
> 
> Review request for Plasma and Bhushan Shah.
> 
> 
> Repository: plasma-workspace
> 
> 
> Description
> -------
> 
> Brings back the ability to drop files onto a icon widget and have the associated application open it. Dropping something onto a folder does not yet work because it should use the konq menu thingie (Copy/Move/Link) but I don't want yet another copy of that (there's one in the folderview containment).
> 
> It should probably check already on drag enter whether it's a viable drop target (eg. executable or folder), also do some coding style cleanups I thought I had already committed earlier.
> 
> 
> Diffs
> -----
> 
>   applets/icon/package/contents/ui/main.qml 018105b 
>   applets/icon/plugin/icon_p.h ba2d9f7 
>   applets/icon/plugin/icon_p.cpp 18806fc 
> 
> Diff: https://git.reviewboard.kde.org/r/124472/diff/
> 
> 
> Testing
> -------
> 
> Dragged Gwenview onto the desktop, an icon was created.
> Dropped one picture onto it, Gwenview opened it.
> Dropped two pictures onto, Gwenview showed both.
> 
> 
> Thanks,
> 
> Kai Uwe Broulik
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20150908/7681fbc0/attachment-0001.html>