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

[Commit Hook]

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


This review has been submitted with commit 3bb8fe97096badedf45863ab81f5868ca01cc2de by Sebastian Kügler to branch master.

- Commit Hook


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/21b3b71c/attachment.html>