From: Michael Bedward <mbedward@ozemail.com.au>
Date: Mon, 9 Aug 1999 07:23:59 +0000 (+0000)
Subject: wxFrame::Create now sets (arbitrary) values for position and size if default values... 
X-Git-Url: https://git.saurik.com/wxWidgets.git/commitdiff_plain/447a039fa338288a974de5e74242feae5755a1fb

wxFrame::Create now sets (arbitrary) values for position and size if default values of -1 have been used


git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@3323 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775
---

diff --git a/src/motif/frame.cpp b/src/motif/frame.cpp
index 121e0bae04..82725c86d7 100644
--- a/src/motif/frame.cpp
+++ b/src/motif/frame.cpp
@@ -144,9 +144,31 @@ bool wxFrame::Create(wxWindow *parent,
 
     wxModelessWindows.Append(this);
 
-    int x = pos.x; int y = pos.y;
-    int width = size.x; int height = size.y;
-
+    int x = pos.x, y = pos.y;
+    int width = size.x, height = size.y;
+    
+    // Set reasonable values for position and size if defaults have
+    // been requested 
+    // 
+    // MB TODO: something better than these arbitrary values ?
+    //
+    if ( width == -1 ) width = 400;
+    if ( height = -1 ) height = 400;
+
+    int displayW, displayH;
+    wxDisplaySize( &displayW, &displayH );
+
+    if ( x == -1 )
+    {
+        x = (displayW - width) / 2;
+        if (x < 10) x = 10;
+    }
+    if ( y == -1 )
+    {
+        y = (displayH - height) / 2;
+        if (y < 10) y = 10;        
+    }
+        
     if (wxTopLevelUsed)
     {
         // Change suggested by Matthew Flatt