[Kde-graphics-devel] Review Request 111564: Also set URL on ksnapshots drags

[Richard Moore]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111564/#review36181
-----------------------------------------------------------

Ship it!


Ship It!

- Richard Moore


On July 18, 2013, 12:09 p.m., Sebastian Kügler wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/111564/
> -----------------------------------------------------------
> 
> (Updated July 18, 2013, 12:09 p.m.)
> 
> 
> Review request for KDE Graphics, Aaron J. Seigo and Richard Moore.
> 
> 
> Description
> -------
> 
> Attach the url of the temp filename to the dragged image in ksnapshot. (Additionally to the actual image data.) This helps applications that are too stupid to decode the entire mimedata, not just the URL, for example chrome on google plus, and a bunch of others (among them our own code, with all due respect).
> 
> Intended for 4.11 and master.
> 
> 
> This addresses bugs 158677, 256492 and 256801.
>     http://bugs.kde.org/show_bug.cgi?id=158677
>     http://bugs.kde.org/show_bug.cgi?id=256492
>     http://bugs.kde.org/show_bug.cgi?id=256801
> 
> 
> Diffs
> -----
> 
>   ksnapshot.cpp a685cd37105738362c58fd1308ce3c3984684281 
> 
> Diff: http://git.reviewboard.kde.org/r/111564/diff/
> 
> 
> Testing
> -------
> 
> Dragged screenshot from ksnapshot into browser, works
> 
> 
> Thanks,
> 
> Sebastian Kügler
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-graphics-devel/attachments/20130719/07bb9394/attachment.html>