<div class="gmail_quote">On Sat, May 7, 2011 at 7:45 PM, Ilkka Ollakka <span dir="ltr"><<a href="mailto:ileoo@videolan.org">ileoo@videolan.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
On Sat, May 07, 2011 at 08:27:45PM +0300, Juha Jeronen wrote:<br><div class="im"></div></blockquote><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"><div class="im">
> > On Sat, May 07, 2011 at 01:37:29AM +0200, Jakob Leben wrote:<br>
> >> For very long time already (couple of months) opening files by drag&dropping<br>
> >> them onto Qt interface does not work in the development version on my Linux<br>
> >> (Kubuntu, both 10.10 and 11.04).<br>
<br>
> Doesn't work for me in KDE4+Konqueror. Maybe something to do with KDE?<br>
<br>
</div>Yes, seems that konqueror doesn't do Qt::CopyAction on default dnd, just<br>
Qt::MoveAction. So basicly it could be just to check if action if either<br>
of those and set it to CopyAction, but not sure if it has some effects<br>
(and kinda too tired to read documentation about it atm.)</blockquote><div><br></div><div>The change in behavior from 1.1.x to current was introduced in commi<font class="Apple-style-span" face="arial, helvetica, sans-serif">t [<span class="Apple-style-span" style="font-size: 13px; ">eeb1cf53993]. Indeed, the previous behavior must have let through other actions than Qt::CopyAction, while the current doesn't.</span></font></div>
<div><font class="Apple-style-span" face="arial, helvetica, sans-serif"><span class="Apple-style-span" style="font-size: 13px; ">I don't see any reason for not responding to MoveAction as well. And I don't see any reason for setting the action in </span></font><span class="Apple-style-span" style="font-family: monospace; font-size: 12px; white-space: pre; ">MainInterface::dropEventPlay </span><span class="Apple-style-span" style="font-size: 12px; white-space: pre; "><font class="Apple-style-span" face="arial, helvetica, sans-serif">either, </font></span><span class="Apple-style-span" style="font-size: 12px; white-space: pre; "><font class="Apple-style-span" face="arial, helvetica, sans-serif">as the drop event will be consumed and not processed any further so after responding to the event it really doesn't matter what the action is anymore.</font></span></div>
<div><font class="Apple-style-span" face="arial, helvetica, sans-serif"><span class="Apple-style-span" style="font-size: 12px; white-space: pre;"><br></span></font></div><div><font class="Apple-style-span" face="arial, helvetica, sans-serif"><span class="Apple-style-span" style="font-size: 12px; white-space: pre;">I propose the attached patch.</span></font></div>
<meta http-equiv="content-type" content="text/html; charset=utf-8"></div>