X-Git-Url: https://git.saurik.com/wxWidgets.git/blobdiff_plain/7a0d28d8e9b299c1e4e9572cd51f05532cfba62f..5d5da6f610400c6a6a7886b154adeb8a69772f98:/src/os2/menuitem.cpp?ds=sidebyside diff --git a/src/os2/menuitem.cpp b/src/os2/menuitem.cpp index 6382eaa1d4..3e0f891a09 100644 --- a/src/os2/menuitem.cpp +++ b/src/os2/menuitem.cpp @@ -54,6 +54,40 @@ #define OWNER_DRAWN_ONLY( code ) #endif // wxUSE_OWNER_DRAWN/!wxUSE_OWNER_DRAWN +// ---------------------------------------------------------------------------- +// static function for translating menu labels +// ---------------------------------------------------------------------------- + +static wxString TextToLabel(const wxString& rTitle) +{ + wxString Title; + const wxChar *pc; + for (pc = rTitle.c_str(); *pc != wxT('\0'); pc++ ) + { + if (*pc == wxT('&') ) + { + if (*(pc+1) == wxT('&')) + { + pc++; + Title << wxT('&'); + } + else + Title << wxT('~'); + } + else + { + if ( *pc == wxT('~') ) + { + // tildes must be doubled to prevent them from being + // interpreted as accelerator character prefix by PM ??? + Title << *pc; + } + Title << *pc; + } + } + return Title; +} + // ============================================================================ // implementation // ============================================================================ @@ -62,11 +96,7 @@ // dynamic classes implementation // ---------------------------------------------------------------------------- - #if wxUSE_OWNER_DRAWN - IMPLEMENT_DYNAMIC_CLASS2(wxMenuItem, wxMenuItemBase, wxOwnerDrawn) - #else //!USE_OWNER_DRAWN - IMPLEMENT_DYNAMIC_CLASS(wxMenuItem, wxMenuItemBase) - #endif //USE_OWNER_DRAWN +IMPLEMENT_DYNAMIC_CLASS(wxMenuItem, wxObject) // ---------------------------------------------------------------------------- // wxMenuItem @@ -75,196 +105,407 @@ // ctor & dtor // ----------- -wxMenuItem::wxMenuItem(wxMenu *pParentMenu, - int id, - const wxString& text, - const wxString& strHelp, - bool bCheckable, - wxMenu *pSubMenu) +wxMenuItem::wxMenuItem( + wxMenu* pParentMenu +, int nId +, const wxString& rsText +, const wxString& rsHelp +, wxItemKind eKind +, wxMenu* pSubMenu +) +: wxMenuItemBase( pParentMenu + ,nId + ,TextToLabel(rsText) + ,rsHelp + ,eKind + ,pSubMenu + ) #if wxUSE_OWNER_DRAWN - : wxOwnerDrawn(text, bCheckable) +, wxOwnerDrawn( TextToLabel(rsText) + ,eKind == wxITEM_CHECK + ) #endif // owner drawn { - wxASSERT_MSG( pParentMenu != NULL, wxT("a menu item should have a parent") ); + wxASSERT_MSG(pParentMenu != NULL, wxT("a menu item should have a parent")); + + Init(); +} // end of wxMenuItem::wxMenuItem + +wxMenuItem::wxMenuItem( + wxMenu* pParentMenu +, int nId +, const wxString& rsText +, const wxString& rsHelp +, bool bIsCheckable +, wxMenu* pSubMenu +) +: wxMenuItemBase( pParentMenu + ,nId + ,TextToLabel(rsText) + ,rsHelp + ,bIsCheckable ? wxITEM_CHECK : wxITEM_NORMAL + ,pSubMenu + ) +#if wxUSE_OWNER_DRAWN +, wxOwnerDrawn( TextToLabel(rsText) + ,bIsCheckable + ) +#endif // owner drawn +{ + wxASSERT_MSG(pParentMenu != NULL, wxT("a menu item should have a parent")); + + Init(); +} // end of wxMenuItem::wxMenuItem + +void wxMenuItem::Init() +{ + m_vRadioGroup.m_nStart = -1; + m_bIsRadioGroupStart = FALSE; #if wxUSE_OWNER_DRAWN - // set default menu colors - #define SYS_COLOR(c) (wxSystemSettings::GetSystemColour(wxSYS_COLOUR_##c)) + // + // Set default menu colors + // + #define SYS_COLOR(c) (wxSystemSettings::GetColour(wxSYS_COLOUR_##c)) SetTextColour(SYS_COLOR(MENUTEXT)); SetBackgroundColour(SYS_COLOR(MENU)); - // we don't want normal items be owner-drawn + #undef SYS_COLOR + + // + // We don't want normal items be owner-drawn + // ResetOwnerDrawn(); - #undef SYS_COLOR + // + // Tell the owner drawing code to to show the accel string as well + // + SetAccelString(m_text.AfterFirst(_T('\t'))); #endif // wxUSE_OWNER_DRAWN - - m_parentMenu = pParentMenu; - m_subMenu = pSubMenu; - m_isEnabled = TRUE; - m_isChecked = FALSE; - m_id = id; - m_text = text; - m_isCheckable = bCheckable; - m_help = strHelp; -} +} // end of wxMenuItem::Init wxMenuItem::~wxMenuItem() { -} +} // end of wxMenuItem::~wxMenuItem -// misc +// +// Misc // ---- -// return the id for calling Win32 API functions +// +// Return the id for calling Win32 API functions +// int wxMenuItem::GetRealId() const { return m_subMenu ? (int)m_subMenu->GetHMenu() : GetId(); -} +} // end of wxMenuItem::GetRealId -// get item state +// +// Get item state // -------------- - bool wxMenuItem::IsChecked() const { -/* - int flag = ::GetMenuState(GetHMenuOf(m_parentMenu), GetId(), MF_BYCOMMAND); + USHORT uFlag = SHORT1FROMMR(::WinSendMsg( GetHMenuOf(m_parentMenu) + ,MM_QUERYITEMATTR + ,MPFROM2SHORT(GetId(), TRUE) + ,MPFROMSHORT(MIA_CHECKED) + )); + + return (uFlag & MIA_CHECKED); +} // end of wxMenuItem::IsChecked + +wxString wxMenuItemBase::GetLabelFromText( + const wxString& rText +) +{ + wxString label; + for ( const wxChar *pc = rText.c_str(); *pc; pc++ ) + { + if ( *pc == wxT('~') || *pc == wxT('&') ) + { + // '~' is the escape character for GTK+ and '&' is the one for + // wxWindows - skip both of them + continue; + } - // don't "and" with MF_ENABLED because its value is 0 - return (flag & MF_DISABLED) == 0; -*/ - return FALSE; + label += *pc; + } + return label; } -wxString wxMenuItem::GetLabel() const +// radio group stuff +// ----------------- + +void wxMenuItem::SetAsRadioGroupStart() { - return wxStripMenuCodes(m_text); -} + m_bIsRadioGroupStart = TRUE; +} // end of wxMenuItem::SetAsRadioGroupStart -// accelerators -// ------------ +void wxMenuItem::SetRadioGroupStart( + int nStart +) +{ + wxASSERT_MSG( !m_bIsRadioGroupStart, + _T("should only be called for the next radio items") ); -#if wxUSE_ACCEL + m_vRadioGroup.m_nStart = nStart; +} // end of wxMenuItem::SetRadioGroupStart -wxAcceleratorEntry *wxMenuItem::GetAccel() const +void wxMenuItem::SetRadioGroupEnd( + int nEnd +) { - return wxGetAccelFromString(GetText()); -} + wxASSERT_MSG( m_bIsRadioGroupStart, + _T("should only be called for the first radio item") ); -#endif // wxUSE_ACCEL + m_vRadioGroup.m_nEnd = nEnd; +} // end of wxMenuItem::SetRadioGroupEnd // change item state // ----------------- -void wxMenuItem::Enable(bool enable) +void wxMenuItem::Enable( + bool bEnable +) { - if ( m_isEnabled == enable ) - return; -/* - long rc = EnableMenuItem(GetHMenuOf(m_parentMenu), - GetRealId(), - MF_BYCOMMAND | - (enable ? MF_ENABLED : MF_GRAYED)); + bool bOk; - if ( rc == -1 ) { + if (m_isEnabled == bEnable) + return; + if (bEnable) + bOk = (bool)::WinSendMsg( GetHMenuOf(m_parentMenu) + ,MM_SETITEMATTR + ,MPFROM2SHORT(GetRealId(), TRUE) + ,MPFROM2SHORT(MIA_DISABLED, FALSE) + ); + else + bOk = (bool)::WinSendMsg( GetHMenuOf(m_parentMenu) + ,MM_SETITEMATTR + ,MPFROM2SHORT(GetRealId(), TRUE) + ,MPFROM2SHORT(MIA_DISABLED, MIA_DISABLED) + ); + if (!bOk) + { wxLogLastError("EnableMenuItem"); } -*/ - wxMenuItemBase::Enable(enable); -} + wxMenuItemBase::Enable(bEnable); +} // end of wxMenuItem::Enable -void wxMenuItem::Check(bool check) +void wxMenuItem::Check( + bool bCheck +) { - wxCHECK_RET( m_isCheckable, wxT("only checkable items may be checked") ); + bool bOk; - if ( m_isChecked == check ) + wxCHECK_RET( IsCheckable(), wxT("only checkable items may be checked") ); + if (m_isChecked == bCheck) return; -/* - long rc = CheckMenuItem(GetHMenuOf(m_parentMenu), - GetRealId(), - MF_BYCOMMAND | - (check ? MF_CHECKED : MF_UNCHECKED)); - if ( rc == -1 ) { + HMENU hMenu = GetHmenuOf(m_parentMenu); + + if ( GetKind() == wxITEM_RADIO ) + { + // + // It doesn't make sense to uncheck a radio item - what would this do? + // + if (!bCheck) + return; + + // + // Get the index of this item in the menu + // + const wxMenuItemList& rItems = m_parentMenu->GetMenuItems(); + int nPos = rItems.IndexOf(this); + int nStart; + int nEnd; + + wxCHECK_RET( nPos != wxNOT_FOUND, + _T("menuitem not found in the menu items list?") ); + + // + // Get the radio group range + // + + if (m_bIsRadioGroupStart) + { + // we already have all information we need + nStart = nPos; + nEnd = m_vRadioGroup.m_nEnd; + } + else // Next radio group item + { + // + // Get the radio group end from the start item + // + nStart = m_vRadioGroup.m_nStart; + nEnd = rItems.Item(nStart)->GetData()->m_vRadioGroup.m_nEnd; + } + + // + // Also uncheck all the other items in this radio group + // + wxMenuItemList::Node* pNode = rItems.Item(nStart); + + for (int n = nStart; n <= nEnd && pNode; n++) + { + if (n != nPos) + { + pNode->GetData()->m_isChecked = FALSE; + } + + if (n == nPos) + { + bOk = (bool)::WinSendMsg( hMenu + ,MM_SETITEMATTR + ,MPFROM2SHORT(n, TRUE) + ,MPFROM2SHORT(MIA_CHECKED, MIA_CHECKED) + ); + } + else + { + bOk = (bool)::WinSendMsg( hMenu + ,MM_SETITEMATTR + ,MPFROM2SHORT(n, TRUE) + ,MPFROM2SHORT(MIA_CHECKED, FALSE) + ); + } + pNode = pNode->GetNext(); + } + } + else // check item + { + if (bCheck) + bOk = (bool)::WinSendMsg( hMenu + ,MM_SETITEMATTR + ,MPFROM2SHORT(GetRealId(), TRUE) + ,MPFROM2SHORT(MIA_CHECKED, MIA_CHECKED) + ); + else + bOk = (bool)::WinSendMsg( hMenu + ,MM_SETITEMATTR + ,MPFROM2SHORT(GetRealId(), TRUE) + ,MPFROM2SHORT(MIA_CHECKED, FALSE) + ); + } + if (!bOk) + { wxLogLastError("CheckMenuItem"); } -*/ - wxMenuItemBase::Check(check); -} + wxMenuItemBase::Check(bCheck); +} // end of wxMenuItem::Check -void wxMenuItem::SetText(const wxString& text) +void wxMenuItem::SetText( + const wxString& rText +) { - // don't do anything if label didn't change - if ( m_text == text ) + // + // Don't do anything if label didn't change + // + + wxString sText = TextToLabel(rText); + if (m_text == sText) return; - wxMenuItemBase::SetText(text); - OWNER_DRAWN_ONLY( wxOwnerDrawn::SetName(text) ); -/* - HMENU hMenu = GetHMenuOf(m_parentMenu); - wxCHECK_RET( hMenu, wxT("menuitem without menu") ); + wxMenuItemBase::SetText(sText); + OWNER_DRAWN_ONLY(wxOwnerDrawn::SetName(sText)); + + HWND hMenu = GetHmenuOf(m_parentMenu); + + wxCHECK_RET(hMenu, wxT("menuitem without menu")); #if wxUSE_ACCEL m_parentMenu->UpdateAccel(this); #endif // wxUSE_ACCEL - UINT id = GetRealId(); - UINT flagsOld = ::GetMenuState(hMenu, id, MF_BYCOMMAND); - if ( flagsOld == 0xFFFFFFFF ) + USHORT uId = GetRealId(); + MENUITEM vItem; + USHORT uFlagsOld; + + if (!::WinSendMsg( hMenu + ,MM_QUERYITEM + ,MPFROM2SHORT(uId, TRUE) + ,(MPARAM)&vItem + )) { wxLogLastError("GetMenuState"); } else { - if ( IsSubMenu() ) + uFlagsOld = vItem.afStyle; + if (IsSubMenu()) { - // high byte contains the number of items in a submenu for submenus - flagsOld &= 0xFF; - flagsOld |= MF_POPUP; + uFlagsOld |= MIS_SUBMENU; } - LPCTSTR data; + BYTE* pData; #if wxUSE_OWNER_DRAWN - if ( IsOwnerDrawn() ) + if (IsOwnerDrawn()) { - flagsOld |= MF_OWNERDRAW; - data = (LPCTSTR)this; + uFlagsOld |= MIS_OWNERDRAW; + pData = (BYTE*)this; } else #endif //owner drawn { - flagsOld |= MF_STRING; - data = (char*) text.c_str(); + uFlagsOld |= MIS_TEXT; + pData = (BYTE*)sText.c_str(); + } + + // + // Set the style + // + if (!::WinSendMsg( hMenu + ,MM_SETITEM + ,MPFROM2SHORT(uId, TRUE) + ,(MPARAM)&vItem + )) + { + wxLogLastError(wxT("ModifyMenu")); } - if ( ::ModifyMenu(hMenu, id, - MF_BYCOMMAND | flagsOld, - id, data) == (int)0xFFFFFFFF ) + // + // Set the text + // + if (::WinSendMsg( hMenu + ,MM_SETITEMTEXT + ,MPFROMSHORT(uId) + ,(MPARAM)pData + )) { wxLogLastError(wxT("ModifyMenu")); } } -*/ -} +} // end of wxMenuItem::SetText -void wxMenuItem::SetCheckable(bool checkable) +void wxMenuItem::SetCheckable( + bool bCheckable +) { - wxMenuItemBase::SetCheckable(checkable); - OWNER_DRAWN_ONLY( wxOwnerDrawn::SetCheckable(checkable) ); -} + wxMenuItemBase::SetCheckable(bCheckable); + OWNER_DRAWN_ONLY(wxOwnerDrawn::SetCheckable(bCheckable)); +} // end of wxMenuItem::SetCheckable // ---------------------------------------------------------------------------- // wxMenuItemBase // ---------------------------------------------------------------------------- -wxMenuItem *wxMenuItemBase::New(wxMenu *parentMenu, - int id, - const wxString& name, - const wxString& help, - bool isCheckable, - wxMenu *subMenu) +wxMenuItem* wxMenuItemBase::New( + wxMenu* pParentMenu +, int nId +, const wxString& rName +, const wxString& rHelp +, wxItemKind kind +, wxMenu* pSubMenu +) { - return new wxMenuItem(parentMenu, id, name, help, isCheckable, subMenu); -} + return new wxMenuItem( pParentMenu + ,nId + ,rName + ,rHelp + ,kind + ,pSubMenu + ); +} // end of wxMenuItemBase::New +