Remove /size: parameter conflict with /monitors: for better compatibility
Issue: /monitors:1,2 with /size:3840x1080 still shows only single monitor Analysis: /size: parameter may conflict with /monitors: in FreeRDP 2.11.5 New approach: - Use /monitors:1,2 WITHOUT /size: parameter - Let /monitors: parameter handle monitor layout and sizing automatically - Remove /size: from command when using specific monitor selection - Keep /size: only for single monitor or manual resolution settings Expected behavior: - /monitors:1,2 should automatically span across monitors 1,2 - Remove potential parameter conflicts causing single monitor fallback - Maintain fullscreen-like behavior through monitor spanning Test command change: Before: xfreerdp /size:3840x1080 /monitors:1,2 After: xfreerdp /monitors:1,2 This follows FreeRDP documentation that /monitors: should handle sizing.
This commit is contained in:
@@ -917,10 +917,16 @@ Multi-Monitor Support:
|
||||
color_depth = conn.get("color_depth", 32)
|
||||
cmd.append(f"/bpp:{color_depth}")
|
||||
|
||||
# Multiple monitors - use /monitors: for specific selection
|
||||
# Multiple monitors - try different approaches for better compatibility
|
||||
if use_specific_monitors:
|
||||
# Method 1: Try /monitors: without /size for better compatibility
|
||||
cmd.append(f"/monitors:{monitor_list}")
|
||||
self.logger.info(f"Using specific monitors for {multimon}: {monitor_list}")
|
||||
self.logger.info(f"Using specific monitors for {multimon}: {monitor_list} (Method 1: /monitors: only)")
|
||||
|
||||
# Remove the /size: parameter we added earlier - let /monitors: handle sizing
|
||||
cmd = [x for x in cmd if not x.startswith('/size:')]
|
||||
self.logger.info("Removed /size: parameter to avoid conflict with /monitors:")
|
||||
|
||||
elif multimon == "All Monitors":
|
||||
cmd.append("/multimon")
|
||||
self.logger.info("Using all available monitors")
|
||||
|
||||
Reference in New Issue
Block a user