From 9e967d5417d49764276cf121a32c35d5770b1332 Mon Sep 17 00:00:00 2001
From: Julian Smart <julian@anthemion.co.uk>
Date: Tue, 6 Nov 2007 10:52:14 +0000
Subject: [PATCH] Moved desktop margin options to a more sensible place

git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@49678 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775
---
 docs/latex/wx/sysopt.tex | 11 ++---------
 src/common/sizer.cpp     | 11 -----------
 src/gtk/utilsgtk.cpp     | 10 ++++++----
 3 files changed, 8 insertions(+), 24 deletions(-)

diff --git a/docs/latex/wx/sysopt.tex b/docs/latex/wx/sysopt.tex
index a8b885da0a..965b5b2dd4 100644
--- a/docs/latex/wx/sysopt.tex
+++ b/docs/latex/wx/sysopt.tex
@@ -53,6 +53,8 @@ setting it to $0$ makes it always unavailable.}
 environment determination. Supported values are \texttt{GNOME} and \texttt{KDE}.}
 \twocolitem{gtk.window.force-background-colour}{If 1, the backgrounds of windows with the wxBG\_STYLE\_COLOUR background style are cleared forcibly instead
 of relying on the underlying GTK+ window colour. This works around a display problem when running applications under KDE with the gtk-qt theme installed (0.6 and below).}
+\twocolitem{gtk.desktopmargin.x}{The horizontal margin to subtract from the desktop size when Xinerama is not available.}
+\twocolitem{gtk.desktopmargin.y}{The vertical margin to subtract from the desktop size when Xinerama is not available.}
 \end{twocollist}
 
 \wxheading{Mac}
@@ -88,15 +90,6 @@ A reasonable default is used if not specified.}
 \twocolitem{motif.largebuttons}{If 1, uses a bigger default size for wxButtons.}
 \end{twocollist}
 
-\wxheading{All}
-
-\twocolwidtha{7cm}
-\begin{twocollist}\itemsep=0pt
-\twocolitem{{\bf Option}}{{\bf Value}}
-\twocolitem{sizer.desktopmargin.x}{The horizontal margin to subtract from the desktop size when computing the maximum top-level window size in wxSizer::Fit.}
-\twocolitem{sizer.desktopmargin.y}{The vertical margin to subtract from the desktop size when computing the maximum top-level window size in wxSizer::Fit.}
-\end{twocollist}
-
 The compile-time option to include or exclude this functionality
 is wxUSE\_SYSTEM\_OPTIONS.
 
diff --git a/src/common/sizer.cpp b/src/common/sizer.cpp
index 364b173dc1..3b1a77280a 100644
--- a/src/common/sizer.cpp
+++ b/src/common/sizer.cpp
@@ -31,10 +31,6 @@
     #include "wx/toplevel.h"
 #endif // WX_PRECOMP
 
-#if wxUSE_SYSTEM_OPTIONS
-#include "wx/sysopt.h"
-#endif
-
 #include "wx/listimpl.cpp"
 
 
@@ -796,13 +792,6 @@ wxSize wxSizer::Fit( wxWindow *window )
             }
 
             sizeMax = wxDisplay(disp).GetClientArea().GetSize();
-
-#if wxUSE_SYSTEM_OPTIONS
-            int marginX = wxSystemOptions::GetOptionInt(wxT("sizer.desktopmargin.x"));
-            int marginY = wxSystemOptions::GetOptionInt(wxT("sizer.desktopmargin.y"));
-            sizeMax.x -= marginX;
-            sizeMax.y -= marginY;
-#endif
         }
     }
 
diff --git a/src/gtk/utilsgtk.cpp b/src/gtk/utilsgtk.cpp
index 9a3e250f44..1abdee7f37 100644
--- a/src/gtk/utilsgtk.cpp
+++ b/src/gtk/utilsgtk.cpp
@@ -18,11 +18,10 @@
     #include "wx/log.h"
 #endif
 
+#include "wx/sysopt.h"
 #include "wx/apptrait.h"
-
 #include "wx/process.h"
 #include "wx/sysopt.h"
-
 #include "wx/unix/execute.h"
 
 #include "wx/gtk/private/timer.h"
@@ -144,8 +143,11 @@ void *wxGetDisplay()
 
 void wxDisplaySize( int *width, int *height )
 {
-    if (width) *width = gdk_screen_width();
-    if (height) *height = gdk_screen_height();
+    int marginX = wxSystemOptions::GetOptionInt(wxT("gtk.desktopmargin.x"));
+    int marginY = wxSystemOptions::GetOptionInt(wxT("gtk.desktopmargin.y"));
+
+    if (width) *width = gdk_screen_width() - marginX;
+    if (height) *height = gdk_screen_height() - marginY;
 }
 
 void wxDisplaySizeMM( int *width, int *height )
-- 
2.45.2