Re: [PATCH wayland] protocol: Improve data source notification around DnD progress

2015-11-18 Thread Carlos Garnacho
Hi Mike, On Fri, Oct 16, 2015 at 5:58 PM, Mike Blumenkrantz wrote: > On Fri, 16 Oct 2015 16:53:28 +0100 > Daniel Stone wrote: > >> Hi, >> >> On Wednesday, 30 September 2015, Carlos Garnacho >> wrote: >> >> > Currently, there's no means for the DnD origin to know whether the >> > destination is

Re: [PATCH wayland] protocol: Improve data source notification around DnD progress

2015-11-03 Thread Bill Spitzak
On Mon, Nov 2, 2015 at 9:28 AM, Carlos Garnacho wrote: > On Fri, Oct 30, 2015 at 7:10 PM, Bill Spitzak wrote: > > I think what is wanted is an indication that the drag really did finish. > For > > That's data_source.drag_finished in the patch you're commenting. > According to the documentation

Re: [PATCH wayland] protocol: Improve data source notification around DnD progress

2015-11-02 Thread Carlos Garnacho
On Fri, Oct 30, 2015 at 7:10 PM, Bill Spitzak wrote: > I think what is wanted is an indication that the drag really did finish. For That's data_source.drag_finished in the patch you're commenting. > instance a file browser acting as the source of a "move" action will want a > confirmation that t

Re: [PATCH wayland] protocol: Improve data source notification around DnD progress

2015-10-30 Thread Carlos Garnacho
On Fri, Oct 30, 2015 at 5:51 AM, Jonas Ådahl wrote: > On Thu, Oct 29, 2015 at 05:51:04PM +0100, Carlos Garnacho wrote: >> Hey Jonas!, >> >> On Thu, Oct 29, 2015 at 9:37 AM, Jonas Ådahl wrote: >> > Hey Carlos, >> > >> > Finally had a look at this one. >> >> Cheers :) >> >> > >> > On Wed, Sep 30, 2

Re: [PATCH wayland] protocol: Improve data source notification around DnD progress

2015-10-30 Thread Bill Spitzak
On Thu, Oct 29, 2015 at 9:51 PM, Jonas Ådahl wrote: > > > > I too would like some clarifications why this is needed. The cursor and > > > button state, I assume, could be reset on the next enter or by > > > cancelled/finished, but that the client may want update its UI in some > > > way. > > As

Re: [PATCH wayland] protocol: Improve data source notification around DnD progress

2015-10-29 Thread Jonas Ådahl
On Thu, Oct 29, 2015 at 05:51:04PM +0100, Carlos Garnacho wrote: > Hey Jonas!, > > On Thu, Oct 29, 2015 at 9:37 AM, Jonas Ådahl wrote: > > Hey Carlos, > > > > Finally had a look at this one. > > Cheers :) > > > > > On Wed, Sep 30, 2015 at 10:45:39PM +0200, Carlos Garnacho wrote: > >> Currently,

Re: [PATCH wayland] protocol: Improve data source notification around DnD progress

2015-10-29 Thread Michael Catanzaro
On Thu, 2015-10-29 at 16:37 +0800, Jonas Ådahl wrote: > Correct me if I'm wrong, but this event is meant to be sent after a > destination has read all the data it needs, meaning we could > effectively > send this for regular clipboard sources as well? Then just call it > "finished". I don't think

Re: [PATCH wayland] protocol: Improve data source notification around DnD progress

2015-10-29 Thread Bill Spitzak
On Thu, Oct 29, 2015 at 9:51 AM, Carlos Garnacho wrote: > So this x11-style "grab" still happens before start_drag on the > compositor, all it currently does face to the wayland compositor is > updating the pointer cursor, but client-side it brings the usual > grabby effects, input still taken "e

Re: [PATCH wayland] protocol: Improve data source notification around DnD progress

2015-10-29 Thread Carlos Garnacho
Hey Jonas!, On Thu, Oct 29, 2015 at 9:37 AM, Jonas Ådahl wrote: > Hey Carlos, > > Finally had a look at this one. Cheers :) > > On Wed, Sep 30, 2015 at 10:45:39PM +0200, Carlos Garnacho wrote: >> Currently, there's no means for the DnD origin to know whether the >> destination is actually finis

Re: [PATCH wayland] protocol: Improve data source notification around DnD progress

2015-10-29 Thread Jonas Ådahl
Hey Carlos, Finally had a look at this one. On Wed, Sep 30, 2015 at 10:45:39PM +0200, Carlos Garnacho wrote: > Currently, there's no means for the DnD origin to know whether the > destination is actually finished with the DnD transaction, short of > finalizing it after the first transfer finishes

Re: [PATCH wayland] protocol: Improve data source notification around DnD progress

2015-10-18 Thread Bill Spitzak
On 10/17/2015 08:59 AM, Michael Catanzaro wrote: On Fri, 2015-10-16 at 18:43 -0700, Bill Spitzak wrote: I think you misunderstood. The *destination* (if the cursor is still pointing at it) is what sets the cursor. To clarify: you mean the destination sets the cursor as soon as the drop complet

Re: [PATCH wayland] protocol: Improve data source notification around DnD progress

2015-10-17 Thread Michael Catanzaro
On Fri, 2015-10-16 at 18:43 -0700, Bill Spitzak wrote: > I think you misunderstood. The *destination* (if the cursor is still > pointing at it) is what sets the cursor. To clarify: you mean the destination sets the cursor as soon as the drop completes, correct? (Currently the source controls the c

Re: [PATCH wayland] protocol: Improve data source notification around DnD progress

2015-10-17 Thread Michael Catanzaro
On Thu, 2015-10-01 at 21:57 +0200, Carlos Garnacho wrote: > Remember, toolkits preserve some state. The drag source is in control > of the pointer cursor as long as the DnD operation holds, so it > should > reset its internal current cursor to the regular one for the next > time > the pointer enter

Re: [PATCH wayland] protocol: Improve data source notification around DnD progress

2015-10-16 Thread Bill Spitzak
On Thu, Oct 1, 2015 at 12:57 PM, Carlos Garnacho wrote: > On Thu, Oct 1, 2015 at 8:48 PM, Bill Spitzak wrote: > > > > > > On Wed, Sep 30, 2015 at 1:45 PM, Carlos Garnacho > wrote: > >> > >> > >> - wl_data_source.drop_performed: Happens when the operation has been > >> physically finished (eg.

Re: [PATCH wayland] protocol: Improve data source notification around DnD progress

2015-10-16 Thread Daniel Stone
[and again, with the actual CC this time ...] On Friday, 16 October 2015, Daniel Stone wrote: > Hi, > > On Wednesday, 30 September 2015, Carlos Garnacho > wrote: > >> Currently, there's no means for the DnD origin to know whether the >> destination is actually finished with the DnD transaction,

Re: [PATCH wayland] protocol: Improve data source notification around DnD progress

2015-10-16 Thread Mike Blumenkrantz
On Fri, 16 Oct 2015 16:53:28 +0100 Daniel Stone wrote: > Hi, > > On Wednesday, 30 September 2015, Carlos Garnacho > wrote: > > > Currently, there's no means for the DnD origin to know whether the > > destination is actually finished with the DnD transaction, short of > > finalizing it after th

Re: [PATCH wayland] protocol: Improve data source notification around DnD progress

2015-10-16 Thread Daniel Stone
[and again, with the actual CC this time ...] On Friday, 16 October 2015, Daniel Stone wrote: > Hi, > > On Wednesday, 30 September 2015, Carlos Garnacho > wrote: > >> Currently, there's no means for the DnD origin to know whether the >> destination is actually finished with the DnD transaction,

Re: [PATCH wayland] protocol: Improve data source notification around DnD progress

2015-10-16 Thread Daniel Stone
Hi, On Wednesday, 30 September 2015, Carlos Garnacho wrote: > Currently, there's no means for the DnD origin to know whether the > destination is actually finished with the DnD transaction, short of > finalizing it after the first transfer finishes, or leaking it forever. > > But this poses othe

Re: [PATCH wayland] protocol: Improve data source notification around DnD progress

2015-10-02 Thread Michael Catanzaro
On Thu, 2015-10-01 at 21:36 +0200, Carlos Garnacho wrote: > But this is a very valid concern, if you preemptively accept() with a > random mimetype, the user quickly finishes the drag before transfers > are done, and it isn't accepted in the end, you'd get the wrong > feedback, and the wrong outcom

Re: [PATCH wayland] protocol: Improve data source notification around DnD progress

2015-10-01 Thread Carlos Garnacho
On Thu, Oct 1, 2015 at 8:48 PM, Bill Spitzak wrote: > > > On Wed, Sep 30, 2015 at 1:45 PM, Carlos Garnacho wrote: >> >> >> - wl_data_source.drop_performed: Happens when the operation has been >> physically finished (eg. the button is released), it could be the right >> place to reset the poin

Re: [PATCH wayland] protocol: Improve data source notification around DnD progress

2015-10-01 Thread Carlos Garnacho
Hey Michael, On Thu, Oct 1, 2015 at 6:46 PM, Michael Catanzaro wrote: > One problem with this model is that a broken or malicious client could > force the drag source to be leaked by [sending accept and] never > destroying its data offer... but only once per user-initiated drag, so > we can proba

Re: [PATCH wayland] protocol: Improve data source notification around DnD progress

2015-10-01 Thread Bill Spitzak
On Wed, Sep 30, 2015 at 1:45 PM, Carlos Garnacho wrote: > > - wl_data_source.drop_performed: Happens when the operation has been > physically finished (eg. the button is released), it could be the right > place to reset the pointer cursor back and undo any other state resulting > from the i

Re: [PATCH wayland] protocol: Improve data source notification around DnD progress

2015-10-01 Thread Michael Catanzaro
One problem with this model is that a broken or malicious client could force the drag source to be leaked by [sending accept and] never destroying its data offer... but only once per user-initiated drag, so we can probably ignore that issue On Thu, 2015-10-01 at 12:00 +0200, Carlos Garnacho wr

Re: [PATCH wayland] protocol: Improve data source notification around DnD progress

2015-10-01 Thread Carlos Garnacho
Hey Michael :), On Thu, Oct 1, 2015 at 12:43 AM, Michael Catanzaro wrote: > Reviewed-by: Michael Catanzaro > > These are important fixes for the protocol that will allow drags to > work between GTK+ and Chrome. Thanks for working on this, Carlos! Thanks for the review! > > Nit #1: the existing

Re: [PATCH wayland] protocol: Improve data source notification around DnD progress

2015-09-30 Thread Michael Catanzaro
On Wed, 2015-09-30 at 17:43 -0500, Michael Catanzaro wrote: > Minor issue: this and the next patch introduce the requirement that > accept is called on the data offer before the mouse grab is released. > I > now see this has always been required by mutter -- not sure why -- > but > it's new for Wes

Re: [PATCH wayland] protocol: Improve data source notification around DnD progress

2015-09-30 Thread Michael Catanzaro
Reviewed-by: Michael Catanzaro These are important fixes for the protocol that will allow drags to work between GTK+ and Chrome. Thanks for working on this, Carlos! Nit #1: the existing documentation doesn't use the DnD abbreviation, so I would write out "drag-and-drop" in the documentation of t