Autres/audacity
1
0
Fork 0
mirror of https://github.com/cookiengineer/audacity synced 2025-06-16 16:10:06 +02:00
Code Issues Releases Wiki Activity

SetCursor() is only needed once

Browse Source 
Thanks to a discussion with Steve, I went back to the docs
and found that SetCursor() is persistent.  So it doesn't
have to be done in a mouse event handler.  The constructor
is perfectly sufficient.
This commit is contained in:
lllucius 2015-01-02 14:58:22 +00:00
parent 15ac9905ad
commit 0c00a5f463
1 changed files with 1 additions and 28 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

29
src/toolbars/ToolBar.cpp
View File

@ -95,8 +95,6 @@ BEGIN_EVENT_TABLE( ToolBarResizer, wxWindow )
EVT_LEFT_DOWN( ToolBarResizer::OnLeftDown )
EVT_LEFT_UP( ToolBarResizer::OnLeftUp )
EVT_MOTION( ToolBarResizer::OnMotion )
EVT_ENTER_WINDOW( ToolBarResizer::OnEnter )
EVT_LEAVE_WINDOW( ToolBarResizer::OnLeave )
EVT_MOUSE_CAPTURE_LOST( ToolBarResizer::OnCaptureLost )
END_EVENT_TABLE();
@ -104,6 +102,7 @@ ToolBarResizer::ToolBarResizer(ToolBar *bar)
: wxWindow(bar, wxID_ANY, wxDefaultPosition, wxSize(RWIDTH, -1))
{
mBar = bar;
SetCursor( wxCURSOR_SIZEWE );
}
ToolBarResizer::~ToolBarResizer()
@ -160,8 +159,6 @@ void ToolBarResizer::OnLeftDown( wxMouseEvent & event )
// We want all of the mouse events
CaptureMouse();
// SetCursor( wxCURSOR_SIZEWE );
}
void ToolBarResizer::OnLeftUp( wxMouseEvent & event )
@ -173,30 +170,6 @@ void ToolBarResizer::OnLeftUp( wxMouseEvent & event )
{
ReleaseMouse();
}
// SetCursor( wxCURSOR_ARROW );
}
void ToolBarResizer::OnEnter( wxMouseEvent & event )
{
// Go ahead and set the event to propagate
event.Skip();
if( !HasCapture() )
{
SetCursor( wxCURSOR_SIZEWE );
}
}
void ToolBarResizer::OnLeave( wxMouseEvent & event )
{
// Go ahead and set the event to propagate
event.Skip();
if( !HasCapture() )
{
SetCursor( wxCURSOR_ARROW );
}
}
void ToolBarResizer::OnMotion( wxMouseEvent & event )