WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
40788
[GTK] Support for image drags
https://bugs.webkit.org/show_bug.cgi?id=40788
Summary
[GTK] Support for image drags
Martin Robinson
Reported
Thursday, June 17, 2010 6:37:04 PM UTC
Dragging images around editable areas or to another application does not work from a GTK+ WebView.
Attachments
Add image dragging support
(14.10 KB, patch)
2010-06-17 10:53 PDT
,
Martin Robinson
gustavo
: review+
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Martin Robinson
Comment 1
Thursday, June 17, 2010 6:53:09 PM UTC
Created
attachment 59012
[details]
Add image dragging support
Gustavo Noronha (kov)
Comment 2
Friday, August 6, 2010 12:51:35 PM UTC
Comment on
attachment 59012
[details]
Add image dragging support 314 void ClipboardGtk::declareAndWriteDragImage(Element* element, const KURL& url, const String& label, Frame* frame) 314315 { 315 CachedImage* cachedImage = getCachedImage(element); 316 if (!cachedImage || !cachedImage->isLoaded()) 316 m_dataObject->setURL(url, label); 317 m_dataObject->setMarkup(createMarkup(element, IncludeNode, 0, AbsoluteURLs)); 318 319 CachedImage* image = getCachedImage(element); 320 if (!image || !image->isLoaded()) 317321 return; So what happens if there is no image? You can then paste the URL or the markup, but not the image, and that's all? Thanks!
Martin Robinson
Comment 3
Friday, August 6, 2010 5:00:14 PM UTC
Committed
r64852
: <
http://trac.webkit.org/changeset/64852
>
Martin Robinson
Comment 4
Friday, August 6, 2010 5:01:47 PM UTC
(In reply to
comment #2
)
> So what happens if there is no image? You can then paste the URL or the markup, but not the image, and that's all? Thanks!
Thanks for the review! If, for some reason, the image cannot be placed on the data object, then there will still be text and markup available for applications that also support those targets.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug