Re: [PATCH virt-viewer] Fix bug with initial placement of fullscreen windows

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

 



On Thu, 2014-10-09 at 11:21 -0500, Jonathon Jongsma wrote:
> The function app_window_try_fullscreen() will lookup the initial monitor
> for the nth monitor internally, so we should pass in the display ID to the function
> rather than the mapped monitor ID. This was causing 2 monitors on the
> same monitor with a configuration like this:
> 
>     monitor-mapping=1:2;2:1
> ---
>  src/virt-viewer-app.c | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
> 
> diff --git a/src/virt-viewer-app.c b/src/virt-viewer-app.c
> index f0c378d..1402986 100644
> --- a/src/virt-viewer-app.c
> +++ b/src/virt-viewer-app.c
> @@ -885,8 +885,7 @@ ensure_window_for_display(VirtViewerApp *self, VirtViewerDisplay *display)
>              g_debug("Found a window without a display, reusing for this display...");
>              virt_viewer_app_set_window_subtitle(self, win, nth);
>              if (self->priv->fullscreen && !self->priv->kiosk)
> -                app_window_try_fullscreen(self, win,
> -                                          virt_viewer_app_get_initial_monitor_for_display(self, nth));
> +                app_window_try_fullscreen(self, win, nth);
>          } else {
>              win = virt_viewer_app_window_new(self, nth);
>          }

ACK!

_______________________________________________
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