Re: [PATCH spice-gtk] Do not use deprecated GtkStock

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Thank you, pushed

Pavel

On Mon, 2016-05-02 at 09:41 +0200, Victor Toso wrote:
> On Thu, Apr 07, 2016 at 01:03:51PM +0200, Pavel Grunt wrote:
> > Use labels or named icons
> > 
> > Deprecated since Gtk 3.10
> 
> Acked-by: Victor Toso <victortoso@xxxxxxxxxx>
> 
> > ---
> >  src/spicy.c             | 23 +++++++++++------------
> >  src/usb-device-widget.c | 12 ++++++------
> >  2 files changed, 17 insertions(+), 18 deletions(-)
> > 
> > diff --git a/src/spicy.c b/src/spicy.c
> > index ca9db84..0475530 100644
> > --- a/src/spicy.c
> > +++ b/src/spicy.c
> > @@ -143,9 +143,9 @@ static int ask_user(GtkWidget *parent, char *title, char
> > *message,
> >      dialog = gtk_dialog_new_with_buttons(title,
> >                                           parent ? GTK_WINDOW(parent) :
> > NULL,
> >                                           GTK_DIALOG_DESTROY_WITH_PARENT,
> > -                                         GTK_STOCK_OK,
> > +                                         "_OK",
> >                                           GTK_RESPONSE_ACCEPT,
> > -                                         GTK_STOCK_CANCEL,
> > +                                         "_Cancel",
> >                                           GTK_RESPONSE_REJECT,
> >                                           NULL);
> >      gtk_dialog_set_default_response(GTK_DIALOG(dialog),
> > GTK_RESPONSE_ACCEPT);
> > @@ -370,7 +370,7 @@ static void menu_cb_select_usb_devices(GtkAction
> > *action, void *data)
> >                      "Select USB devices for redirection",
> >                      GTK_WINDOW(win->toplevel),
> >                      GTK_DIALOG_MODAL | GTK_DIALOG_DESTROY_WITH_PARENT,
> > -                    GTK_STOCK_CLOSE, GTK_RESPONSE_ACCEPT,
> > +                    "_Close", GTK_RESPONSE_ACCEPT,
> >                      NULL);
> >      gtk_dialog_set_default_response(GTK_DIALOG(dialog),
> > GTK_RESPONSE_ACCEPT);
> >      gtk_container_set_border_width(GTK_CONTAINER(dialog), 12);
> > @@ -462,7 +462,7 @@ static void menu_cb_about(GtkAction *action, void *data)
> >                            "authors",         authors,
> >                            "comments",        comments,
> >                            "copyright",       copyright,
> > -                          "logo-icon-name",  GTK_STOCK_ABOUT,
> > +                          "logo-icon-name",  "help-about",
> >                            "website",         website,
> >                            "version",         PACKAGE_VERSION,
> >                            "license",         "LGPLv2.1",
> > @@ -640,12 +640,12 @@ static const GtkActionEntry entries[] = {
> >  
> >          /* File menu */
> >          .name        = "Connect",
> > -        .stock_id    = GTK_STOCK_CONNECT,
> > +        .stock_id    = "_Connect",
> >          .label       = "_Connect ...",
> >          .callback    = G_CALLBACK(menu_cb_connect),
> >      },{
> >          .name        = "Close",
> > -        .stock_id    = GTK_STOCK_CLOSE,
> > +        .stock_id    = "window-close",
> >          .label       = "_Close",
> >          .callback    = G_CALLBACK(menu_cb_close),
> >          .accelerator = "", /* none (disable default "<control>W") */
> > @@ -653,13 +653,13 @@ static const GtkActionEntry entries[] = {
> >  
> >          /* Edit menu */
> >          .name        = "CopyToGuest",
> > -        .stock_id    = GTK_STOCK_COPY,
> > +        .stock_id    = "edit-copy",
> >          .label       = "_Copy to guest",
> >          .callback    = G_CALLBACK(menu_cb_copy),
> >          .accelerator = "", /* none (disable default "<control>C") */
> >      },{
> >          .name        = "PasteFromGuest",
> > -        .stock_id    = GTK_STOCK_PASTE,
> > +        .stock_id    = "edit-paste",
> >          .label       = "_Paste from guest",
> >          .callback    = G_CALLBACK(menu_cb_paste),
> >          .accelerator = "", /* none (disable default "<control>V") */
> > @@ -667,7 +667,7 @@ static const GtkActionEntry entries[] = {
> >  
> >          /* View menu */
> >          .name        = "Fullscreen",
> > -        .stock_id    = GTK_STOCK_FULLSCREEN,
> > +        .stock_id    = "view-fullscreen",
> >          .label       = "_Fullscreen",
> >          .callback    = G_CALLBACK(menu_cb_fullscreen),
> >          .accelerator = "<shift>F11",
> > @@ -695,7 +695,7 @@ static const GtkActionEntry entries[] = {
> >  
> >          /* Help menu */
> >          .name        = "About",
> > -        .stock_id    = GTK_STOCK_ABOUT,
> > +        .stock_id    = "help-about",
> >          .label       = "_About ...",
> >          .callback    = G_CALLBACK(menu_cb_about),
> >      }
> > @@ -1486,8 +1486,7 @@ TransferTaskWidgets
> > *transfer_task_widgets_new(SpiceFileTransferTask *task)
> >  
> >      widgets->vbox = gtk_box_new(GTK_ORIENTATION_VERTICAL, 0);
> >      widgets->hbox = gtk_box_new(GTK_ORIENTATION_HORIZONTAL, 6);
> > -    widgets->cancel = gtk_button_new_from_icon_name(GTK_STOCK_CANCEL,
> > -                                                    GTK_ICON_SIZE_SMALL_TOO
> > LBAR);
> > +    widgets->cancel = gtk_button_new_with_label("_Cancel");
> >  
> >      widgets->progress = gtk_progress_bar_new();
> >      widgets->label =
> > gtk_label_new(spice_file_transfer_task_get_filename(task));
> > diff --git a/src/usb-device-widget.c b/src/usb-device-widget.c
> > index d3ea79a..05c7054 100644
> > --- a/src/usb-device-widget.c
> > +++ b/src/usb-device-widget.c
> > @@ -166,8 +166,8 @@
> > spice_usb_device_widget_show_info_bar(SpiceUsbDeviceWidget *self,
> >      hbox = gtk_box_new(GTK_ORIENTATION_HORIZONTAL, 12);
> >      gtk_container_add(GTK_CONTAINER(content_area), hbox);
> >  
> > -    widget = gtk_image_new_from_stock(stock_icon_id,
> > -                                      GTK_ICON_SIZE_SMALL_TOOLBAR);
> > +    widget = gtk_image_new_from_icon_name(stock_icon_id,
> > +                                          GTK_ICON_SIZE_SMALL_TOOLBAR);
> >      gtk_box_pack_start(GTK_BOX(hbox), widget, FALSE, FALSE, 0);
> >  
> >      widget = gtk_label_new(message);
> > @@ -214,7 +214,7 @@ static GObject *spice_usb_device_widget_constructor(
> >      if (err) {
> >          spice_usb_device_widget_show_info_bar(self, err->message,
> >                                                GTK_MESSAGE_WARNING,
> > -                                              GTK_STOCK_DIALOG_WARNING);
> > +                                              "dialog-warning");
> >          g_clear_error(&err);
> >          return obj;
> >      }
> > @@ -432,13 +432,13 @@ static gboolean
> > spice_usb_device_widget_update_status(gpointer user_data)
> >      if (priv->err_msg) {
> >          spice_usb_device_widget_show_info_bar(self, priv->err_msg,
> >                                                GTK_MESSAGE_INFO,
> > -                                              GTK_STOCK_DIALOG_WARNING);
> > +                                              "dialog-warning");
> >          g_free(priv->err_msg);
> >          priv->err_msg = NULL;
> >      } else if (redirecting) {
> >          spice_usb_device_widget_show_info_bar(self, _("Redirecting USB
> > Device..."),
> >                                                GTK_MESSAGE_INFO,
> > -                                              GTK_STOCK_DIALOG_INFO);
> > +                                              "dialog-information");
> >      } else {
> >          spice_usb_device_widget_hide_info_bar(self);
> >      }
> > @@ -446,7 +446,7 @@ static gboolean
> > spice_usb_device_widget_update_status(gpointer user_data)
> >      if (priv->device_count == 0)
> >          spice_usb_device_widget_show_info_bar(self, _("No USB devices
> > detected"),
> >                                                GTK_MESSAGE_INFO,
> > -                                              GTK_STOCK_DIALOG_INFO);
> > +                                              "dialog-information");
> >      return FALSE;
> >  }
> >  
> > -- 
> > 2.8.0
> > 
> > _______________________________________________
> > Spice-devel mailing list
> > Spice-devel@xxxxxxxxxxxxxxxxxxxxx
> > https://lists.freedesktop.org/mailman/listinfo/spice-devel
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]