Re: [PATCH virt-viewer 1/2] window: save window geometry if the window is realized

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

 



Agree with Hans's ACK, just a few nits:

On Tue, Mar 26, 2013 at 12:37:35PM +0100, Marc-André Lureau wrote:
> Protect against re-entering fullscreen by moving pre-condition,
> keey the last know window geometry, since it stays valid.

keep the last known

> ---
>  src/virt-viewer-window.c | 11 +++++------
>  1 file changed, 5 insertions(+), 6 deletions(-)
> 
> diff --git a/src/virt-viewer-window.c b/src/virt-viewer-window.c
> index 1cd0eee..9e064c7 100644
> --- a/src/virt-viewer-window.c
> +++ b/src/virt-viewer-window.c
> @@ -519,7 +519,6 @@ virt_viewer_window_leave_fullscreen(VirtViewerWindow *self)
>          gtk_window_resize(GTK_WINDOW(priv->window),
>                            priv->before_fullscreen.width,
>                            priv->before_fullscreen.height);
> -        priv->before_saved = FALSE;
>      }
>  }
>  
> @@ -530,7 +529,11 @@ virt_viewer_window_enter_fullscreen(VirtViewerWindow *self, gint monitor)
>      GtkWidget *menu = GTK_WIDGET(gtk_builder_get_object(priv->builder, "top-menu"));
>      GtkCheckMenuItem *check = GTK_CHECK_MENU_ITEM(gtk_builder_get_object(priv->builder, "menu-view-fullscreen"));
>  
> -    if (!priv->before_saved) {
> +    if (priv->fullscreen)
> +        return;
> +    priv->fullscreen = TRUE;
> +
> +    if (gtk_widget_get_realized(priv->window)) {

This needs a compat #define for older gtk+

Christophe

Attachment: pgpVRT0XxOcZn.pgp
Description: PGP signature

_______________________________________________
virt-tools-list mailing list
virt-tools-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/virt-tools-list

[Index of Archives]     [Linux Virtualization]     [KVM Development]     [CentOS Virtualization]     [Netdev]     [Ethernet Bridging]     [Linux Wireless]     [Kernel Newbies]     [Security]     [Linux for Hams]     [Netfilter]     [Bugtraq]     [Yosemite Forum]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux Admin]     [Samba]     [Video 4 Linux]

  Powered by Linux