X-Git-Url: https://git.saurik.com/wxWidgets.git/blobdiff_plain/d46342af50c9d995c6a2752f0200907a86d40263..62dc9cb4f159e2d4cac08308b96e63238935e6bb:/src/mac/carbon/window.cpp diff --git a/src/mac/carbon/window.cpp b/src/mac/carbon/window.cpp index 156527a083..ddb8eebf2b 100644 --- a/src/mac/carbon/window.cpp +++ b/src/mac/carbon/window.cpp @@ -75,7 +75,6 @@ BEGIN_EVENT_TABLE(wxWindowMac, wxWindowBase) EVT_ERASE_BACKGROUND(wxWindowMac::OnEraseBackground) EVT_SYS_COLOUR_CHANGED(wxWindowMac::OnSysColourChanged) EVT_INIT_DIALOG(wxWindowMac::OnInitDialog) - EVT_IDLE(wxWindowMac::OnIdle) EVT_SET_FOCUS(wxWindowMac::OnSetFocus) EVT_MOUSE_EVENTS(wxWindowMac::OnMouseEvent) END_EVENT_TABLE() @@ -104,17 +103,7 @@ void wxWindowMac::Init() // generic InitBase(); - // MSW specific - m_doubleClickAllowed = 0; - m_winCaptured = FALSE; - m_isBeingDeleted = FALSE; - - m_useCtl3D = FALSE; - m_mouseInWindow = FALSE; - - m_xThumbSize = 0; - m_yThumbSize = 0; m_backgroundTransparent = FALSE; // as all windows are created with WS_VISIBLE style... @@ -340,7 +329,7 @@ void wxWindowMac::SetDropTarget(wxDropTarget *pDropTarget) if ( m_dropTarget != 0 ) { delete m_dropTarget; } - + m_dropTarget = pDropTarget; if ( m_dropTarget != 0 ) { @@ -398,7 +387,7 @@ bool wxWindowMac::DoPopupMenu(wxMenu *menu, int x, int y) if (item->IsCheckable()) { item->Check( !item->IsChecked() ) ; - } + } menu->SendEvent( id , item->IsCheckable() ? item->IsChecked() : -1 ) ; } menu->MacAfterDisplay( true ) ; @@ -426,7 +415,7 @@ void wxWindowMac::DoScreenToClient(int *x, int *y) const if(x) *x = localwhere.h ; if(y) *y = localwhere.v ; - + MacRootWindowToWindow( x , y ) ; if ( x ) *x -= MacGetLeftBorderSize() ; @@ -500,7 +489,7 @@ bool wxWindowMac::SetCursor(const wxCursor& cursor) { if (m_cursor == cursor) return FALSE; - + if (wxNullCursor == cursor) { if ( ! wxWindowBase::SetCursor( *wxSTANDARD_CURSOR ) ) @@ -511,16 +500,16 @@ bool wxWindowMac::SetCursor(const wxCursor& cursor) if ( ! wxWindowBase::SetCursor( cursor ) ) return FALSE ; } - + wxASSERT_MSG( m_cursor.Ok(), wxT("cursor must be valid after call to the base version")); - + Point pt ; wxWindowMac *mouseWin ; GetMouse( &pt ) ; - + // Change the cursor NOW if we're within the correct window - + if ( MacGetWindowFromPoint( wxPoint( pt.h , pt.v ) , &mouseWin ) ) { if ( mouseWin == this && !wxIsBusy() ) @@ -528,7 +517,7 @@ bool wxWindowMac::SetCursor(const wxCursor& cursor) m_cursor.MacInstall() ; } } - + return TRUE ; } @@ -539,22 +528,22 @@ void wxWindowMac::DoGetClientSize(int *x, int *y) const int ww, hh; ww = m_width ; hh = m_height ; - + ww -= MacGetLeftBorderSize( ) + MacGetRightBorderSize( ) ; hh -= MacGetTopBorderSize( ) + MacGetBottomBorderSize( ); - + if ( (m_vScrollBar && m_vScrollBar->IsShown()) || (m_hScrollBar && m_hScrollBar->IsShown()) ) { int x1 = 0 ; int y1 = 0 ; int w = m_width ; int h = m_height ; - + MacClientToRootWindow( &x1 , &y1 ) ; MacClientToRootWindow( &w , &h ) ; - + wxWindowMac *iter = (wxWindowMac*)this ; - + int totW = 10000 , totH = 10000; while( iter ) { @@ -564,10 +553,10 @@ void wxWindowMac::DoGetClientSize(int *x, int *y) const totH = iter->m_height ; break ; } - + iter = iter->GetParent() ; } - + if (m_hScrollBar && m_hScrollBar->IsShown() ) { hh -= MAC_SCROLLBAR_SIZE; @@ -599,7 +588,7 @@ void wxWindowMac::DoGetClientSize(int *x, int *y) const void wxWindowMac::DoSetToolTip(wxToolTip *tooltip) { wxWindowBase::DoSetToolTip(tooltip); - + if ( m_tooltip ) m_tooltip->SetWindow(this); } @@ -612,12 +601,12 @@ void wxWindowMac::DoMoveWindow(int x, int y, int width, int height) int former_y = m_y ; int former_w = m_width ; int former_h = m_height ; - + int actualWidth = width; int actualHeight = height; int actualX = x; int actualY = y; - + if ((m_minWidth != -1) && (actualWidth < m_minWidth)) actualWidth = m_minWidth; if ((m_minHeight != -1) && (actualHeight < m_minHeight)) @@ -626,10 +615,10 @@ void wxWindowMac::DoMoveWindow(int x, int y, int width, int height) actualWidth = m_maxWidth; if ((m_maxHeight != -1) && (actualHeight > m_maxHeight)) actualHeight = m_maxHeight; - + bool doMove = false ; bool doResize = false ; - + if ( actualX != former_x || actualY != former_y ) { doMove = true ; @@ -638,13 +627,13 @@ void wxWindowMac::DoMoveWindow(int x, int y, int width, int height) { doResize = true ; } - + if ( doMove || doResize ) { // erase former position - + bool partialRepaint = false ; - + if ( HasFlag(wxNO_FULL_REPAINT_ON_RESIZE) ) { wxPoint oldPos( m_x , m_y ) ; @@ -669,25 +658,25 @@ void wxWindowMac::DoMoveWindow(int x, int y, int width, int height) DisposeRgn(diffRgn) ; } } - + if ( !partialRepaint ) Refresh() ; - + m_x = actualX ; m_y = actualY ; m_width = actualWidth ; m_height = actualHeight ; - + // update any low-level frame-relative positions - + MacUpdateDimensions() ; // erase new position - + if ( !partialRepaint ) Refresh() ; if ( doMove ) wxWindowMac::MacSuperChangedPosition() ; // like this only children will be notified - + MacRepositionScrollBars() ; if ( doMove ) { @@ -705,7 +694,7 @@ void wxWindowMac::DoMoveWindow(int x, int y, int width, int height) GetEventHandler()->ProcessEvent(event); } } - + } // set the size of the window: if the dimensions are positive, just use them, @@ -721,10 +710,10 @@ void wxWindowMac::DoSetSize(int x, int y, int width, int height, int sizeFlags) // get the current size and position... int currentX, currentY; GetPosition(¤tX, ¤tY); - + int currentW,currentH; GetSize(¤tW, ¤tH); - + // ... and don't do anything (avoiding flicker) if it's already ok if ( x == currentX && y == currentY && width == currentW && height == currentH ) @@ -732,14 +721,14 @@ void wxWindowMac::DoSetSize(int x, int y, int width, int height, int sizeFlags) MacRepositionScrollBars() ; // we might have a real position shift return; } - + if ( x == -1 && !(sizeFlags & wxSIZE_ALLOW_MINUS_ONE) ) x = currentX; if ( y == -1 && !(sizeFlags & wxSIZE_ALLOW_MINUS_ONE) ) y = currentY; - + AdjustForParentClientOrigin(x, y, sizeFlags); - + wxSize size(-1, -1); if ( width == -1 ) { @@ -754,7 +743,7 @@ void wxWindowMac::DoSetSize(int x, int y, int width, int height, int sizeFlags) width = currentW; } } - + if ( height == -1 ) { if ( sizeFlags & wxSIZE_AUTO_HEIGHT ) @@ -764,7 +753,7 @@ void wxWindowMac::DoSetSize(int x, int y, int width, int height, int sizeFlags) size = DoGetBestSize(); } //else: already called DoGetBestSize() above - + height = size.y; } else @@ -773,9 +762,9 @@ void wxWindowMac::DoSetSize(int x, int y, int width, int height, int sizeFlags) height = currentH; } } - + DoMoveWindow(x, y, width, height); - + } // For implementation purposes - sometimes decorations make the client area // smaller @@ -800,25 +789,9 @@ bool wxWindowMac::Show(bool show) if ( !wxWindowBase::Show(show) ) return FALSE; -/* - WindowRef window = (WindowRef) MacGetRootWindow() ; - wxWindowMac* win = wxFindWinFromMacWindow( window ) ; - if ( win == NULL && win->m_isBeingDeleted ) - return FALSE ; -*/ MacSuperShown( show ) ; Refresh() ; -/* - if ( !show ) - { - if ( win && !win->m_isBeingDeleted ) - Refresh() ; - } - else - { - Refresh() ; - } -*/ + return TRUE; } @@ -827,7 +800,7 @@ void wxWindowMac::MacSuperShown( bool show ) wxWindowListNode *node = GetChildren().GetFirst(); while ( node ) { - wxWindowMac *child = (wxWindowMac *)node->GetData(); + wxWindowMac *child = node->GetData(); if ( child->m_isShown ) child->MacSuperShown( show ) ; node = node->GetNext(); @@ -912,7 +885,10 @@ void wxWindowMac::Refresh(bool eraseBack, const wxRect *rect) if ( MacGetTopLevelWindow() == NULL ) return ; - wxPoint client = GetClientAreaOrigin(); + if ( !MacIsReallyShown() ) + return ; + + wxPoint client = GetClientAreaOrigin(); int x1 = -client.x; int y1 = -client.y; int x2 = m_width - client.x; @@ -1011,9 +987,9 @@ const wxBrush& wxWindowMac::MacGetBackgroundBrush() // on mac we have the difficult situation, that 3dface gray can be different colours, depending whether // it is on a notebook panel or not, in order to take care of that we walk up the hierarchy until we have // either a non gray background color or a non control window - + WindowRef window = (WindowRef) MacGetRootWindow() ; - + wxWindowMac* parent = GetParent() ; while( parent ) { @@ -1023,7 +999,7 @@ const wxBrush& wxWindowMac::MacGetBackgroundBrush() parent = NULL ; break ; } - + { if ( parent->m_backgroundColour != wxSystemSettings::GetColour(wxSYS_COLOUR_3DFACE ) && parent->m_backgroundColour != wxSystemSettings::GetColour(wxSYS_COLOUR_APPWORKSPACE) ) @@ -1060,8 +1036,8 @@ const wxBrush& wxWindowMac::MacGetBackgroundBrush() { m_macBackgroundBrush.SetColour( m_backgroundColour ) ; } - - return m_macBackgroundBrush ; + + return m_macBackgroundBrush ; } void wxWindowMac::OnEraseBackground(wxEraseEvent& event) @@ -1257,11 +1233,11 @@ void wxWindowMac::ScrollWindow(int dx, int dy, const wxRect *rect) { wxClientDC dc(this) ; wxMacPortSetter helper(&dc) ; - + { int width , height ; GetClientSize( &width , &height ) ; - + Rect scrollrect = { dc.YLOG2DEVMAC(0) , dc.XLOG2DEVMAC(0) , dc.YLOG2DEVMAC(height) , dc.XLOG2DEVMAC(width) } ; RgnHandle updateRgn = NewRgn() ; ClipRect( &scrollrect ) ; @@ -1275,21 +1251,21 @@ void wxWindowMac::ScrollWindow(int dx, int dy, const wxRect *rect) InvalWindowRgn( (WindowRef) MacGetRootWindow() , updateRgn ) ; DisposeRgn( updateRgn ) ; } - + for (wxWindowListNode *node = GetChildren().GetFirst(); node; node = node->GetNext()) { - wxWindowMac *child = (wxWindowMac*)node->GetData(); + wxWindowMac *child = node->GetData(); if (child == m_vScrollBar) continue; if (child == m_hScrollBar) continue; if (child->IsTopLevel()) continue; - + int x,y; child->GetPosition( &x, &y ); int w,h; child->GetSize( &w, &h ); child->SetSize( x+dx, y+dy, w, h ); } - + } void wxWindowMac::MacOnScroll(wxScrollEvent &event ) @@ -1300,30 +1276,25 @@ void wxWindowMac::MacOnScroll(wxScrollEvent &event ) wevent.SetPosition(event.GetPosition()); wevent.SetOrientation(event.GetOrientation()); wevent.m_eventObject = this; - - if (event.m_eventType == wxEVT_SCROLL_TOP) { + + if (event.m_eventType == wxEVT_SCROLL_TOP) wevent.m_eventType = wxEVT_SCROLLWIN_TOP; - } else - if (event.m_eventType == wxEVT_SCROLL_BOTTOM) { - wevent.m_eventType = wxEVT_SCROLLWIN_BOTTOM; - } else - if (event.m_eventType == wxEVT_SCROLL_LINEUP) { - wevent.m_eventType = wxEVT_SCROLLWIN_LINEUP; - } else - if (event.m_eventType == wxEVT_SCROLL_LINEDOWN) { - wevent.m_eventType = wxEVT_SCROLLWIN_LINEDOWN; - } else - if (event.m_eventType == wxEVT_SCROLL_PAGEUP) { - wevent.m_eventType = wxEVT_SCROLLWIN_PAGEUP; - } else - if (event.m_eventType == wxEVT_SCROLL_PAGEDOWN) { - wevent.m_eventType = wxEVT_SCROLLWIN_PAGEDOWN; - } else - if (event.m_eventType == wxEVT_SCROLL_THUMBTRACK) { - wevent.m_eventType = wxEVT_SCROLLWIN_THUMBTRACK; - } - - GetEventHandler()->ProcessEvent(wevent); + else if (event.m_eventType == wxEVT_SCROLL_BOTTOM) + wevent.m_eventType = wxEVT_SCROLLWIN_BOTTOM; + else if (event.m_eventType == wxEVT_SCROLL_LINEUP) + wevent.m_eventType = wxEVT_SCROLLWIN_LINEUP; + else if (event.m_eventType == wxEVT_SCROLL_LINEDOWN) + wevent.m_eventType = wxEVT_SCROLLWIN_LINEDOWN; + else if (event.m_eventType == wxEVT_SCROLL_PAGEUP) + wevent.m_eventType = wxEVT_SCROLLWIN_PAGEUP; + else if (event.m_eventType == wxEVT_SCROLL_PAGEDOWN) + wevent.m_eventType = wxEVT_SCROLLWIN_PAGEDOWN; + else if (event.m_eventType == wxEVT_SCROLL_THUMBTRACK) + wevent.m_eventType = wxEVT_SCROLLWIN_THUMBTRACK; + else if (event.m_eventType == wxEVT_SCROLL_THUMBRELEASE) + wevent.m_eventType = wxEVT_SCROLLWIN_THUMBRELEASE; + + GetEventHandler()->ProcessEvent(wevent); } } @@ -1372,7 +1343,7 @@ void wxWindowMac::OnSetFocus(wxFocusEvent& event) // notice that it's also important to do it upwards the tree becaus // otherwise when the top level panel gets focus, it won't set it back to // us, but to some other sibling - + // CS:don't know if this is still needed: //wxChildFocusEvent eventFocus(this); //(void)GetEventHandler()->ProcessEvent(eventFocus); @@ -1395,26 +1366,12 @@ void wxWindowMac::SetupColours() SetBackgroundColour(GetParent()->GetBackgroundColour()); } -void wxWindowMac::OnIdle(wxIdleEvent& event) +void wxWindowMac::OnInternalIdle() { -/* - // Check if we need to send a LEAVE event - if (m_mouseInWindow) - { - POINT pt; - ::GetCursorPos(&pt); - if (::WindowFromPoint(pt) != (HWND) GetHWND()) - { - // Generate a LEAVE event - m_mouseInWindow = FALSE; - MSWOnMouseLeave(pt.x, pt.y, 0); - } - } -*/ - // This calls the UI-update mechanism (querying windows for // menu/toolbar/control state information) - UpdateWindowUI(); + if (wxUpdateUIEvent::CanUpdate(this)) + UpdateWindowUI(wxUPDATE_UI_FROMIDLE); } // Raise the window to the top of the Z order @@ -1461,20 +1418,20 @@ bool wxWindowMac::MacGetWindowFromPointSub( const wxPoint &point , wxWindowMac** (point.x > (m_x + m_width)) || (point.y > (m_y + m_height))) return FALSE; } - + WindowRef window = (WindowRef) MacGetRootWindow() ; - + wxPoint newPoint( point ) ; - + if ( !IsTopLevel() ) { newPoint.x -= m_x; newPoint.y -= m_y; } - + for (wxWindowListNode *node = GetChildren().GetFirst(); node; node = node->GetNext()) { - wxWindowMac *child = (wxWindowMac*)node->GetData(); + wxWindowMac *child = node->GetData(); // added the m_isShown test --dmazzoni if ( child->MacGetRootWindow() == window && child->m_isShown ) { @@ -1482,7 +1439,7 @@ bool wxWindowMac::MacGetWindowFromPointSub( const wxPoint &point , wxWindowMac** return TRUE; } } - + *outWin = this ; return TRUE; } @@ -1490,25 +1447,25 @@ bool wxWindowMac::MacGetWindowFromPointSub( const wxPoint &point , wxWindowMac** bool wxWindowMac::MacGetWindowFromPoint( const wxPoint &screenpoint , wxWindowMac** outWin ) { WindowRef window ; - + Point pt = { screenpoint.y , screenpoint.x } ; if ( ::FindWindow( pt , &window ) == 3 ) { - + wxWindowMac* win = wxFindWinFromMacWindow( window ) ; if ( win ) { // No, this yields the CLIENT are, we need the whole frame. RR. // point = win->ScreenToClient( point ) ; - + GrafPtr port; ::GetPort( &port ) ; ::SetPort( UMAGetWindowPort( window ) ) ; ::GlobalToLocal( &pt ) ; ::SetPort( port ) ; - + wxPoint point( pt.h, pt.v ) ; - + return win->MacGetWindowFromPointSub( point , outWin ) ; } } @@ -1520,14 +1477,14 @@ static wxWindow *gs_lastWhich = NULL; bool wxWindowMac::MacSetupCursor( const wxPoint& pt) { // first trigger a set cursor event - + wxPoint clientorigin = GetClientAreaOrigin() ; wxSize clientsize = GetClientSize() ; wxCursor cursor ; if ( wxRect2DInt( clientorigin.x , clientorigin.y , clientsize.x , clientsize.y ).Contains( wxPoint2DInt( pt ) ) ) { wxSetCursorEvent event( pt.x , pt.y ); - + bool processedEvtSetCursor = GetEventHandler()->ProcessEvent(event); if ( processedEvtSetCursor && event.HasCursor() ) { @@ -1535,7 +1492,7 @@ bool wxWindowMac::MacSetupCursor( const wxPoint& pt) } else { - + // the test for processedEvtSetCursor is here to prevent using m_cursor // if the user code caught EVT_SET_CURSOR() and returned nothing from // it - this is a way to say that our cursor shouldn't be used for this @@ -1564,32 +1521,32 @@ bool wxWindowMac::MacDispatchMouseEvent(wxMouseEvent& event) if ((event.m_x < m_x) || (event.m_y < m_y) || (event.m_x > (m_x + m_width)) || (event.m_y > (m_y + m_height))) return FALSE; - - + + if ( IsKindOf( CLASSINFO ( wxStaticBox ) ) /* || IsKindOf( CLASSINFO( wxSpinCtrl ) ) */) return FALSE ; - + WindowRef window = (WindowRef) MacGetRootWindow() ; - + event.m_x -= m_x; event.m_y -= m_y; - + int x = event.m_x ; int y = event.m_y ; - + for (wxWindowListNode *node = GetChildren().GetFirst(); node; node = node->GetNext()) { - wxWindowMac *child = (wxWindowMac*)node->GetData(); + wxWindowMac *child = node->GetData(); if ( child->MacGetRootWindow() == window && child->IsShown() && child->IsEnabled() ) { if (child->MacDispatchMouseEvent(event)) return TRUE; } } - + wxWindow* cursorTarget = this ; wxPoint cursorPoint( x , y ) ; - + while( cursorTarget && !cursorTarget->MacSetupCursor( cursorPoint ) ) { cursorTarget = cursorTarget->GetParent() ; @@ -1599,40 +1556,40 @@ bool wxWindowMac::MacDispatchMouseEvent(wxMouseEvent& event) event.m_x = x ; event.m_y = y ; event.SetEventObject( this ) ; - + if ( event.GetEventType() == wxEVT_LEFT_DOWN ) { // set focus to this window if (AcceptsFocus() && FindFocus()!=this) SetFocus(); } - + #if wxUSE_TOOLTIPS if ( event.GetEventType() == wxEVT_MOTION || event.GetEventType() == wxEVT_ENTER_WINDOW || event.GetEventType() == wxEVT_LEAVE_WINDOW ) wxToolTip::RelayEvent( this , event); #endif // wxUSE_TOOLTIPS - + if (gs_lastWhich != this) { gs_lastWhich = this; - + // Double clicks must always occur on the same window if (event.GetEventType() == wxEVT_LEFT_DCLICK) event.SetEventType( wxEVT_LEFT_DOWN ); if (event.GetEventType() == wxEVT_RIGHT_DCLICK) event.SetEventType( wxEVT_RIGHT_DOWN ); - + // Same for mouse up events if (event.GetEventType() == wxEVT_LEFT_UP) return TRUE; if (event.GetEventType() == wxEVT_RIGHT_UP) return TRUE; } - + GetEventHandler()->ProcessEvent( event ) ; - + return TRUE; } @@ -1642,7 +1599,7 @@ wxString wxWindowMac::MacGetToolTipString( wxPoint &pt ) { return m_tooltip->GetTip() ; } - return "" ; + return wxEmptyString ; } void wxWindowMac::Update() @@ -1676,19 +1633,19 @@ const wxRegion& wxWindowMac::MacGetVisibleRegion( bool respectChildrenAndSibling RgnHandle visRgn = NewRgn() ; RgnHandle tempRgn = NewRgn() ; RgnHandle tempStaticBoxRgn = NewRgn() ; - + SetRectRgn( visRgn , 0 , 0 , m_width , m_height ) ; - + //TODO : as soon as the new scheme has proven to work correctly, move this to wxStaticBox if ( IsKindOf( CLASSINFO( wxStaticBox ) ) ) { int borderTop = 14 ; int borderOther = 4 ; - + SetRectRgn( tempStaticBoxRgn , borderOther , borderTop , m_width - borderOther , m_height - borderOther ) ; DiffRgn( visRgn , tempStaticBoxRgn , visRgn ) ; } - + if ( !IsTopLevel() ) { wxWindow* parent = GetParent() ; @@ -1699,12 +1656,12 @@ const wxRegion& wxWindowMac::MacGetVisibleRegion( bool respectChildrenAndSibling x = y = 0 ; parent->MacWindowToRootWindow( &x, &y ) ; MacRootWindowToWindow( &x , &y ) ; - + SetRectRgn( tempRgn , x + parent->MacGetLeftBorderSize() , y + parent->MacGetTopBorderSize() , x + size.x - parent->MacGetRightBorderSize(), y + size.y - parent->MacGetBottomBorderSize()) ; - + SectRgn( visRgn , tempRgn , visRgn ) ; if ( parent->IsTopLevel() ) break ; @@ -1717,8 +1674,8 @@ const wxRegion& wxWindowMac::MacGetVisibleRegion( bool respectChildrenAndSibling { for (wxWindowListNode *node = GetChildren().GetFirst(); node; node = node->GetNext()) { - wxWindowMac *child = (wxWindowMac*)node->GetData(); - + wxWindowMac *child = node->GetData(); + if ( !child->IsTopLevel() && child->IsShown() ) { SetRectRgn( tempRgn , child->m_x , child->m_y , child->m_x + child->m_width , child->m_y + child->m_height ) ; @@ -1726,7 +1683,7 @@ const wxRegion& wxWindowMac::MacGetVisibleRegion( bool respectChildrenAndSibling { int borderTop = 14 ; int borderOther = 4 ; - + SetRectRgn( tempStaticBoxRgn , child->m_x + borderOther , child->m_y + borderTop , child->m_x + child->m_width - borderOther , child->m_y + child->m_height - borderOther ) ; DiffRgn( tempRgn , tempStaticBoxRgn , tempRgn ) ; } @@ -1734,13 +1691,13 @@ const wxRegion& wxWindowMac::MacGetVisibleRegion( bool respectChildrenAndSibling } } } - + if ( (GetWindowStyle() & wxCLIP_SIBLINGS) && GetParent() ) { bool thisWindowThrough = false ; for (wxWindowListNode *node = GetParent()->GetChildren().GetFirst(); node; node = node->GetNext()) { - wxWindowMac *sibling = (wxWindowMac*)node->GetData(); + wxWindowMac *sibling = node->GetData(); if ( sibling == this ) { thisWindowThrough = true ; @@ -1750,7 +1707,7 @@ const wxRegion& wxWindowMac::MacGetVisibleRegion( bool respectChildrenAndSibling { continue ; } - + if ( !sibling->IsTopLevel() && sibling->IsShown() ) { SetRectRgn( tempRgn , sibling->m_x - m_x , sibling->m_y - m_y , sibling->m_x + sibling->m_width - m_x , sibling->m_y + sibling->m_height - m_y ) ; @@ -1758,7 +1715,7 @@ const wxRegion& wxWindowMac::MacGetVisibleRegion( bool respectChildrenAndSibling { int borderTop = 14 ; int borderOther = 4 ; - + SetRectRgn( tempStaticBoxRgn , sibling->m_x - m_x + borderOther , sibling->m_y - m_y + borderTop , sibling->m_x + sibling->m_width - m_x - borderOther , sibling->m_y + sibling->m_height - m_y - borderOther ) ; DiffRgn( tempRgn , tempStaticBoxRgn , tempRgn ) ; } @@ -1779,16 +1736,16 @@ void wxWindowMac::MacRedraw( WXHRGN updatergnr , long time, bool erase) RgnHandle updatergn = (RgnHandle) updatergnr ; // updatergn is always already clipped to our boundaries // it is in window coordinates, not in client coordinates - + WindowRef window = (WindowRef) MacGetRootWindow() ; - + { // ownUpdateRgn is the area that this window has to repaint, it is in window coordinates RgnHandle ownUpdateRgn = NewRgn() ; CopyRgn( updatergn , ownUpdateRgn ) ; - + SectRgn( ownUpdateRgn , (RgnHandle) MacGetVisibleRegion().GetWXHRGN() , ownUpdateRgn ) ; - + // newupdate is the update region in client coordinates RgnHandle newupdate = NewRgn() ; wxSize point = GetClientSize() ; @@ -1798,7 +1755,7 @@ void wxWindowMac::MacRedraw( WXHRGN updatergnr , long time, bool erase) OffsetRgn( newupdate , -origin.x , -origin.y ) ; m_updateRegion = newupdate ; DisposeRgn( newupdate ) ; // it's been cloned to m_updateRegion - + if ( erase && !EmptyRgn(ownUpdateRgn) ) { wxWindowDC dc(this); @@ -1807,7 +1764,7 @@ void wxWindowMac::MacRedraw( WXHRGN updatergnr , long time, bool erase) wxEraseEvent eevent( GetId(), &dc ); eevent.SetEventObject( this ); GetEventHandler()->ProcessEvent( eevent ); - + wxNcPaintEvent eventNc( GetId() ); eventNc.SetEventObject( this ); GetEventHandler()->ProcessEvent( eventNc ); @@ -1821,15 +1778,15 @@ void wxWindowMac::MacRedraw( WXHRGN updatergnr , long time, bool erase) GetEventHandler()->ProcessEvent(event); } } - + // now intersect for each of the children their rect with the updateRgn and call MacRedraw recursively - + RgnHandle childupdate = NewRgn() ; for (wxWindowListNode *node = GetChildren().GetFirst(); node; node = node->GetNext()) { // calculate the update region for the child windows by intersecting the window rectangle with our own // passed in update region and then offset it to be client-wise window coordinates again - wxWindowMac *child = (wxWindowMac*)node->GetData(); + wxWindowMac *child = node->GetData(); SetRectRgn( childupdate , child->m_x , child->m_y , child->m_x + child->m_width , child->m_y + child->m_height ) ; SectRgn( childupdate , updatergn , childupdate ) ; OffsetRgn( childupdate , -child->m_x , -child->m_y ) ; @@ -1841,7 +1798,7 @@ void wxWindowMac::MacRedraw( WXHRGN updatergnr , long time, bool erase) } DisposeRgn( childupdate ) ; // eventually a draw grow box here - + } WXHWND wxWindowMac::MacGetRootWindow() const @@ -1855,13 +1812,13 @@ WXHWND wxWindowMac::MacGetRootWindow() const iter = iter->GetParent() ; } - wxASSERT_MSG( 1 , "No valid mac root window" ) ; + wxASSERT_MSG( 1 , wxT("No valid mac root window") ) ; return NULL ; } void wxWindowMac::MacCreateScrollBars( long style ) { - wxASSERT_MSG( m_vScrollBar == NULL && m_hScrollBar == NULL , "attempt to create window twice" ) ; + wxASSERT_MSG( m_vScrollBar == NULL && m_hScrollBar == NULL , wxT("attempt to create window twice") ) ; bool hasBoth = ( style & wxVSCROLL ) && ( style & wxHSCROLL ) ; int adjust = hasBoth ? MAC_SCROLLBAR_SIZE - 1: 0 ; @@ -1989,7 +1946,7 @@ void wxWindowMac::MacSuperChangedPosition() wxWindowListNode *node = GetChildren().GetFirst(); while ( node ) { - wxWindowMac *child = (wxWindowMac *)node->GetData(); + wxWindowMac *child = node->GetData(); child->MacSuperChangedPosition() ; node = node->GetNext(); } @@ -2002,7 +1959,7 @@ void wxWindowMac::MacTopLevelWindowChangedPosition() wxWindowListNode *node = GetChildren().GetFirst(); while ( node ) { - wxWindowMac *child = (wxWindowMac *)node->GetData(); + wxWindowMac *child = node->GetData(); child->MacTopLevelWindowChangedPosition() ; node = node->GetNext(); } @@ -2079,12 +2036,12 @@ wxPoint wxGetMousePosition() return wxPoint(x, y); } -void wxWindowMac::OnMouseEvent( wxMouseEvent &event ) +void wxWindowMac::OnMouseEvent( wxMouseEvent &event ) { if ( event.GetEventType() == wxEVT_RIGHT_DOWN ) { // copied from wxGTK : CS - // generate a "context menu" event: this is similar to wxEVT_RIGHT_UP + // generate a "context menu" event: this is similar to wxEVT_RIGHT_DOWN // except that: // // (a) it's a command event and so is propagated to the parent @@ -2093,7 +2050,8 @@ void wxWindowMac::OnMouseEvent( wxMouseEvent &event ) wxContextMenuEvent evtCtx(wxEVT_CONTEXT_MENU, this->GetId(), this->ClientToScreen(event.GetPosition())); - this->GetEventHandler()->ProcessEvent(evtCtx); + if ( ! GetEventHandler()->ProcessEvent(evtCtx) ) + event.Skip() ; } else {