// Author: Robert Roebling, Vadim Zeitlin (universalization)
// Modified by:
// Created: 20.02.02
-// Id: $Id$
// Copyright: (c) 2001 Robert Roebling,
// (c) 2002 SciTech Software, Inc. (www.scitechsoft.com)
// Licence: wxWindows licence
// headers
// ----------------------------------------------------------------------------
-#ifdef __GNUG__
- #pragma implementation "univtoolbar.h"
-#endif
-
// For compilers that support precompilation, includes "wx.h".
#include "wx/wxprec.h"
#if wxUSE_TOOLBAR
+#include "wx/toolbar.h"
+
#ifndef WX_PRECOMP
#include "wx/utils.h"
#include "wx/app.h"
+ #include "wx/log.h"
+ #include "wx/frame.h"
+ #include "wx/dc.h"
+ #include "wx/image.h"
#endif
#include "wx/univ/renderer.h"
-#include "wx/toolbar.h"
-#include "wx/image.h"
+// ----------------------------------------------------------------------------
+// wxStdToolbarInputHandler: translates SPACE and ENTER keys and the left mouse
+// click into button press/release actions
+// ----------------------------------------------------------------------------
+
+class WXDLLEXPORT wxStdToolbarInputHandler : public wxStdInputHandler
+{
+public:
+ wxStdToolbarInputHandler(wxInputHandler *inphand);
+
+ virtual bool HandleKey(wxInputConsumer *consumer,
+ const wxKeyEvent& event,
+ bool pressed);
+ virtual bool HandleMouse(wxInputConsumer *consumer,
+ const wxMouseEvent& event);
+ virtual bool HandleMouseMove(wxInputConsumer *consumer, const wxMouseEvent& event);
+ virtual bool HandleFocus(wxInputConsumer *consumer, const wxFocusEvent& event);
+ virtual bool HandleActivation(wxInputConsumer *consumer, bool activated);
+
+private:
+ wxWindow *m_winCapture;
+ wxToolBarToolBase *m_toolCapture;
+ wxToolBarToolBase *m_toolLast;
+};
// ----------------------------------------------------------------------------
// constants
// ----------------------------------------------------------------------------
// value meaning that m_widthSeparator is not initialized
-static const wxCoord INVALID_WIDTH = -1;
+static const wxCoord INVALID_WIDTH = wxDefaultCoord;
// ----------------------------------------------------------------------------
// wxToolBarTool: our implementation of wxToolBarToolBase
{
// no position yet
m_x =
- m_y = -1;
+ m_y = wxDefaultCoord;
+ m_width =
+ m_height = 0;
// not pressed yet
- m_isInverted = FALSE;
-
+ m_isInverted = false;
+
// mouse not here yet
- m_underMouse = FALSE;
+ m_underMouse = false;
+ }
+
+ wxToolBarTool(wxToolBar *tbar, wxControl *control, const wxString& label)
+ : wxToolBarToolBase(tbar, control, label)
+ {
+ // no position yet
+ m_x =
+ m_y = wxDefaultCoord;
+ m_width =
+ m_height = 0;
+
+ // not pressed yet
+ m_isInverted = false;
+
+ // mouse not here yet
+ m_underMouse = false;
}
// is this tool pressed, even temporarily? (this is different from being
// press the tool temporarily by inverting its toggle state
void Invert() { m_isInverted = !m_isInverted; }
-
+
// Set underMouse
- void SetUnderMouse( bool under = TRUE ) { m_underMouse = under; }
+ void SetUnderMouse( bool under = true ) { m_underMouse = under; }
bool IsUnderMouse() { return m_underMouse; }
public:
- // the tool position (the size is known by the toolbar itself)
- int m_x,
- m_y;
+ // the tool position (for controls)
+ wxCoord m_x;
+ wxCoord m_y;
+ wxCoord m_width;
+ wxCoord m_height;
private:
- // TRUE if the tool is pressed
+ // true if the tool is pressed
bool m_isInverted;
-
- // TRUE if the tool is under the mouse
+
+ // true if the tool is under the mouse
bool m_underMouse;
};
// wxToolBar implementation
// ============================================================================
-IMPLEMENT_DYNAMIC_CLASS(wxToolBar, wxControl);
+IMPLEMENT_DYNAMIC_CLASS(wxToolBar, wxControl)
// ----------------------------------------------------------------------------
// wxToolBar creation
void wxToolBar::Init()
{
// no tools yet
- m_needsLayout = FALSE;
+ m_needsLayout = false;
// unknown widths for the tools and separators
m_widthSeparator = INVALID_WIDTH;
if ( !wxToolBarBase::Create(parent, id, pos, size, style,
wxDefaultValidator, name) )
{
- return FALSE;
+ return false;
}
+ FixupStyle();
+
CreateInputHandler(wxINP_HANDLER_TOOLBAR);
- SetBestSize(size);
+ SetInitialSize(size);
- return TRUE;
+ return true;
}
wxToolBar::~wxToolBar()
{
+ // Make sure the toolbar is removed from the parent.
+ SetSize(0,0);
}
void wxToolBar::SetMargins(int x, int y)
{
// This required for similar visual effects under
// native platforms and wxUniv.
- wxToolBarBase::SetMargins( x + 2, y + 2 );
+ wxToolBarBase::SetMargins( x + 3, y + 3 );
}
// ----------------------------------------------------------------------------
return NULL;
}
- for ( wxToolBarToolsList::Node *node = m_tools.GetFirst();
+ for ( wxToolBarToolsList::compatibility_iterator node = m_tools.GetFirst();
node;
node = node->GetNext() )
{
{
wxToolBarToolBase *tool = FindById(id);
- wxCHECK_RET( tool, _T("SetToolShortHelp: no such tool") );
+ wxCHECK_RET( tool, wxT("SetToolShortHelp: no such tool") );
tool->SetShortHelp(help);
}
wxToolBarToolBase * WXUNUSED(tool))
{
// recalculate the toolbar geometry before redrawing it the next time
- m_needsLayout = TRUE;
+ m_needsLayout = true;
// and ensure that we indeed are going to redraw
Refresh();
- return TRUE;
+ return true;
}
bool wxToolBar::DoDeleteTool(size_t WXUNUSED(pos),
wxToolBarToolBase * WXUNUSED(tool))
{
// as above
- m_needsLayout = TRUE;
+ m_needsLayout = true;
Refresh();
- return TRUE;
+ return true;
}
void wxToolBar::DoEnableTool(wxToolBarToolBase *tool, bool enable)
{
+#if wxUSE_IMAGE
// created disabled-state bitmap on demand
- if ( !enable && !tool->GetDisabledBitmap().Ok() )
+ if ( !enable && !tool->GetDisabledBitmap().IsOk() )
{
- wxImage image( tool->GetNormalBitmap().ConvertToImage() );
-
- // TODO: don't hardcode 180
- unsigned char bg_red = 180;
- unsigned char bg_green = 180;
- unsigned char bg_blue = 180;
-
- unsigned char mask_red = image.GetMaskRed();
- unsigned char mask_green = image.GetMaskGreen();
- unsigned char mask_blue = image.GetMaskBlue();
-
- bool has_mask = image.HasMask();
-
- int x,y;
- for (y = 0; y < image.GetHeight(); y++)
- {
- for (x = 0; x < image.GetWidth(); x++)
- {
- unsigned char red = image.GetRed(x,y);
- unsigned char green = image.GetGreen(x,y);
- unsigned char blue = image.GetBlue(x,y);
- if (!has_mask || red != mask_red || green != mask_green || blue != mask_blue)
- {
- red = (((wxInt32) red - bg_red) >> 1) + bg_red;
- green = (((wxInt32) green - bg_green) >> 1) + bg_green;
- blue = (((wxInt32) blue - bg_blue) >> 1) + bg_blue;
- image.SetRGB( x, y, red, green, blue );
- }
- }
- }
-
- for (y = 0; y < image.GetHeight(); y++)
- {
- for (x = y % 2; x < image.GetWidth(); x += 2)
- {
- unsigned char red = image.GetRed(x,y);
- unsigned char green = image.GetGreen(x,y);
- unsigned char blue = image.GetBlue(x,y);
- if (!has_mask || red != mask_red || green != mask_green || blue != mask_blue)
- {
- red = (((wxInt32) red - bg_red) >> 1) + bg_red;
- green = (((wxInt32) green - bg_green) >> 1) + bg_green;
- blue = (((wxInt32) blue - bg_blue) >> 1) + bg_blue;
- image.SetRGB( x, y, red, green, blue );
- }
- }
- }
+ wxImage image(tool->GetNormalBitmap().ConvertToImage());
- tool->SetDisabledBitmap(image);
+ tool->SetDisabledBitmap(image.ConvertToGreyscale());
}
+#endif // wxUSE_IMAGE
RefreshTool(tool);
}
clientData, shortHelp, longHelp);
}
-wxToolBarToolBase *wxToolBar::CreateTool(wxControl *control)
+wxToolBarToolBase *
+wxToolBar::CreateTool(wxControl *control, const wxString& label)
{
- wxFAIL_MSG( wxT("Toolbar doesn't support controls yet (TODO)") );
-
- return NULL;
+ return new wxToolBarTool(this, control, label);
}
// ----------------------------------------------------------------------------
wxRect rect;
- wxCHECK_MSG( tool, rect, _T("GetToolRect: NULL tool") );
+ wxCHECK_MSG( tool, rect, wxT("GetToolRect: NULL tool") );
// ensure that we always have the valid tool position
if ( m_needsLayout )
if ( IsVertical() )
{
- rect.width = m_defaultWidth;
- rect.height = tool->IsSeparator() ? m_widthSeparator : m_defaultHeight;
+ if (tool->IsButton())
+ {
+ if(!HasFlag(wxTB_TEXT))
+ {
+ rect.width = m_defaultWidth;
+ rect.height = m_defaultHeight;
+ }
+ else
+ {
+ rect.width = m_defaultWidth +
+ GetFont().GetPointSize() * tool->GetLabel().length();
+ rect.height = m_defaultHeight;
+ }
+ }
+ else if (tool->IsSeparator())
+ {
+ rect.width = m_defaultWidth;
+ rect.height = m_widthSeparator;
+ }
+ else // control
+ {
+ rect.width = tool->m_width;
+ rect.height = tool->m_height;
+ }
}
else // horizontal
{
- rect.width = tool->IsSeparator() ? m_widthSeparator : m_defaultWidth;
- rect.height = m_defaultHeight;
+ if (tool->IsButton())
+ {
+ if(!HasFlag(wxTB_TEXT))
+ {
+ rect.width = m_defaultWidth;
+ rect.height = m_defaultHeight;
+ }
+ else
+ {
+ rect.width = m_defaultWidth +
+ GetFont().GetPointSize() * tool->GetLabel().length();
+ rect.height = m_defaultHeight;
+ }
+ }
+ else if (tool->IsSeparator())
+ {
+ rect.width = m_widthSeparator;
+ rect.height = m_defaultHeight;
+ }
+ else // control
+ {
+ rect.width = tool->m_width;
+ rect.height = tool->m_height;
+ }
}
rect.width += 2*m_xMargin;
bool wxToolBar::Realize()
{
if ( !wxToolBarBase::Realize() )
- return FALSE;
+ return false;
- m_needsLayout = TRUE;
+ m_needsLayout = true;
DoLayout();
- SetBestSize(wxDefaultSize);
+ // the first item in the radio group is checked by default to be consistent
+ // with wxGTK and the menu radio items
+ int radioGroupCount = 0;
- return TRUE;
+ for ( wxToolBarToolsList::compatibility_iterator node = m_tools.GetFirst();
+ node;
+ node = node->GetNext() )
+ {
+ wxToolBarTool *tool = (wxToolBarTool*) node->GetData();
+
+ if ( !tool->IsButton() || tool->GetKind() != wxITEM_RADIO )
+ {
+ radioGroupCount = 0;
+ continue;
+ }
+
+ bool toggle = !radioGroupCount++;
+ if ( tool->Toggle(toggle) )
+ {
+ DoToggleTool(tool, toggle);
+ }
+ }
+
+ SetInitialSize(wxDefaultSize);
+
+ return true;
+}
+
+void wxToolBar::SetWindowStyleFlag( long style )
+{
+ wxToolBarBase::SetWindowStyleFlag(style);
+
+ m_needsLayout = true;
+
+ Refresh();
}
void wxToolBar::DoLayout()
{
- wxASSERT_MSG( m_needsLayout, _T("why are we called?") );
+ wxASSERT_MSG( m_needsLayout, wxT("why are we called?") );
- m_needsLayout = FALSE;
+ m_needsLayout = false;
wxCoord x = m_xMargin,
y = m_yMargin;
- const wxCoord widthTool = IsVertical() ? m_defaultHeight : m_defaultWidth;
- wxCoord margin = IsVertical() ? m_xMargin : m_yMargin,
- *pCur = IsVertical() ? &y : &x;
+ wxCoord widthTool = 0, maxWidthTool = 0;
+ wxCoord heightTool = 0, maxHeightTool = 0;
+ wxCoord margin = IsVertical() ? m_xMargin : m_yMargin;
+ wxCoord *pCur = IsVertical() ? &y : &x;
// calculate the positions of all elements
- for ( wxToolBarToolsList::Node *node = m_tools.GetFirst();
+ for ( wxToolBarToolsList::compatibility_iterator node = m_tools.GetFirst();
node;
node = node->GetNext() )
{
tool->m_x = x;
tool->m_y = y;
- *pCur += (tool->IsSeparator() ? m_widthSeparator : widthTool) + margin;
+ // TODO ugly number fiddling
+ if (tool->IsButton())
+ {
+ if (IsVertical())
+ {
+ widthTool = m_defaultHeight;
+ heightTool = m_defaultWidth;
+ if(HasFlag(wxTB_TEXT))
+ heightTool += GetFont().GetPointSize() * tool->GetLabel().length();
+ }
+ else
+ {
+ widthTool = m_defaultWidth;
+ if(HasFlag(wxTB_TEXT))
+ widthTool += GetFont().GetPointSize() * tool->GetLabel().length();
+
+ heightTool = m_defaultHeight;
+ }
+
+ if(widthTool > maxWidthTool) // Record max width of tool
+ {
+ maxWidthTool = widthTool;
+ }
+
+ if(heightTool > maxHeightTool) // Record max width of tool
+ {
+ maxHeightTool = heightTool;
+ }
+
+ *pCur += widthTool;
+ }
+ else if (tool->IsSeparator())
+ {
+ *pCur += m_widthSeparator;
+ }
+ else if (!IsVertical()) // horizontal control
+ {
+ wxControl *control = tool->GetControl();
+ wxSize size = control->GetSize();
+ tool->m_y += (m_defaultHeight - size.y)/2;
+ tool->m_width = size.x;
+ tool->m_height = size.y;
+
+ *pCur += tool->m_width;
+ }
+ *pCur += margin;
}
// calculate the total toolbar size
- wxCoord xMin = m_defaultWidth + 2*m_xMargin,
- yMin = m_defaultHeight + 2*m_yMargin;
+ wxCoord xMin, yMin;
+
+ if(!HasFlag(wxTB_TEXT))
+ {
+ xMin = m_defaultWidth + 2*m_xMargin;
+ yMin = m_defaultHeight + 2*m_yMargin;
+ }
+ else
+ {
+ if (IsVertical())
+ {
+ xMin = heightTool + 2*m_xMargin;
+ yMin = widthTool + 2*m_xMargin;
+ }
+ else
+ {
+ xMin = maxWidthTool + 2*m_xMargin;
+ yMin = heightTool + 2*m_xMargin;
+ }
+ }
m_maxWidth = x < xMin ? xMin : x;
m_maxHeight = y < yMin ? yMin : y;
return wxSize(m_maxWidth, m_maxHeight);
}
+void wxToolBar::DoSetSize(int x, int y, int width, int height, int sizeFlags)
+{
+ int old_width, old_height;
+ GetSize(&old_width, &old_height);
+
+ wxToolBarBase::DoSetSize(x, y, width, height, sizeFlags);
+
+ // Correct width and height if needed.
+ if ( width == wxDefaultCoord || height == wxDefaultCoord )
+ {
+ int tmp_width, tmp_height;
+ GetSize(&tmp_width, &tmp_height);
+
+ if ( width == wxDefaultCoord )
+ width = tmp_width;
+ if ( height == wxDefaultCoord )
+ height = tmp_height;
+ }
+
+ // We must refresh the frame size when the toolbar changes size
+ // otherwise the toolbar can be shown incorrectly
+ if ( old_width != width || old_height != height )
+ {
+ SendSizeEventToParent();
+ }
+}
+
// ----------------------------------------------------------------------------
// wxToolBar drawing
// ----------------------------------------------------------------------------
wxCoord *start,
wxCoord *end) const
{
- wxCHECK_RET( start && end, _T("NULL pointer in GetRectLimits") );
+ wxCHECK_RET( start && end, wxT("NULL pointer in GetRectLimits") );
if ( IsVertical() )
{
// prepare the variables used below
wxDC& dc = renderer->GetDC();
wxRenderer *rend = renderer->GetRenderer();
- // dc.SetFont(GetFont()); -- uncomment when we support labels
+ dc.SetFont(GetFont());
// draw the border separating us from the menubar (if there is no menubar
// we probably shouldn't draw it?)
GetRectLimits(rectUpdate, &start, &end);
// and redraw all the tools intersecting it
- for ( wxToolBarToolsList::Node *node = m_tools.GetFirst();
+ for ( wxToolBarToolsList::compatibility_iterator node = m_tools.GetFirst();
node;
node = node->GetNext() )
{
// we're beyond the area to redraw, nothing left to do
break;
}
-
+
+ if (tool->IsSeparator() && !HasFlag(wxTB_FLAT))
+ {
+ // Draw separators only in flat mode
+ continue;
+ }
+
// deal with the flags
int flags = 0;
wxBitmap bitmap;
if ( !tool->IsSeparator() )
{
- // label = tool->GetLabel();
+ label = tool->GetLabel();
bitmap = tool->GetBitmap();
+
+ if ( !bitmap.IsOk() )
+ {
+ // it's better not to draw anything than to assert inside
+ // drawing code as this results in an almost guaranteed crash
+ // as we're likely to be called by a paint event handler and so
+ // the assert is going to be triggered again and again and ...
+ continue;
+ }
}
//else: leave both the label and the bitmap invalid to draw a separator
- rend->DrawToolBarButton(dc, label, bitmap, rectTool, flags);
+ if ( !tool->IsControl() )
+ {
+ int tbStyle = HasFlag(wxTB_VERTICAL) ? wxTB_VERTICAL : wxTB_HORIZONTAL;
+ if ( HasFlag(wxTB_TEXT) )
+ tbStyle |= wxTB_TEXT;
+
+ rend->DrawToolBarButton(dc, label, bitmap, rectTool, flags,
+ tool->GetStyle(), tbStyle);
+ }
+ else // control
+ {
+ wxControl *control = tool->GetControl();
+ control->Move(tool->m_x, tool->m_y);
+ }
}
}
const wxString& strArg)
{
wxToolBarTool *tool = (wxToolBarTool*) FindById(numArg);
-
+ if (!tool)
+ return false;
+
if ( action == wxACTION_TOOLBAR_TOGGLE )
{
PerformAction( wxACTION_BUTTON_RELEASE, numArg );
PerformAction( wxACTION_BUTTON_CLICK, numArg );
+
+ // Set mouse leave toolbar button range (If still in the range,
+ // toolbar button would get focus again
+ PerformAction( wxACTION_TOOLBAR_LEAVE, numArg );
}
else if ( action == wxACTION_TOOLBAR_PRESS )
{
- wxLogTrace(_T("toolbar"), _T("Button '%s' pressed."), tool->GetShortHelp().c_str());
-
+ wxLogTrace(wxT("toolbar"), wxT("Button '%s' pressed."), tool->GetShortHelp().c_str());
+
tool->Invert();
RefreshTool( tool );
}
else if ( action == wxACTION_TOOLBAR_RELEASE )
{
- wxLogTrace(_T("toolbar"), _T("Button '%s' released."), tool->GetShortHelp().c_str());
+ wxLogTrace(wxT("toolbar"), wxT("Button '%s' released."), tool->GetShortHelp().c_str());
- wxASSERT_MSG( tool->IsInverted(), _T("release unpressed button?") );
-
- tool->Invert();
+ wxASSERT_MSG( tool->IsInverted(), wxT("release unpressed button?") );
+
+ if(tool->IsInverted())
+ {
+ tool->Invert();
+ }
RefreshTool( tool );
}
bool isToggled;
if ( tool->CanBeToggled() )
{
- tool->Toggle();
+ if ( tool->IsButton() && tool->GetKind() == wxITEM_RADIO )
+ {
+ UnToggleRadioGroup(tool);
+ tool->Toggle(true);
+ }
+ else
+ {
+ tool->Toggle();
+ }
RefreshTool( tool );
}
else // simple non-checkable tool
{
- isToggled = FALSE;
+ isToggled = false;
}
OnLeftClick( tool->GetId(), isToggled );
}
else if ( action == wxACTION_TOOLBAR_ENTER )
{
- wxCHECK_MSG( tool, FALSE, _T("no tool to enter?") );
-
+ wxCHECK_MSG( tool, false, wxT("no tool to enter?") );
+
if ( HasFlag(wxTB_FLAT) && tool->IsEnabled() )
{
- tool->SetUnderMouse( TRUE );
-
+ tool->SetUnderMouse( true );
+
if ( !tool->IsToggled() )
RefreshTool( tool );
}
}
else if ( action == wxACTION_TOOLBAR_LEAVE )
{
- wxCHECK_MSG( tool, FALSE, _T("no tool to leave?") );
-
+ wxCHECK_MSG( tool, false, wxT("no tool to leave?") );
+
if ( HasFlag(wxTB_FLAT) && tool->IsEnabled() )
{
- tool->SetUnderMouse( FALSE );
-
+ tool->SetUnderMouse( false );
+
if ( !tool->IsToggled() )
RefreshTool( tool );
}
else
return wxControl::PerformAction(action, numArg, strArg);
- return TRUE;
+ return true;
+}
+
+/* static */
+wxInputHandler *wxToolBar::GetStdInputHandler(wxInputHandler *handlerDef)
+{
+ static wxStdToolbarInputHandler s_handler(handlerDef);
+
+ return &s_handler;
}
// ============================================================================
if ( event.Button(1) )
{
- if ( !tool || !tool->IsEnabled() )
- return TRUE;
if ( event.LeftDown() || event.LeftDClick() )
{
+ if ( !tool || !tool->IsEnabled() )
+ return true;
+
m_winCapture = tbar;
m_winCapture->CaptureMouse();
-
+
m_toolCapture = tool;
consumer->PerformAction( wxACTION_BUTTON_PRESS, tool->GetId() );
- return TRUE;
+ return true;
}
else if ( event.LeftUp() )
{
m_winCapture = NULL;
}
- if ( tool == m_toolCapture )
+ if (m_toolCapture)
{
- // this will generate a click event
- consumer->PerformAction( wxACTION_BUTTON_TOGGLE, tool->GetId() );
-
- m_toolCapture = NULL;
-
- return TRUE;
+ if ( tool == m_toolCapture )
+ consumer->PerformAction( wxACTION_BUTTON_TOGGLE, m_toolCapture->GetId() );
+ else
+ consumer->PerformAction( wxACTION_TOOLBAR_LEAVE, m_toolCapture->GetId() );
}
- //else: the mouse was released outside the tool or in
- // a different tool
-
+
m_toolCapture = NULL;
-
+
+ return true;
}
//else: don't do anything special about the double click
}
if ( !wxStdInputHandler::HandleMouseMove(consumer, event) )
{
wxToolBar *tbar = wxStaticCast(consumer->GetInputWindow(), wxToolBar);
-
+
wxToolBarTool *tool;
if ( event.Leaving() )
{
{
tool = (wxToolBarTool*) tbar->FindToolForPosition( event.GetX(), event.GetY() );
}
-
- if ((tool) && (tool == m_toolLast))
- {
- // Still over the same tool as last time
- return TRUE;
- }
-
- if (m_toolLast)
+
+ if (m_toolCapture)
{
- // Leave old tool if any
- consumer->PerformAction( wxACTION_TOOLBAR_LEAVE, m_toolLast->GetId() );
+ // During capture we only care of the captured tool
+ if (tool && (tool != m_toolCapture))
+ tool = NULL;
+
+ if (tool == m_toolLast)
+ return true;
+
+ if (tool)
+ consumer->PerformAction( wxACTION_BUTTON_PRESS, m_toolCapture->GetId() );
+ else
+ consumer->PerformAction( wxACTION_BUTTON_RELEASE, m_toolCapture->GetId() );
+
+ m_toolLast = tool;
}
-
- if (m_toolCapture && (m_toolCapture != tool))
- m_toolLast = NULL;
else
- m_toolLast = tool;
-
- if (m_toolLast)
{
- // Enter new tool if any
- consumer->PerformAction( wxACTION_TOOLBAR_ENTER, m_toolLast->GetId() );
+ if (tool == m_toolLast)
+ return true;
+
+ if (m_toolLast)
+ {
+ // Leave old tool if any
+ consumer->PerformAction( wxACTION_TOOLBAR_LEAVE, m_toolLast->GetId() );
+ }
+
+ if (tool)
+ {
+ // Enter new tool if any
+ consumer->PerformAction( wxACTION_TOOLBAR_ENTER, tool->GetId() );
+ }
+
+ m_toolLast = tool;
}
-
- return TRUE;
+
+ return true;
}
- return FALSE;
+ return false;
}
bool wxStdToolbarInputHandler::HandleFocus(wxInputConsumer *consumer,
- const wxFocusEvent& event)
+ const wxFocusEvent& WXUNUSED(event))
{
- if (m_toolCapture)
+ if ( m_toolCapture )
{
// We shouldn't be left with a highlighted button
consumer->PerformAction( wxACTION_TOOLBAR_LEAVE, m_toolCapture->GetId() );
}
- return TRUE;
+ return true;
}
bool wxStdToolbarInputHandler::HandleActivation(wxInputConsumer *consumer,
consumer->PerformAction( wxACTION_TOOLBAR_LEAVE, m_toolCapture->GetId() );
}
- return TRUE;
+ return true;
}
#endif // wxUSE_TOOLBAR
-
projects / wxWidgets.git / blobdiff