By the comment in the code:
"For the gtk2 build, we need to queue a resize even if the zoom level
 hasn't changed.  This is due to the fact that VirtViewerWindow will
 queue a resize event for itself immediately after calling this
 function (in  order to shrink the window to fit the new display size
 if necessary). If we don't queue a resize here, the window will become
 tiny because we will only request 50x50 during the window resize."

And it doesn't happen on gtk3 at all. So, let's just remove the comment
and just quere the resize when zoom-changes actually happen.

Signed-off-by: Fabiano Fidêncio <[email protected]>
---
 src/virt-viewer-display.c | 9 +--------
 1 file changed, 1 insertion(+), 8 deletions(-)

diff --git a/src/virt-viewer-display.c b/src/virt-viewer-display.c
index 16fb930..c2e6c4d 100644
--- a/src/virt-viewer-display.c
+++ b/src/virt-viewer-display.c
@@ -489,19 +489,12 @@ void virt_viewer_display_set_zoom_level(VirtViewerDisplay 
*display,
     if (zoom > MAX_ZOOM_LEVEL)
         zoom = MAX_ZOOM_LEVEL;
 
-    // For the gtk2 build, we need to queue a resize even if the zoom level
-    // hasn't changed.  This is due to the fact that VirtViewerWindow will 
queue
-    // a resize event for itself immediately after calling this function (in
-    // order to shrink the window to fit the new display size if necessary). If
-    // we don't queue a resize here, the window will become tiny because we 
will
-    // only request 50x50 during the window resize
-    virt_viewer_display_queue_resize(display);
-
     if (priv->zoom_level == zoom)
         return;
 
     priv->zoom_level = zoom;
 
+    virt_viewer_display_queue_resize(display);
     g_object_notify(G_OBJECT(display), "zoom-level");
 }
 
-- 
2.7.4

_______________________________________________
virt-tools-list mailing list
[email protected]
https://www.redhat.com/mailman/listinfo/virt-tools-list

Reply via email to