Re: [PATCH] window: move window again after fullscreen

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

 



Hi,

On 10/04/2013 06:43 PM, Jonathon Jongsma wrote:
----- Original Message -----
From: "Marc-André Lureau" <marcandre.lureau@xxxxxxxxx>
To: virt-tools-list@xxxxxxxxxx
Sent: Friday, October 4, 2013 8:13:08 AM
Subject:  [PATCH] window: move window again after fullscreen

Some WM get fullscreen position wrong, in some racy ways (metacity on
rhel6 for instance).  Setting window position after entering fullscreen
seems to solve the problem. Though it's better to move before entering
fullscreen, to avoid extra resizes.

https://bugzilla.redhat.com/show_bug.cgi?id=809546
---
  src/virt-viewer-window.c | 3 +++
  1 file changed, 3 insertions(+)

diff --git a/src/virt-viewer-window.c b/src/virt-viewer-window.c
index 0f62feb..d6b1c54 100644
--- a/src/virt-viewer-window.c
+++ b/src/virt-viewer-window.c
@@ -552,6 +552,9 @@ virt_viewer_window_enter_fullscreen(VirtViewerWindow
*self, gint monitor)
      virt_viewer_window_move_to_monitor(self);

      gtk_window_fullscreen(GTK_WINDOW(priv->window));
+
+    /* move again, some WM get fullscreen position wrong, in some racy
ways.. */
+    virt_viewer_window_move_to_monitor(self);
  }

  #define MAX_KEY_COMBO 4
--
1.8.3.1

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



I mentioned this via IRC already, but I suspect that this workaround may not work properly when client monitors are different resolutions due to the fact that we ignore further allocations after the first one when we go into fullscreen mode. Can you get somebody to test this?

From my own experience working on this, I believe you are right, but there
is not a whole lot we can do here. This way at least setups with
identically sized monitors should work. If we get complaints about things
not working on setups with different size monitors, we should probably
file a bug against metacity, rather then to try to hack around this.

Regards,

Hans

_______________________________________________
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