GNOME.org
 

[gtk/wip/matthiasc/popup4: 131/140] Stop using gtk_widget_grab_default

commit c7f5551c4313281f790a397e51758bea63898cec
Author: Matthias Clasen <mclasen redhat com>
Date:   Sat Apr 27 03:20:41 2019 +0000

    Stop using gtk_widget_grab_default
    
    Replace gtk_widget_grab_default by
    gtk_window_set_default.

 demos/gtk-demo/menus.c     | 11 +----------
 gtk/gtkassistant.c         | 12 ++++++------
 gtk/gtkdialog.c            |  8 ++++----
 gtk/gtkfilechooserdialog.c |  2 +-
 gtk/gtkinfobar.c           |  7 ++++++-
 gtk/gtkwindow.c            |  4 +---
 tests/testcalendar.c       |  2 +-
 tests/testgtk.c            | 24 ++++++++++++------------
 8 files changed, 32 insertions(+), 38 deletions(-)
---
diff --git a/demos/gtk-demo/menus.c b/demos/gtk-demo/menus.c
index 47785300f8..078ba7ef9f 100644
--- a/demos/gtk-demo/menus.c
+++ b/demos/gtk-demo/menus.c
@@ -101,51 +101,42 @@ do_menus (GtkWidget *do_widget)
 
       box = gtk_box_new (GTK_ORIENTATION_HORIZONTAL, 0);
       gtk_container_add (GTK_CONTAINER (window), box);
-      gtk_widget_show (box);
 
       box1 = gtk_box_new (GTK_ORIENTATION_VERTICAL, 0);
       gtk_container_add (GTK_CONTAINER (box), box1);
-      gtk_widget_show (box1);
 
       menubar = gtk_menu_bar_new ();
       gtk_widget_set_hexpand (menubar, TRUE);
       gtk_container_add (GTK_CONTAINER (box1), menubar);
-      gtk_widget_show (menubar);
 
       menu = create_menu (2);
 
       menuitem = gtk_menu_item_new_with_label ("test\nline2");
       gtk_menu_item_set_submenu (GTK_MENU_ITEM (menuitem), menu);
       gtk_menu_shell_append (GTK_MENU_SHELL (menubar), menuitem);
-      gtk_widget_show (menuitem);
 
       menuitem = gtk_menu_item_new_with_label ("foo");
       gtk_menu_item_set_submenu (GTK_MENU_ITEM (menuitem), create_menu (3));
       gtk_menu_shell_append (GTK_MENU_SHELL (menubar), menuitem);
-      gtk_widget_show (menuitem);
 
       menuitem = gtk_menu_item_new_with_label ("bar");
       gtk_menu_item_set_submenu (GTK_MENU_ITEM (menuitem), create_menu (4));
       gtk_menu_shell_append (GTK_MENU_SHELL (menubar), menuitem);
-      gtk_widget_show (menuitem);
 
       box2 = gtk_box_new (GTK_ORIENTATION_VERTICAL, 10);
       gtk_container_add (GTK_CONTAINER (box1), box2);
-      gtk_widget_show (box2);
 
       button = gtk_button_new_with_label ("Flip");
       g_signal_connect (button, "clicked",
                         G_CALLBACK (change_orientation), menubar);
       gtk_container_add (GTK_CONTAINER (box2), button);
-      gtk_widget_show (button);
 
       button = gtk_button_new_with_label ("Close");
       g_signal_connect_swapped (button, "clicked",
                                 G_CALLBACK(gtk_widget_destroy), window);
       gtk_container_add (GTK_CONTAINER (box2), button);
       gtk_widget_set_can_default (button, TRUE);
-      gtk_widget_grab_default (button);
-      gtk_widget_show (button);
+      gtk_window_set_default (GTK_WINDOW (window), button);
     }
 
   if (!gtk_widget_get_visible (window))
diff --git a/gtk/gtkassistant.c b/gtk/gtkassistant.c
index 1306bb4bd2..2ad008d28c 100644
--- a/gtk/gtkassistant.c
+++ b/gtk/gtkassistant.c
@@ -432,7 +432,7 @@ add_action_widgets (GtkAssistant *assistant)
 
           if (has_default)
             {
-              gtk_widget_grab_default (child);
+              gtk_window_set_default (GTK_WINDOW (assistant), child);
               gtk_style_context_add_class (gtk_widget_get_style_context (child), 
GTK_STYLE_CLASS_SUGGESTED_ACTION);
             }
         }
@@ -807,7 +807,7 @@ update_buttons_state (GtkAssistant *assistant)
     case GTK_ASSISTANT_PAGE_INTRO:
       gtk_widget_set_sensitive (priv->cancel, TRUE);
       gtk_widget_set_sensitive (priv->forward, priv->current_page->complete);
-      gtk_widget_grab_default (priv->forward);
+      gtk_window_set_default (GTK_WINDOW (assistant), priv->forward);
       gtk_widget_show (priv->forward);
       gtk_widget_hide (priv->back);
       gtk_widget_hide (priv->apply);
@@ -818,7 +818,7 @@ update_buttons_state (GtkAssistant *assistant)
       gtk_widget_set_sensitive (priv->cancel, TRUE);
       gtk_widget_set_sensitive (priv->back, TRUE);
       gtk_widget_set_sensitive (priv->apply, priv->current_page->complete);
-      gtk_widget_grab_default (priv->apply);
+      gtk_window_set_default (GTK_WINDOW (assistant), priv->apply);
       gtk_widget_show (priv->back);
       gtk_widget_show (priv->apply);
       gtk_widget_hide (priv->forward);
@@ -829,7 +829,7 @@ update_buttons_state (GtkAssistant *assistant)
       gtk_widget_set_sensitive (priv->cancel, TRUE);
       gtk_widget_set_sensitive (priv->back, TRUE);
       gtk_widget_set_sensitive (priv->forward, priv->current_page->complete);
-      gtk_widget_grab_default (priv->forward);
+      gtk_window_set_default (GTK_WINDOW (assistant), priv->forward);
       gtk_widget_show (priv->back);
       gtk_widget_show (priv->forward);
       gtk_widget_hide (priv->apply);
@@ -838,7 +838,7 @@ update_buttons_state (GtkAssistant *assistant)
       break;
     case GTK_ASSISTANT_PAGE_SUMMARY:
       gtk_widget_set_sensitive (priv->close, priv->current_page->complete);
-      gtk_widget_grab_default (priv->close);
+      gtk_window_set_default (GTK_WINDOW (assistant), priv->close);
       gtk_widget_show (priv->close);
       gtk_widget_hide (priv->back);
       gtk_widget_hide (priv->forward);
@@ -849,7 +849,7 @@ update_buttons_state (GtkAssistant *assistant)
       gtk_widget_set_sensitive (priv->cancel, priv->current_page->complete);
       gtk_widget_set_sensitive (priv->back, priv->current_page->complete);
       gtk_widget_set_sensitive (priv->forward, priv->current_page->complete);
-      gtk_widget_grab_default (priv->forward);
+      gtk_window_set_default (GTK_WINDOW (assistant), priv->forward);
       gtk_widget_show (priv->back);
       gtk_widget_hide (priv->apply);
       gtk_widget_hide (priv->close);
diff --git a/gtk/gtkdialog.c b/gtk/gtkdialog.c
index 361913e300..0d17ae9b9a 100644
--- a/gtk/gtkdialog.c
+++ b/gtk/gtkdialog.c
@@ -456,7 +456,7 @@ gtk_dialog_constructed (GObject *object)
           g_object_unref (child);
 
           if (has_default)
-            gtk_widget_grab_default (child);
+            gtk_window_set_default (GTK_WINDOW (dialog), child);
         }
       g_list_free (children);
 
@@ -865,7 +865,7 @@ gtk_dialog_add_action_widget (GtkDialog *dialog,
 
       if (gtk_widget_has_default (child))
         {
-          gtk_widget_grab_default (child);
+          gtk_window_set_default (GTK_WINDOW (dialog), child);
           update_suggested_action (dialog);
         }
     }
@@ -1028,7 +1028,7 @@ gtk_dialog_set_default_response (GtkDialog *dialog,
       ResponseData *rd = get_response_data (widget, FALSE);
 
       if (rd && rd->response_id == response_id)
-       gtk_widget_grab_default (widget);
+        gtk_window_set_default (GTK_WINDOW (dialog), widget);
 
       tmp_list = tmp_list->next;
     }
@@ -1528,7 +1528,7 @@ gtk_dialog_buildable_custom_finished (GtkBuildable *buildable,
         }
 
       if (item->is_default)
-        gtk_widget_grab_default (GTK_WIDGET (object));
+        gtk_window_set_default (GTK_WINDOW (dialog), GTK_WIDGET (object));
     }
 
   g_slist_free_full (data->items, free_action_widget_info);
diff --git a/gtk/gtkfilechooserdialog.c b/gtk/gtkfilechooserdialog.c
index ce29143f8b..050f588257 100644
--- a/gtk/gtkfilechooserdialog.c
+++ b/gtk/gtkfilechooserdialog.c
@@ -563,7 +563,7 @@ ensure_default_response (GtkFileChooserDialog *dialog)
 
   widget = get_accept_action_widget (GTK_DIALOG (dialog), TRUE);
   if (widget)
-    gtk_widget_grab_default (widget);
+    gtk_window_set_default (GTK_WINDOW (dialog), widget);
 }
 
 static void
diff --git a/gtk/gtkinfobar.c b/gtk/gtkinfobar.c
index a396264e89..318c1add37 100644
--- a/gtk/gtkinfobar.c
+++ b/gtk/gtkinfobar.c
@@ -744,7 +744,12 @@ gtk_info_bar_set_default_response (GtkInfoBar *info_bar,
       ResponseData *rd = get_response_data (widget, FALSE);
 
       if (rd && rd->response_id == response_id)
-        gtk_widget_grab_default (widget);
+        {
+          GtkWidget *window;
+
+          window = gtk_widget_get_ancestor (GTK_WIDGET (info_bar), GTK_TYPE_WINDOW);
+          gtk_window_set_default (GTK_WINDOW (window), widget);
+        }
     }
 
   g_list_free (children);
diff --git a/gtk/gtkwindow.c b/gtk/gtkwindow.c
index 29aa75ee63..2b5a915957 100644
--- a/gtk/gtkwindow.c
+++ b/gtk/gtkwindow.c
@@ -2716,9 +2716,7 @@ gtk_window_set_startup_id (GtkWindow   *window,
  *
  * The default widget is the widget that’s activated when the user
  * presses Enter in a dialog (for example). This function sets or
- * unsets the default widget for a #GtkWindow. When setting (rather
- * than unsetting) the default widget it’s generally easier to call
- * gtk_widget_grab_default() on the widget. Before making a widget
+ * unsets the default widget for a #GtkWindow. Before making a widget
  * the default widget, you must call gtk_widget_set_can_default() on
  * the widget you’d like to make the default.
  */
diff --git a/tests/testcalendar.c b/tests/testcalendar.c
index b89951e8cd..cc29c5ac0d 100644
--- a/tests/testcalendar.c
+++ b/tests/testcalendar.c
@@ -681,7 +681,7 @@ create_calendar(void)
   gtk_container_add (GTK_CONTAINER (window), vbox);
 
   gtk_widget_set_can_default (button, TRUE);
-  gtk_widget_grab_default (button);
+  gtk_window_set_default (GTK_WINDOW (window), button);
 
   gtk_window_set_default_size (GTK_WINDOW (window), 600, 0);
   gtk_widget_show (window);
diff --git a/tests/testgtk.c b/tests/testgtk.c
index e7719e39a5..0fcb86e434 100644
--- a/tests/testgtk.c
+++ b/tests/testgtk.c
@@ -318,7 +318,7 @@ create_buttons (GtkWidget *widget)
                                window);
       gtk_container_add (GTK_CONTAINER (box2), button[9]);
       gtk_widget_set_can_default (button[9], TRUE);
-      gtk_widget_grab_default (button[9]);
+      gtk_window_set_default (GTK_WINDOW (window), button[9]);
     }
 
   if (!gtk_widget_get_visible (window))
@@ -379,7 +379,7 @@ create_toggle_buttons (GtkWidget *widget)
                                window);
       gtk_container_add (GTK_CONTAINER (box2), button);
       gtk_widget_set_can_default (button, TRUE);
-      gtk_widget_grab_default (button);
+      gtk_window_set_default (GTK_WINDOW (window), button);
     }
 
   if (!gtk_widget_get_visible (window))
@@ -788,7 +788,7 @@ create_statusbar (GtkWidget *widget)
                                window);
       gtk_container_add (GTK_CONTAINER (box2), button);
       gtk_widget_set_can_default (button, TRUE);
-      gtk_widget_grab_default (button);
+      gtk_window_set_default (GTK_WINDOW (window), button);
     }
 
   if (!gtk_widget_get_visible (window))
@@ -1283,7 +1283,7 @@ create_pixbuf (GtkWidget *widget)
                                window);
       gtk_container_add (GTK_CONTAINER (box2), button);
       gtk_widget_set_can_default (button, TRUE);
-      gtk_widget_grab_default (button);
+      gtk_window_set_default (GTK_WINDOW (window), button);
     }
 
   if (!gtk_widget_get_visible (window))
@@ -1373,7 +1373,7 @@ create_tooltips (GtkWidget *widget)
                                window);
       gtk_container_add (GTK_CONTAINER (box2), button);
       gtk_widget_set_can_default (button, TRUE);
-      gtk_widget_grab_default (button);
+      gtk_window_set_default (GTK_WINDOW (window), button);
 
       gtk_widget_set_tooltip_text (button, "Push this button to close window");
     }
@@ -1807,7 +1807,7 @@ create_menus (GtkWidget *widget)
                                window);
       gtk_container_add (GTK_CONTAINER (box2), button);
       gtk_widget_set_can_default (button, TRUE);
-      gtk_widget_grab_default (button);
+      gtk_window_set_default (GTK_WINDOW (window), button);
       gtk_widget_show (button);
     }
 
@@ -2410,7 +2410,7 @@ create_entry (GtkWidget *widget)
                                window);
       gtk_container_add (GTK_CONTAINER (box2), button);
       gtk_widget_set_can_default (button, TRUE);
-      gtk_widget_grab_default (button);
+      gtk_window_set_default (GTK_WINDOW (window), button);
     }
 
   if (!gtk_widget_get_visible (window))
@@ -3926,7 +3926,7 @@ create_range_controls (GtkWidget *widget)
                                window);
       gtk_container_add (GTK_CONTAINER (box2), button);
       gtk_widget_set_can_default (button, TRUE);
-      gtk_widget_grab_default (button);
+      gtk_window_set_default (GTK_WINDOW (window), button);
       gtk_widget_show (button);
     }
 
@@ -4304,7 +4304,7 @@ create_notebook (GtkWidget *widget)
                                window);
       gtk_container_add (GTK_CONTAINER (box1), button);
       gtk_widget_set_can_default (button, TRUE);
-      gtk_widget_grab_default (button);
+      gtk_window_set_default (GTK_WINDOW (window), button);
     }
 
   if (!gtk_widget_get_visible (window))
@@ -4929,7 +4929,7 @@ create_wmhints (GtkWidget *widget)
 
       gtk_container_add (GTK_CONTAINER (box2), button);
       gtk_widget_set_can_default (button, TRUE);
-      gtk_widget_grab_default (button);
+      gtk_window_set_default (GTK_WINDOW (window), button);
       gtk_widget_show (button);
     }
 
@@ -5916,7 +5916,7 @@ create_timeout_test (GtkWidget *widget)
                                window);
       gtk_widget_set_can_default (button, TRUE);
       gtk_container_add (GTK_CONTAINER (action_area), button);
-      gtk_widget_grab_default (button);
+      gtk_window_set_default (GTK_WINDOW (window), button);
       gtk_widget_show (button);
 
       button = gtk_button_new_with_label ("start");
@@ -6511,7 +6511,7 @@ create_main_window (void)
                    window);
   gtk_container_add (GTK_CONTAINER (box2), button);
   gtk_widget_set_can_default (button, TRUE);
-  gtk_widget_grab_default (button);
+  gtk_window_set_default (GTK_WINDOW (window), button);
 
   gtk_widget_show (window);
 }
[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]