entrycompletion: Realize popup after setting screen
authorJonas Ådahl <jadahl@gmail.com>
Wed, 14 Mar 2018 03:54:02 +0000 (11:54 +0800)
committerJonas Ådahl <jadahl@gmail.com>
Wed, 14 Mar 2018 07:37:26 +0000 (15:37 +0800)
Realization is done as a side effect of calling
_gtk_entry_completion_resize_popup(), but if this is done before the
GdkScreen of the GtkWindow is set up correctly, it may result in the
widget being unrealized when the screen is updated. This may happen
when the file dialog parent window is not using the default GdkDisplay.

To avoid this issue, realize the popup after the screen has been
properly set up.

Fixes #83 in gtk4

gtk/gtkentrycompletion.c

index 32d564b7c1c47d74b152c01c5fc4906206f53bf0..a5e6aa68af9f6878610269970f0950f9e35be74e 100644 (file)
@@ -1514,8 +1514,6 @@ gtk_entry_completion_popup (GtkEntryCompletion *completion)
   /* default on no match */
   completion->priv->current_selected = -1;
 
-  _gtk_entry_completion_resize_popup (completion);
-
   toplevel = gtk_widget_get_toplevel (completion->priv->entry);
   if (GTK_IS_WINDOW (toplevel))
     {
@@ -1529,6 +1527,8 @@ gtk_entry_completion_popup (GtkEntryCompletion *completion)
                           gtk_widget_get_display (completion->priv->entry));
   gtk_widget_realize (completion->priv->popup_window);
 
+  _gtk_entry_completion_resize_popup (completion);
+
   if (completion->priv->device)
     {
       gtk_grab_add (completion->priv->popup_window);