[virt-viewer 1/3] display: Only queue a resize on zoom-level changes

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

 



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 <fidencio@xxxxxxxxxx>
---
 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
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