X-Git-Url: https://git.saurik.com/wxWidgets.git/blobdiff_plain/5d44b24ee63543390aa28e1eb4ff162da4264838..d0332cbc07d487c30d83515fb3351790565ec1de:/src/os2/radiobut.cpp diff --git a/src/os2/radiobut.cpp b/src/os2/radiobut.cpp index 83444cd1f0..677b804d7a 100644 --- a/src/os2/radiobut.cpp +++ b/src/os2/radiobut.cpp @@ -1,5 +1,5 @@ ///////////////////////////////////////////////////////////////////////////// -// Name: radiobut.cpp +// Name: src/os2/radiobut.cpp // Purpose: wxRadioButton // Author: David Webster // Modified by: @@ -13,137 +13,123 @@ #include "wx/wxprec.h" #ifdef __BORLANDC__ -#pragma hdrstop + #pragma hdrstop #endif -#ifndef WX_PRECOMP -#include -#include "wx/setup.h" #include "wx/radiobut.h" -#include "wx/brush.h" + +#ifndef WX_PRECOMP + #include + #include "wx/brush.h" + #include "wx/dcscreen.h" + #include "wx/settings.h" #endif #include "wx/os2/private.h" IMPLEMENT_DYNAMIC_CLASS(wxRadioButton, wxControl) -void wxRadioButton::Command ( - wxCommandEvent& rEvent -) +extern void wxAssociateWinWithHandle( HWND hWnd + ,wxWindowOS2* pWin + ); + +void wxRadioButton::Init() +{ + m_bFocusJustSet = false; +} // end of wxRadioButton::Init + +void wxRadioButton::Command ( wxCommandEvent& rEvent ) { SetValue ((rEvent.GetInt() != 0) ); ProcessCommand (rEvent); } // end of wxRadioButton::Command -bool wxRadioButton::Create( - wxWindow* pParent -, wxWindowID vId -, const wxString& rsLabel -, const wxPoint& rPos -, const wxSize& rSize -, long lStyle -#if wxUSE_VALIDATORS -, const wxValidator& rValidator -#endif -, const wxString& rsName -) +bool wxRadioButton::Create( wxWindow* pParent, + wxWindowID vId, + const wxString& rsLabel, + const wxPoint& rPos, + const wxSize& rSize, + long lStyle, + const wxValidator& rValidator, + const wxString& rsName ) { - int nX = rPos.x; - int nY = rPos.y; - int nWidth = rSize.x; - int nHeight = rSize.y; - long lsStyle = 0L; - long lGroupStyle = 0L; - - SetName(rsName); -#if wxUSE_VALIDATORS - SetValidator(rValidator); -#endif + if ( !CreateControl( pParent + ,vId + ,rPos + ,rSize + ,lStyle + ,rValidator + ,rsName)) + return false; - if (pParent) - pParent->AddChild(this); + long lSstyle = WS_TABSTOP; - SetBackgroundColour(pParent->GetBackgroundColour()); - SetForegroundColour(pParent->GetForegroundColour()); + if (HasFlag(wxRB_GROUP)) + lSstyle |= WS_GROUP; - if (vId == -1) - m_windowId = (int)NewControlId(); - else - m_windowId = vId; + // + // wxRB_SINGLE is a temporary workaround for the following problem: if you + // have 2 radiobuttons in the same group but which are not consecutive in + // the dialog, Windows can enter an infinite loop! The simplest way to + // reproduce it is to create radio button, then a panel and then another + // radio button: then checking the last button hangs the app. + // + // Ideally, we'd detect (and avoid) such situation automatically but for + // now, as I don't know how to do it, just allow the user to create + // BS_RADIOBUTTON buttons for such situations. + // + lSstyle |= HasFlag(wxRB_SINGLE) ? BS_RADIOBUTTON : BS_AUTORADIOBUTTON; + if (HasFlag(wxCLIP_SIBLINGS)) + lSstyle |= WS_CLIPSIBLINGS; - m_windowStyle = lStyle ; + if (!OS2CreateControl( wxT("BUTTON") + ,lSstyle + ,rPos + ,rSize + ,rsLabel + ,0 + )) + return false; - if (m_windowStyle & wxRB_GROUP) - lGroupStyle = WS_GROUP; + wxAssociateWinWithHandle(m_hWnd, this); + if (HasFlag(wxRB_GROUP)) + SetValue(true); - lsStyle = lGroupStyle | BS_AUTORADIOBUTTON | WS_VISIBLE ; + SetFont(*wxSMALL_FONT); + SetSize( rPos.x, rPos.y, rSize.x, rSize.y ); + return true; +} // end of wxRadioButton::Create - if (m_windowStyle & wxCLIP_SIBLINGS ) - lsStyle |= WS_CLIPSIBLINGS; - // - // If the parent is a scrolled window the controls must - // have this style or they will overlap the scrollbars - // - if (pParent) - if (pParent->IsKindOf(CLASSINFO(wxScrolledWindow)) || - pParent->IsKindOf(CLASSINFO(wxGenericScrolledWindow))) - lsStyle |= WS_CLIPSIBLINGS; - - m_hWnd = (WXHWND)::WinCreateWindow ( GetHwndOf(pParent) - ,WC_BUTTON - ,rsLabel.c_str() - ,lsStyle - ,0, 0, 0, 0 - ,GetWinHwnd(pParent) - ,HWND_TOP - ,(HMENU)m_windowId - ,NULL - ,NULL - ); - wxCHECK_MSG(m_hWnd, FALSE, wxT("Failed to create radiobutton")); - - if (rsLabel != wxT("")) +wxSize wxRadioButton::DoGetBestSize() const +{ + // We should probably compute snRadioSize but it seems to be a constant + // independent of its label's font size and not made available by OS/2. + static int snRadioSize = RADIO_SIZE; + + wxString sStr = wxGetWindowText(GetHwnd()); + int nRadioWidth; + int nRadioHeight; + + if (!sStr.empty()) { - int nLabelWidth; - int nLabelHeight; - - GetTextExtent( rsLabel - ,&nLabelWidth - ,&nLabelHeight - ,NULL - ,NULL - ,&this->GetFont() + GetTextExtent( sStr + ,&nRadioWidth + ,&nRadioHeight ); - if (nWidth < 0) - nWidth = (int)(nLabelWidth + RADIO_SIZE); - if (nHeight<0) - { - nHeight = (int)(nLabelHeight); - if (nHeight < RADIO_SIZE) - nHeight = RADIO_SIZE; - } + nRadioWidth += snRadioSize; + if (nRadioHeight < snRadioSize) + nRadioHeight = snRadioSize; } else { - if (nWidth < 0) - nWidth = RADIO_SIZE; - if (nHeight < 0) - nHeight = RADIO_SIZE; + nRadioWidth = snRadioSize; + nRadioHeight = snRadioSize; } - - // - // Subclass again for purposes of dialog editing mode - // - SubclassWin((WXHWND)m_hWnd); - SetFont(pParent->GetFont()); - SetSize( nX - ,nY - ,nWidth - ,nHeight - ); - return FALSE; -} // end of wxRadioButton::Create + return wxSize( nRadioWidth + ,nRadioHeight + ); +} // end of wxRadioButton::DoGetBestSize // // Get single selection, for single choice list items @@ -153,36 +139,149 @@ bool wxRadioButton::GetValue() const return((::WinSendMsg((HWND) GetHWND(), BM_QUERYCHECK, (MPARAM)0L, (MPARAM)0L) != 0)); } // end of wxRadioButton::GetValue -bool wxRadioButton::OS2Command( - WXUINT wParam -, WXWORD wId -) +bool wxRadioButton::OS2Command( WXUINT wParam, WXWORD WXUNUSED(wId) ) { - if (wParam == BN_CLICKED) + if (wParam != BN_CLICKED) + return false; + + if (m_bFocusJustSet) { - wxCommandEvent rEvent( wxEVT_COMMAND_RADIOBUTTON_SELECTED - ,m_windowId - ); + // + // See above: we want to ignore this event + // + m_bFocusJustSet = false; + } + else + { + bool bIsChecked = GetValue(); + if (HasFlag(wxRB_SINGLE)) + { + // + // When we use a "manual" radio button, we have to check the button + // ourselves -- but it's reset to unchecked state by the user code + // (presumably when another button is pressed) + // + if (!bIsChecked ) + SetValue(true); + } + wxCommandEvent rEvent( wxEVT_COMMAND_RADIOBUTTON_SELECTED, m_windowId ); rEvent.SetEventObject(this); ProcessCommand(rEvent); - return TRUE; } - else - return FALSE; + return true; } // end of wxRadioButton::OS2Command -void wxRadioButton::SetLabel( - const wxString& rsLabel -) +void wxRadioButton::SetFocus() { - ::WinSetWindowText((HWND)GetHWND(), (const char *)rsLabel.c_str()); + // when the radio button receives a WM_SETFOCUS message it generates a + // BN_CLICKED which is totally unexpected and leads to catastrophic results + // if you pop up a dialog from the radio button event handler as, when the + // dialog is dismissed, the focus is returned to the radio button which + // generates BN_CLICKED which leads to showing another dialog and so on + // without end! + // + // to avoid this, we drop the pseudo BN_CLICKED events generated when the + // button gains focus + m_bFocusJustSet = true; + + wxControl::SetFocus(); +} + +void wxRadioButton::SetLabel( const wxString& rsLabel ) +{ + wxString sLabel = ::wxPMTextToLabel(rsLabel); + ::WinSetWindowText((HWND)GetHWND(), (const char *)sLabel.c_str()); } // end of wxRadioButton::SetLabel -void wxRadioButton::SetValue( - bool bValue -) +void wxRadioButton::SetValue( bool bValue ) { ::WinSendMsg((HWND)GetHWND(), BM_SETCHECK, (MPARAM)bValue, (MPARAM)0); + if (bValue) + { + const wxWindowList& rSiblings = GetParent()->GetChildren(); + wxWindowList::compatibility_iterator nodeThis = rSiblings.Find(this); + + wxCHECK_RET(nodeThis, wxT("radio button not a child of its parent?")); + + // + // If it's not the first item of the group ... + // + if ( !HasFlag(wxRB_GROUP) ) + { + // + // ...turn off all radio buttons before this one + // + for ( wxWindowList::compatibility_iterator nodeBefore = nodeThis->GetPrevious(); + nodeBefore; + nodeBefore = nodeBefore->GetPrevious() ) + { + wxRadioButton* pBtn = wxDynamicCast( nodeBefore->GetData() + ,wxRadioButton + ); + if (!pBtn) + { + // + // The radio buttons in a group must be consecutive, so there + // are no more of them + // + break; + } + pBtn->SetValue(false); + if (pBtn->HasFlag(wxRB_GROUP)) + { + // + // Even if there are other radio buttons before this one, + // they're not in the same group with us + // + break; + } + } + } + + // + // ... and all after this one + // + for (wxWindowList::compatibility_iterator nodeAfter = nodeThis->GetNext(); + nodeAfter; + nodeAfter = nodeAfter->GetNext()) + { + wxRadioButton* pBtn = wxDynamicCast( nodeAfter->GetData() + ,wxRadioButton + ); + + if (!pBtn || pBtn->HasFlag(wxRB_GROUP) ) + { + // + // No more buttons or the first button of the next group + // + break; + } + pBtn->SetValue(false); + } + } } // end of wxRadioButton::SetValue +MRESULT wxRadioButton::OS2WindowProc( + WXUINT uMsg +, WXWPARAM wParam +, WXLPARAM lParam +) +{ + if (uMsg == WM_SETFOCUS) + { + m_bFocusJustSet = true; + + MRESULT mRc = wxControl::OS2WindowProc( uMsg + ,wParam + ,lParam + ); + + m_bFocusJustSet = false; + return mRc; + } + return wxControl::OS2WindowProc( uMsg + ,wParam + ,lParam + ); +} // end of wxRadioButton::OS2WindowProc