[virt-tools-list] [PATCH] virt-viewer-app: Always allow users to close displays from the displays menu

Marc-André Lureau mlureau at redhat.com
Fri Apr 5 14:10:11 UTC 2013


ack

----- Mensaje original -----
> Marking display menu items as non sensitive for shown displays make no sense,
> since the user can always close them through the window-manager.
> 
> Having a window for a display shown when the display is not selectable nor
> ready, can happen when the agent goes away. This happens for example when
> using
> a dual monitor config with a Linux guest and then switching to a text console
> inside the guest.
> 
> Signed-off-by: Hans de Goede <hdegoede at redhat.com>
> ---
>  src/virt-viewer-app.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/src/virt-viewer-app.c b/src/virt-viewer-app.c
> index a05941b..3fd7613 100644
> --- a/src/virt-viewer-app.c
> +++ b/src/virt-viewer-app.c
> @@ -1822,7 +1822,7 @@ window_update_menu_displays_cb(gpointer key
> G_GNUC_UNUSED,
>          VirtViewerWindow *vwin =
>          VIRT_VIEWER_WINDOW(g_hash_table_lookup(self->priv->windows, nth));
>          VirtViewerDisplay *display = virt_viewer_window_get_display(vwin);
>          GtkWidget *item;
> -        gboolean visible, sensitive = FALSE;
> +        gboolean visible, sensitive;
>          gchar *label;
>  
>          label = g_strdup_printf(_("Display %d"), *nth + 1);
> @@ -1832,6 +1832,7 @@ window_update_menu_displays_cb(gpointer key
> G_GNUC_UNUSED,
>          visible =
>          gtk_widget_get_visible(GTK_WIDGET(virt_viewer_window_get_window(vwin)));
>          gtk_check_menu_item_set_active(GTK_CHECK_MENU_ITEM(item), visible);
>  
> +        sensitive = visible;
>          if (display) {
>              guint hint = virt_viewer_display_get_show_hint(display);
>  
> @@ -1841,7 +1842,6 @@ window_update_menu_displays_cb(gpointer key
> G_GNUC_UNUSED,
>              if (virt_viewer_display_get_selectable(display))
>                  sensitive = TRUE;
>          }
> -
>          gtk_widget_set_sensitive(item, sensitive);
>  
>          g_signal_connect(G_OBJECT(item),
> --
> 1.8.2
> 
> _______________________________________________
> virt-tools-list mailing list
> virt-tools-list at redhat.com
> https://www.redhat.com/mailman/listinfo/virt-tools-list
> 




More information about the virt-tools-list mailing list