make XPM data pointer parameter fully const

[wxWidgets.git] / src / mac / carbon / dialog.cpp

diff --git a/src/mac/carbon/dialog.cpp b/src/mac/carbon/dialog.cpp
index 7c984cea0c10d8044631223b9fc946145892b80e..858736fd33b960265b67d24b4fbca90d59dc3a6c 100644 (file)
--- a/src/mac/carbon/dialog.cpp
+++ b/src/mac/carbon/dialog.cpp
@@ -53,13 +53,6 @@ bool wxDialog::Create( wxWindow *parent,
     if ( !wxTopLevelWindow::Create( parent, id, title, pos, size, style, name ) )
         return false;
 
     if ( !wxTopLevelWindow::Create( parent, id, title, pos, size, style, name ) )
         return false;
 

-#if TARGET_API_MAC_OSX
-    HIViewRef growBoxRef = 0 ;
-    OSStatus err = HIViewFindByID( HIViewGetRoot( (WindowRef)m_macWindow ), kHIViewWindowGrowBoxID, &growBoxRef  );
-    if ( err == noErr && growBoxRef != 0 )
-        HIGrowBoxViewSetTransparent( growBoxRef, true ) ;
-#endif
-

     return true;
 }
 
     return true;
 }
 


@@ -71,9 +64,7 @@ void wxDialog::SetModal( bool flag )
 
         wxModelessWindows.DeleteObject( this );
 
 
         wxModelessWindows.DeleteObject( this );
 

-#if TARGET_CARBON

         SetWindowModality( (WindowRef)MacGetWindowRef(), kWindowModalityAppModal, NULL ) ;
         SetWindowModality( (WindowRef)MacGetWindowRef(), kWindowModalityAppModal, NULL ) ;

-#endif

     }
     else
     {
     }
     else
     {


@@ -112,10 +103,15 @@ bool wxDialog::Show(bool show)
         // nothing to do
         return false;
 
         // nothing to do
         return false;
 

+    if (show && CanDoLayoutAdaptation())
+        DoLayoutAdaptation();
+

     if ( show )
         // usually will result in TransferDataToWindow() being called
         InitDialog();
 
     if ( show )
         // usually will result in TransferDataToWindow() being called
         InitDialog();
 

+    HiliteMenu(0);
+

     if ( m_isModalStyle )
     {
         if ( show )
     if ( m_isModalStyle )
     {
         if ( show )


@@ -133,38 +129,39 @@ bool wxDialog::Show(bool show)
     return true;
 }
 
     return true;
 }
 

-#if !TARGET_CARBON
-extern bool s_macIsInModalLoop ;
-#endif
-

 void wxDialog::DoShowModal()
 {
 void wxDialog::DoShowModal()
 {

-    wxCHECK_RET( !IsModalShowing(), wxT("DoShowModal() called twice") );
+    wxCHECK_RET( !IsModal(), wxT("DoShowModal() called twice") );

 
     wxModalDialogs.Append(this);
 
     SetFocus() ;
 
 
     wxModalDialogs.Append(this);
 
     SetFocus() ;
 

-#if TARGET_CARBON
-    BeginAppModalStateForWindow(  (WindowRef) MacGetWindowRef()) ;
-#else
-    // TODO : test whether parent gets disabled
-    bool formerModal = s_macIsInModalLoop ;
-    s_macIsInModalLoop = true ;
-#endif
+    WindowRef windowRef = (WindowRef) MacGetWindowRef();
+    WindowGroupRef windowGroup;
+    WindowGroupRef formerParentGroup;
+    bool resetGroupParent = false;

 
 

-    while ( IsModalShowing() )
+    if ( GetParent() == NULL )
+    {
+        windowGroup = GetWindowGroup(windowRef) ;
+        formerParentGroup = GetWindowGroupParent( windowGroup );
+        SetWindowGroupParent( windowGroup, GetWindowGroupOfClass( kMovableModalWindowClass ) );
+        resetGroupParent = true;
+    }
+    BeginAppModalStateForWindow(windowRef) ;
+
+    while ( IsModal() )

     {
         wxTheApp->MacDoOneEvent() ;
         // calls process idle itself
     }
 
     {
         wxTheApp->MacDoOneEvent() ;
         // calls process idle itself
     }
 

-#if TARGET_CARBON
-    EndAppModalStateForWindow( (WindowRef) MacGetWindowRef() ) ;
-#else
-    // TODO probably reenable the parent window if any
-    s_macIsInModalLoop = formerModal ;
-#endif
+    EndAppModalStateForWindow(windowRef) ;
+    if ( resetGroupParent )
+    {
+        SetWindowGroupParent( windowGroup , formerParentGroup );
+    }

 }
 
 
 }