From f5de202a03d32e70a2f7451025dd9210faa417be Mon Sep 17 00:00:00 2001
From: Simon Rettberg
Date: Wed, 26 Aug 2020 12:20:42 +0200
Subject: [run-virt] Always use xfwm4 for multi-screen (not just vmware)

While VMware just doesnt work with openbox wrt multi-screen, VBox does
mostly work with openbox, but sometimes, all virtual desktop windows
will end up on the same physical screen, covering each other.
This hasn't been observed yet with xfwm, so let's switch for now. If
this still happens, we need some really ugly hackery involving xdotool.
---
 .../vmchooser/run-virt-includes/start_windowmanager.inc     | 13 +++++++++++--
 1 file changed, 11 insertions(+), 2 deletions(-)

(limited to 'core/modules/run-virt/data/opt')

diff --git a/core/modules/run-virt/data/opt/openslx/vmchooser/run-virt-includes/start_windowmanager.inc b/core/modules/run-virt/data/opt/openslx/vmchooser/run-virt-includes/start_windowmanager.inc
index 473b7f2e..04ee51ec 100644
--- a/core/modules/run-virt/data/opt/openslx/vmchooser/run-virt-includes/start_windowmanager.inc
+++ b/core/modules/run-virt/data/opt/openslx/vmchooser/run-virt-includes/start_windowmanager.inc
@@ -7,10 +7,19 @@
 # using eg. vmware without a window manager.
 start_wm() {
 	# xfwm4 and metacity make the VM appear frozen after openslx.exe
-	# tries to change the resolution
+	# tries to change the resolution. for xfwm4 the fix is to disable
+	# composition. xfwm4 is better suited for mutli-screen. openbox
+	# sometimes puts both virtual screens on the same physical screen,
+	# vmware outright acts up if the window manager doesn't expose the
+	# _NET_WM_FULLSCREEN_MONITORS atom (which openbox doesn't).
+	if [ "$DISPLAY_COUNT" -gt 1 ]; then
+		set -- "$@" xfwm4 openbox
+	else
+		set -- "$@" openbox xfwm4
+	fi
 	local i dm
 	dm=
-	for i in "$@" openbox blackbox kwin; do
+	for i in "$@" blackbox kwin; do
 		if command -v "${i}" &> /dev/null ; then
 			dm="$i"
 			break
-- 
cgit v1.2.3-55-g7522