X-Git-Url: https://git.saurik.com/wxWidgets.git/blobdiff_plain/421962befc8d6b340e9c2d6201cf73574319409d..54753c3d75cce7721667aef855fe30a0f90dcedd:/src/msw/statusbar.cpp diff --git a/src/msw/statusbar.cpp b/src/msw/statusbar.cpp index c8921219cb..c2767b34df 100644 --- a/src/msw/statusbar.cpp +++ b/src/msw/statusbar.cpp @@ -9,6 +9,14 @@ // Licence: wxWindows licence /////////////////////////////////////////////////////////////////////////////// +// ============================================================================ +// declarations +// ============================================================================ + +// ---------------------------------------------------------------------------- +// headers +// ---------------------------------------------------------------------------- + // for compilers that support precompilation, includes "wx.h". #include "wx/wxprec.h" @@ -38,8 +46,17 @@ #include "wx/msw/uxtheme.h" #endif +// ---------------------------------------------------------------------------- +// constants +// ---------------------------------------------------------------------------- + +namespace +{ + // no idea for a default width, just choose something -#define DEFAULT_FIELD_WIDTH 25 +static const int DEFAULT_FIELD_WIDTH = 25; + +} // anonymous namespace // ---------------------------------------------------------------------------- // macros @@ -67,25 +84,17 @@ wxStatusBar::wxStatusBar() m_pDC = NULL; } -bool wxStatusBar::Create(wxWindow *parent, - wxWindowID id, - long style, - const wxString& name) +WXDWORD wxStatusBar::MSWGetStyle(long style, WXDWORD *exstyle) const { - wxCHECK_MSG( parent, false, "status bar must have a parent" ); - - SetName(name); - SetWindowStyleFlag(style); - SetParent(parent); - - parent->AddChild(this); - - m_windowId = id == wxID_ANY ? NewControlId() : id; + WXDWORD msStyle = wxStatusBarBase::MSWGetStyle(style, exstyle); - DWORD wstyle = WS_CHILD | WS_VISIBLE; - - if ( style & wxCLIP_SIBLINGS ) - wstyle |= WS_CLIPSIBLINGS; + // wxSTB_SIZEGRIP is part of our default style but it doesn't make sense to + // show size grip if this is the status bar of a non-resizable TLW so turn + // it off in such case + wxWindow * const parent = GetParent(); + wxCHECK_MSG( parent, msStyle, wxS("Status bar must have a parent") ); + if ( parent->IsTopLevel() && !parent->HasFlag(wxRESIZE_BORDER) ) + style &= ~wxSTB_SIZEGRIP; // setting SBARS_SIZEGRIP is perfectly useless: it's always on by default // (at least in the version of comctl32.dll I'm using), and the only way to @@ -94,47 +103,38 @@ bool wxStatusBar::Create(wxWindow *parent, // is not given if ( !(style & wxSTB_SIZEGRIP) ) { - wstyle |= CCS_TOP; + msStyle |= CCS_TOP; } else { #ifndef __WXWINCE__ // may be some versions of comctl32.dll do need it - anyhow, it won't // do any harm - wstyle |= SBARS_SIZEGRIP; + msStyle |= SBARS_SIZEGRIP; #endif } - m_hWnd = CreateWindow - ( - STATUSCLASSNAME, - _T(""), - wstyle, - 0, 0, 0, 0, - GetHwndOf(parent), - (HMENU)wxUIntToPtr(m_windowId.GetValue()), - wxGetInstance(), - NULL - ); - if ( m_hWnd == 0 ) - { - wxLogSysError(_("Failed to create a status bar.")); + return msStyle; +} + +bool wxStatusBar::Create(wxWindow *parent, + wxWindowID id, + long style, + const wxString& name) +{ + if ( !CreateControl(parent, id, wxDefaultPosition, wxDefaultSize, + style, wxDefaultValidator, name) ) + return false; + if ( !MSWCreateControl(STATUSCLASSNAME, wxString(), + wxDefaultPosition, wxDefaultSize) ) return false; - } SetFieldsCount(1); - SubclassWin(m_hWnd); - // cache the DC instance used by UpdateFieldText: - // NOTE: create the DC before calling InheritAttributes() since - // it may result in a call to our SetFont() + // cache the DC instance used by DoUpdateStatusText: m_pDC = new wxClientDC(this); - InheritAttributes(); - - SetBackgroundColour(wxSystemSettings::GetColour(wxSYS_COLOUR_MENUBAR)); - // we must refresh the frame size when the statusbar is created, because // its client area might change // @@ -153,15 +153,13 @@ wxStatusBar::~wxStatusBar() // occupy PostSizeEventToParent(); +#if wxUSE_TOOLTIPS // delete existing tooltips for (size_t i=0; i 0) && (nFields < 255), "too many fields" ); - wxStatusBarBase::SetFieldsCount(nFields, widths); - - SetFieldsWidth(); - // keep in synch also our m_tooltips array +#if wxUSE_TOOLTIPS // reset all current tooltips for (size_t i=0; i= 0) && ((size_t)nField < m_panes.GetCount()), - "invalid statusbar field index" ); + // The total width of the panes passed to Windows must be equal to the + // total width available, including the grip. Otherwise we get an extra + // separator line just before it. + pWidths[count - 1] += gripWidth; - if ( strText == GetStatusText(nField) ) + if ( !StatusBar_SetParts(GetHwnd(), count, pWidths) ) { - // don't call StatusBar_SetText() to avoid flicker - return; + wxLogLastError("StatusBar_SetParts"); } - wxStatusBarBase::SetStatusText(strText, nField); + // Now that all parts have been created, set their text. + for ( i = 0; i < count; i++ ) + { + DoUpdateStatusText(i); + } - UpdateFieldText(nField); + delete [] pWidths; } -void wxStatusBar::UpdateFieldText(int nField) +void wxStatusBar::DoUpdateStatusText(int nField) { if (!m_pDC) return; @@ -276,6 +275,7 @@ void wxStatusBar::UpdateFieldText(int nField) style = SBT_NOBORDERS; break; + case wxSB_SUNKEN: case wxSB_NORMAL: default: style = 0; @@ -285,13 +285,8 @@ void wxStatusBar::UpdateFieldText(int nField) wxRect rc; GetFieldRect(nField, rc); - int margin; - if (nField == GetFieldsCount()-1) - margin = -6; // windows reports a smaller rect for the last field; enlarge it - else - margin = 4; + const int maxWidth = rc.GetWidth() - MSWGetMetrics().textMargin; - int maxWidth = rc.GetWidth() - margin; // leave a small margin wxString text = GetStatusText(nField); // do we need to ellipsize this string? @@ -309,23 +304,26 @@ void wxStatusBar::UpdateFieldText(int nField) } else { - text = wxControl::Ellipsize(text, + text = wxControl::Ellipsize(text, *m_pDC, ellmode, maxWidth, - wxELLIPSIZE_EXPAND_TAB); - - // update the ellipsization status for this pane; this is used later to - // decide whether a tooltip should be shown or not for this pane + wxELLIPSIZE_FLAGS_EXPAND_TABS); + + // update the ellipsization status for this pane; this is used later to + // decide whether a tooltip should be shown or not for this pane // (if we have wxSTB_SHOW_TIPS) SetEllipsizedFlag(nField, text != GetStatusText(nField)); } - // Set the status text in the native control passing both field number and style. + // Set the status text in the native control passing both field number and style. // NOTE: MSDN library doesn't mention that nField and style have to be 'ORed' - if ( !StatusBar_SetText(GetHwnd(), nField | style, text.wx_str()) ) + if ( !StatusBar_SetText(GetHwnd(), nField | style, text.t_str()) ) + { wxLogLastError("StatusBar_SetText"); + } +#if wxUSE_TOOLTIPS if (HasFlag(wxSTB_SHOW_TIPS)) { wxASSERT(m_tooltips.size() == m_panes.GetCount()); @@ -343,8 +341,7 @@ void wxStatusBar::UpdateFieldText(int nField) else { // delete the tooltip associated with this pane; it's not needed anymore - delete m_tooltips[nField]; - m_tooltips[nField] = NULL; + wxDELETE(m_tooltips[nField]); } } else @@ -355,27 +352,80 @@ void wxStatusBar::UpdateFieldText(int nField) //else: leave m_tooltips[nField]==NULL } } +#endif // wxUSE_TOOLTIPS } -int wxStatusBar::GetBorderX() const +wxStatusBar::MSWBorders wxStatusBar::MSWGetBorders() const { int aBorders[3]; SendMessage(GetHwnd(), SB_GETBORDERS, 0, (LPARAM)aBorders); - return aBorders[0]; + MSWBorders borders; + borders.horz = aBorders[0]; + borders.vert = aBorders[1]; + borders.between = aBorders[2]; + return borders; +} + +int wxStatusBar::GetBorderX() const +{ + return MSWGetBorders().horz; } int wxStatusBar::GetBorderY() const { - int aBorders[3]; - SendMessage(GetHwnd(), SB_GETBORDERS, 0, (LPARAM)aBorders); + return MSWGetBorders().vert; +} + +int wxStatusBar::MSWGetBorderWidth() const +{ + return MSWGetBorders().between; +} + +/* static */ +const wxStatusBar::MSWMetrics& wxStatusBar::MSWGetMetrics() +{ + static MSWMetrics s_metrics = { 0, 0 }; + if ( !s_metrics.textMargin ) + { + // Grip size should be self explanatory (the only problem with it is + // that it's hard coded as we don't know how to find its size using + // API) but the margin might merit an explanation: Windows offsets the + // text drawn in status bar panes so we need to take this extra margin + // into account to make sure the text drawn by user fits inside the + // pane. Notice that it's not the value returned by SB_GETBORDERS + // which, at least on this Windows 2003 system, returns {0, 2, 2} +#if wxUSE_UXTHEME + if ( wxUxThemeEngine::GetIfActive() ) + { + s_metrics.gripWidth = 20; + s_metrics.textMargin = 8; + } + else // classic/unthemed look +#endif // wxUSE_UXTHEME + { + s_metrics.gripWidth = 18; + s_metrics.textMargin = 4; + } + } - return aBorders[1]; + return s_metrics; } void wxStatusBar::SetMinHeight(int height) { - SendMessage(GetHwnd(), SB_SETMINHEIGHT, height + 2*GetBorderY(), 0); + // It looks like we need to count the border twice to really make the + // controls taking exactly height pixels fully fit in the status bar: + // at least under Windows 7 the checkbox in the custom status bar of the + // statbar sample gets truncated otherwise. + height += 4*GetBorderY(); + + // We need to set the size and not the size to reflect the height because + // wxFrame uses our size and not the minimal size as it assumes that the + // size of a status bar never changes anyhow. + SetSize(-1, height); + + SendMessage(GetHwnd(), SB_SETMINHEIGHT, height, 0); // we have to send a (dummy) WM_SIZE to redraw it now SendMessage(GetHwnd(), WM_SIZE, 0, 0); @@ -388,7 +438,9 @@ bool wxStatusBar::GetFieldRect(int i, wxRect& rect) const RECT r; if ( !::SendMessage(GetHwnd(), SB_GETRECT, i, (LPARAM)&r) ) + { wxLogLastError("SendMessage(SB_GETRECT)"); + } #if wxUSE_UXTHEME wxUxThemeHandle theme(const_cast(this), L"Status"); @@ -414,8 +466,7 @@ bool wxStatusBar::GetFieldRect(int i, wxRect& rect) const wxSize wxStatusBar::DoGetBestSize() const { - int borders[3]; - SendMessage(GetHwnd(), SB_GETBORDERS, 0, (LPARAM)borders); + const MSWBorders borders = MSWGetBorders(); // calculate width int width = 0; @@ -435,7 +486,7 @@ wxSize wxStatusBar::DoGetBestSize() const } // add the space between fields - width += borders[2]; + width += borders.between; } if ( !width ) @@ -444,11 +495,10 @@ wxSize wxStatusBar::DoGetBestSize() const width = 2*DEFAULT_FIELD_WIDTH; } - // calculate height - int height; - wxGetCharSize(GetHWND(), NULL, &height, GetFont()); - height = EDIT_HEIGHT_FROM_CHAR_HEIGHT(height); - height += borders[1]; + // calculate height: by default it should be just big enough to show text + // (see SetMinHeight() for the explanation of 4 factor) + int height = GetCharHeight(); + height += 4*borders.vert; wxSize best(width, height); CacheBestSize(best); @@ -475,9 +525,6 @@ void wxStatusBar::DoMoveWindow(int x, int y, int width, int height) ); } - // adjust fields widths to the new size - SetFieldsWidth(); - // we have to trigger wxSizeEvent if there are children window in status // bar because GetFieldRect returned incorrect (not updated) values up to // here, which almost certainly resulted in incorrectly redrawn statusbar @@ -507,6 +554,7 @@ void wxStatusBar::SetStatusStyles(int n, const int styles[]) case wxSB_FLAT: style = SBT_NOBORDERS; break; + case wxSB_SUNKEN: case wxSB_NORMAL: default: style = 0; @@ -514,11 +562,13 @@ void wxStatusBar::SetStatusStyles(int n, const int styles[]) } // The SB_SETTEXT message is both used to set the field's text as well as - // the fields' styles. + // the fields' styles. // NOTE: MSDN library doesn't mention that nField and style have to be 'ORed' wxString text = GetStatusText(i); - if (!StatusBar_SetText(GetHwnd(), style | i, text.wx_str())) + if (!StatusBar_SetText(GetHwnd(), style | i, text.t_str())) + { wxLogLastError("StatusBar_SetText"); + } } } @@ -566,15 +616,21 @@ wxStatusBar::MSWWindowProc(WXUINT nMsg, WXWPARAM wParam, WXLPARAM lParam) } #endif - bool needsEllipsization = HasFlag(wxSTB_ELLIPSIZE_START) || - HasFlag(wxSTB_ELLIPSIZE_MIDDLE) || - HasFlag(wxSTB_ELLIPSIZE_END); - if ( nMsg == WM_SIZE && needsEllipsization ) + if ( nMsg == WM_SIZE ) { - for (int i=0; i