From c131829a1533b1a3242cde4836b31956f1e409a0 Mon Sep 17 00:00:00 2001
From: kervala <kervala@localhost>
Date: Thu, 18 Nov 2010 15:04:18 +0100
Subject: [PATCH] Fixed: #1198 Window is too large when switching from
 fullscreen to windowed mode

---
 .../src/interface_v3/action_handler_game.cpp  | 44 +++++++++++++++----
 1 file changed, 36 insertions(+), 8 deletions(-)

diff --git a/code/ryzom/client/src/interface_v3/action_handler_game.cpp b/code/ryzom/client/src/interface_v3/action_handler_game.cpp
index c07081704..e44516aba 100644
--- a/code/ryzom/client/src/interface_v3/action_handler_game.cpp
+++ b/code/ryzom/client/src/interface_v3/action_handler_game.cpp
@@ -3217,18 +3217,46 @@ class CHandlerGameConfigApply : public IActionHandler
 				}
 
 				ClientCfg.Windowed = !bFullscreen;
-				// Write the modified client.cfg
-				ClientCfg.writeInt("FullScreen", bFullscreen?1:0);
+
+				UDriver::CMode screenMode;
+				Driver->getCurrentScreenMode(screenMode);
 
 				if (bFullscreen)
 				{
-					ClientCfg.Width = w;
-					ClientCfg.Height = h;
-					ClientCfg.Depth = GAME_CONFIG_VIDEO_DEPTH_REQ;
+					ClientCfg.Depth = screenMode.Depth;
 					ClientCfg.Frequency = freq;
-					ClientCfg.writeInt("Width", w);
-					ClientCfg.writeInt("Height", h);
-					ClientCfg.writeInt("Depth", GAME_CONFIG_VIDEO_DEPTH_REQ);
+				}
+				else
+				{
+					uint32 width, height;
+					Driver->getWindowSize(width, height);
+
+					// window is too large
+					if (width >= screenMode.Width || height >= screenMode.Height)
+					{
+						// choose a smaller size
+						w = 1024;
+						h = 768;
+					}
+					else
+					{
+						// take previous mode
+						w = width;
+						h = height;
+					}
+				}
+
+				ClientCfg.Width = w;
+				ClientCfg.Height = h;
+
+				// Write the modified client.cfg
+				ClientCfg.writeBool("FullScreen", bFullscreen);
+				ClientCfg.writeInt("Width", w);
+				ClientCfg.writeInt("Height", h);
+
+				if (bFullscreen)
+				{
+					ClientCfg.writeInt("Depth", screenMode.Depth);
 					ClientCfg.writeInt("Frequency", freq);
 				}
 			}