[Wayland-bugs] [Bug 759168] wayland: Implement DND icon hotspot API

[gtk+ (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=759168

--- Comment #8 from Jonas Ådahl <jadahl at gmail.com> ---
(In reply to Matthias Clasen from comment #3)
> Review of attachment 316924 [details] [review]:
> 
> ::: gdk/wayland/gdkdnd-wayland.c
> @@ +51,3 @@
>    gdouble y;
> +  gint prev_hot_x;
> +  gint prev_hot_y;
> 
> I don't see the need to store these in the context. They can just be locals
> in set_hot_spot

True, this was part of an idea of making the offset being retrieved in another
way, but now that things are done this way its not needed. Will remove.

> 
> @@ +329,3 @@
> +                                .width = 1,
> +                                .height = 1,
> +                              },
> 
> This kind of on-the-fly struct looks a little out of style in the GTK+
> codebase. Do we do this elsewhere in the Wayland backend ?

Hmm, no. We use it in mutter here and there, but if you prefer I can leave it
out of GTK+.

In fact, we might not need the invalidation call at all if we can safely assume
that the hotspot is always set while setting an icon that will be drawn anyway.
The reason for adding the invalidation is to ensure we get a wl_surface.attach.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/wayland-bugs/attachments/20151209/8cc77b65/attachment.html>