[virt-tools-list] [PATCH virt-viewer] Fix bug with initial placement of fullscreen windows

Jonathon Jongsma jjongsma at redhat.com
Thu Oct 9 16:21:07 UTC 2014


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);
         }
-- 
1.9.3




More information about the virt-tools-list mailing list