X-Git-Url: https://git.saurik.com/wxWidgets.git/blobdiff_plain/57ff8a875bff6f8ba8a6616ed7095b74a835c622..9807f995059c9049505a67c1ba3535e6c43362d1:/src/os2/ownerdrw.cpp?ds=sidebyside diff --git a/src/os2/ownerdrw.cpp b/src/os2/ownerdrw.cpp index c352f843de..64abe8b8ce 100644 --- a/src/os2/ownerdrw.cpp +++ b/src/os2/ownerdrw.cpp @@ -1,36 +1,32 @@ /////////////////////////////////////////////////////////////////////////////// -// Name: msw/ownerdrw.cpp +// Name: src/os2/ownerdrw.cpp // Purpose: implementation of wxOwnerDrawn class // Author: David Webster // Modified by: // Created: 10/12/99 // RCS-ID: $Id$ // Copyright: (c) David Webster -// Licence: wxWindows license +// Licence: wxWindows licence /////////////////////////////////////////////////////////////////////////////// -#ifdef __GNUG__ -#pragma implementation -#endif - // For compilers that support precompilation, includes "wx.h". #include "wx/wxprec.h" +#if wxUSE_OWNER_DRAWN + #ifndef WX_PRECOMP - #include "wx/window.h" - #include "wx/os2/private.h" - #include "wx/font.h" - #include "wx/bitmap.h" - #include "wx/dcmemory.h" - #include "wx/menu.h" - #include "wx/utils.h" + #include "wx/window.h" + #include "wx/os2/private.h" + #include "wx/font.h" + #include "wx/bitmap.h" + #include "wx/dcmemory.h" + #include "wx/menu.h" + #include "wx/utils.h" + #include "wx/settings.h" + #include "wx/menuitem.h" #endif -#if wxUSE_OWNER_DRAWN - #include "wx/ownerdrw.h" -#include "wx/menuitem.h" - // ============================================================================ // implementation of wxOwnerDrawn class @@ -40,21 +36,21 @@ // ctor // ---- // -wxOwnerDrawn::wxOwnerDrawn( - const wxString& rsStr -, bool bCheckable -, bool bMenuItem -) +wxOwnerDrawn::wxOwnerDrawn( const wxString& rsStr, + bool bCheckable, + bool WXUNUSED(bMenuItem) ) : m_strName(rsStr) { m_bCheckable = bCheckable; - m_bOwnerDrawn = FALSE; + m_bOwnerDrawn = false; m_nHeight = 0; m_nMarginWidth = ms_nLastMarginWidth; if (wxNORMAL_FONT) m_font = *wxNORMAL_FONT; } // end of wxOwnerDrawn::wxOwnerDrawn +wxOwnerDrawn::~wxOwnerDrawn() { } + size_t wxOwnerDrawn::ms_nDefaultMarginWidth = 15; size_t wxOwnerDrawn::ms_nLastMarginWidth = ms_nDefaultMarginWidth; @@ -64,23 +60,20 @@ size_t wxOwnerDrawn::ms_nLastMarginWidth = ms_nDefaultMarginWidth; // ------- // -bool wxOwnerDrawn::OnMeasureItem( - size_t* pWidth -, size_t* pHeight -) +bool wxOwnerDrawn::OnMeasureItem( size_t* pWidth, + size_t* pHeight ) { - wxMemoryDC vDC; - + wxMemoryDC vDC; - wxString sStr = wxStripMenuCodes(m_strName); + wxString sStr = wxStripMenuCodes(m_strName); // // If we have a valid accel string, then pad out - // the menu string so the menu and accel string are not - // placed ontop of eachother. + // the menu string so that the menu and accel string are not + // placed on top of each other. if (!m_strAccel.empty() ) { - sStr.Pad(sStr.Length()%8); + sStr.Pad(sStr.length()%8); sStr += m_strAccel; } vDC.SetFont(GetFont()); @@ -88,10 +81,10 @@ bool wxOwnerDrawn::OnMeasureItem( ,(long *)pWidth ,(long *)pHeight ); - if (!m_strAccel.IsEmpty()) + if (!m_strAccel.empty()) { // - // Measure the accelerator string, and add it's width to + // Measure the accelerator string, and add its width to // the total item width, plus 16 (Accelerators are right justified, // with the right edge of the text rectangle 16 pixels left of // the right edge of the menu) @@ -107,10 +100,10 @@ bool wxOwnerDrawn::OnMeasureItem( } // - // Add space at the end of the menu for the submenu expansion arrow - // this will also allow offsetting the accel string from the right edge + // Add space at the end of the menu for the submenu expansion arrow. + // This will also allow offsetting the accel string from the right edge // - *pWidth += GetDefaultMarginWidth() * 1.5; + *pWidth = (size_t)(*pWidth + GetDefaultMarginWidth() * 1.5); // // JACS: items still look too tightly packed, so adding 5 pixels. @@ -165,22 +158,20 @@ bool wxOwnerDrawn::OnMeasureItem( if (*pHeight < m_nMinHeight) *pHeight = m_nMinHeight; m_nHeight = *pHeight; // remember height for use in OnDrawItem - return TRUE; + return true; } // end of wxOwnerDrawn::OnMeasureItem // draw the item -bool wxOwnerDrawn::OnDrawItem( - wxDC& rDC -, const wxRect& rRect -, wxODAction eAction -, wxODStatus eStatus -) +bool wxOwnerDrawn::OnDrawItem( wxDC& rDC, + const wxRect& rRect, + wxODAction eAction, + wxODStatus eStatus ) { // // We do nothing on focus change // if (eAction == wxODFocusChanged ) - return TRUE; + return true; // // Select the font and draw the text @@ -209,11 +200,11 @@ bool wxOwnerDrawn::OnDrawItem( } // - // Base on the status of the menu item pick the right colors + // Based on the status of the menu item, pick the right colors // if (eStatus & wxODSelected) { - wxColour vCol2("WHITE"); + wxColour vCol2(wxT("WHITE")); vColBack.Set( (unsigned char)0 ,(unsigned char)0 ,(unsigned char)160 @@ -245,7 +236,7 @@ bool wxOwnerDrawn::OnDrawItem( // Fall back to default colors if none explicitly specified // vRef = ::WinQuerySysColor( HWND_DESKTOP - ,SYSCLR_MENU // we are using gray for all our window backgrounds in wxWindows + ,SYSCLR_MENU // we are using gray for all our window backgrounds in wxWidgets ,0L ); vColBack.Set( GetRValue(vRef) @@ -290,24 +281,24 @@ bool wxOwnerDrawn::OnDrawItem( // // Unfortunately, unlike Win32, PM has no owner drawn specific text // drawing methods like ::DrawState that can cleanly handle accel - // pneumonics and deal, automatically, with various states, so we have + // mnemonics and deal, automatically, with various states, so we have // to handle them ourselves. Notice Win32 can't handle \t in ownerdrawn - // strings either. We cannot handle mneumonics either. We display - // it, though, in hopes we can figure it out some day. + // strings either. We cannot handle mnemonics either. We display + // them, though, in the hope we can figure them out some day. // // - // Display main text and accel text separately to allign better + // Display main text and accel text separately to align better // - wxString sTgt = "\t"; - wxString sFullString = m_strName; // need to save the original text - wxString sAccel; - size_t nIndex; - size_t nWidth; - size_t nCharWidth; - size_t nHeight; - bool bFoundMneumonic = FALSE; - bool bFoundAccel = FALSE; + wxString sTgt = wxT("\t"); + wxString sFullString = m_strName; // need to save the original text + wxString sAccel; + int nIndex; + size_t nWidth; + size_t nCharWidth; + size_t nHeight; + bool bFoundMnemonic = false; + bool bFoundAccel = false; // // Deal with the tab, extracting the Accel text @@ -315,32 +306,32 @@ bool wxOwnerDrawn::OnDrawItem( nIndex = sFullString.Find(sTgt.c_str()); if (nIndex != -1) { - bFoundAccel = TRUE; + bFoundAccel = true; sAccel = sFullString.Mid(nIndex + 1); sFullString.Remove(nIndex); } // - // Deal with the mneumonic character + // Deal with the mnemonic character // - sTgt = "~"; + sTgt = wxT("~"); nIndex = sFullString.Find(sTgt.c_str()); if (nIndex != -1) { - wxString sTmp = sFullString; + wxString sTmp = sFullString; - bFoundMneumonic = TRUE; + bFoundMnemonic = true; sTmp.Remove(nIndex); rDC.GetTextExtent( sTmp ,(long *)&nWidth ,(long *)&nHeight ); - sTmp = sFullString[nIndex + 1]; + sTmp = sFullString[(size_t)(nIndex + 1)]; rDC.GetTextExtent( sTmp ,(long *)&nCharWidth ,(long *)&nHeight ); - sFullString.Replace(sTgt.c_str(), "", TRUE); + sFullString.Replace(sTgt.c_str(), wxEmptyString, true); } // @@ -352,10 +343,10 @@ bool wxOwnerDrawn::OnDrawItem( ,sFullString.length() ,(PCH)sFullString.c_str() ); - if (bFoundMneumonic) + if (bFoundMnemonic) { // - // Underline the mneumonic -- still won't work, but at least it "looks" right + // Underline the mnemonic -- still won't work, but at least it "looks" right // wxPen vPen; POINTL vPntEnd = {nX + nWidth + nCharWidth - 3, rRect.y + 2}; //CharWidth is bit wide @@ -423,7 +414,7 @@ bool wxOwnerDrawn::OnDrawItem( // // For uncheckable item we use only the 'checked' bitmap // - wxBitmap vBmp(GetBitmap(IsCheckable() ? ((eStatus & wxODChecked) != 0) : TRUE)); + wxBitmap vBmp(GetBitmap(IsCheckable() ? ((eStatus & wxODChecked) != 0) : TRUE)); if (vBmp.Ok()) { @@ -458,16 +449,11 @@ bool wxOwnerDrawn::OnDrawItem( ,0 ,0 ,wxCOPY - ,TRUE + ,true ); if (eStatus & wxODSelected) { - RECT vRectBmp = { rRect.x - ,rRect.y - ,rRect.x + GetMarginWidth() - 1 - ,rRect.y + m_nHeight - 1 - }; POINTL vPnt1 = {rRect.x + 1, rRect.y + 3}; // Leave a little background border POINTL vPnt2 = {rRect.x + GetMarginWidth(), rRect.y + m_nHeight - 3}; @@ -491,7 +477,7 @@ bool wxOwnerDrawn::OnDrawItem( vBmp.SetSelectedInto(NULL); } } - return TRUE; + return true; } // end of wxOwnerDrawn::OnDrawItem #endif //wxUSE_OWNER_DRAWN