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

Bug1134: Click and drag to move a track should make only one undo item

Browse Source
This commit is contained in:
Paul Licameli 2015-08-08 18:55:31 -04:00
parent ad7c87359a
commit 9a53625ceb
2 changed files with 14 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

22
src/TrackPanel.cpp
View File

@ -5545,6 +5545,7 @@ void TrackPanel::HandleLabelClick(wxMouseEvent & event)
// JH: also, capture the current track for rearranging, so the user
// can drag the track up or down to swap it with others
if (!unsafe) {
mRearrangeCount = 0;
SetCapturedTrack( t, IsRearranging );
TrackPanel::CalculateRearrangingThresholds(event);
}
@ -5580,7 +5581,16 @@ void TrackPanel::HandleRearrange(wxMouseEvent & event)
{
// are we finishing the drag?
if (event.LeftUp()) {
SetCapturedTrack( NULL );
if (mRearrangeCount != 0) {
wxString dir;
dir = mRearrangeCount < 0 ? _("up") : _("down");
MakeParentPushState(wxString::Format(_("Moved '%s' %s"),
mCapturedTrack->GetName().c_str(),
dir.c_str()),
_("Move Track"));
}
SetCapturedTrack(NULL);
SetCursor(*mArrowCursor);
return;
}
@ -5591,10 +5601,9 @@ void TrackPanel::HandleRearrange(wxMouseEvent & event)
return;
MixerBoard* pMixerBoard = this->GetMixerBoard(); // Update mixer board, too.
wxString dir;
if (event.m_y < mMoveUpThreshold || event.m_y < 0) {
mTracks->MoveUp(mCapturedTrack);
dir = _("up");
--mRearrangeCount;
#ifdef EXPERIMENTAL_MIDI_OUT
if (pMixerBoard && (mCapturedTrack->GetKind() == Track::Wave ||
mCapturedTrack->GetKind() == Track::Note))
@ -5606,8 +5615,8 @@ void TrackPanel::HandleRearrange(wxMouseEvent & event)
}
else if (event.m_y > mMoveDownThreshold || event.m_y > GetRect().GetHeight()) {
mTracks->MoveDown(mCapturedTrack);
++mRearrangeCount;
/* i18n-hint: a direction as in up or down.*/
dir = _("down");
#ifdef EXPERIMENTAL_MIDI_OUT
if (pMixerBoard && (mCapturedTrack->GetKind() == Track::Wave ||
mCapturedTrack->GetKind() == Track::Note))
@ -5622,11 +5631,6 @@ void TrackPanel::HandleRearrange(wxMouseEvent & event)
return;
}
MakeParentPushState(wxString::Format(_("Moved '%s' %s"),
mCapturedTrack->GetName().c_str(),
dir.c_str()),
_("Move Track"));
// JH: if we moved up or down, recalculate the thresholds and make sure the
// track is fully on-screen.
TrackPanel::CalculateRearrangingThresholds(event);

1
src/TrackPanel.h
View File

@ -773,6 +773,7 @@ protected:
// coordinate should the dragging track move up or down?
int mMoveUpThreshold;
int mMoveDownThreshold;
int mRearrangeCount;
#ifdef EXPERIMENTAL_SCRUBBING_BASIC
int mScrubToken;