[Wayland-bugs] [Bug 757558] gtk_window_move() unreliable in Wayland?

[gtk+ (GNOME Bugzilla)]

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

--- Comment #6 from Carlos Garnacho <carlosg at gnome.org> ---
(In reply to Milan Crha from comment #5)
> Sure, for the emoticon it's created here:
> https://git.gnome.org/browse/evolution/tree/e-util/e-emoticon-tool-button.
> c#n573
> and repositioned here:
> https://git.gnome.org/browse/evolution/tree/e-util/e-emoticon-tool-button.
> c#n124
> 
> I compared the code with the one for GtkComboBox and it seems to me it's
> pretty much the same, except we hold the popup window for the whole lifetime
> of the widget, while GtkComboBox seems to build and destroy the popup on
> demand.
> 
> Please note that the X11 works properly, with the same code in the evolution.

Thanks!

I see that you're setting the transient_for on the popup conditionally around
https://git.gnome.org/browse/evolution/tree/e-util/e-emoticon-tool-button.c#n581
 . At that time the toplevel will still be NULL, so that if() is never stepped
in.

The transient_for window is essential for popups to appear properly positioned
in wayland, so that needs to be done at some point when the button has a
toplevel, maybe hierarchy_changed(), or before showing.

-- 
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/20151104/0978dd7e/attachment.html>