From 76a53c43b2f0fc43c95b9da01c5f4bc52111a877 Mon Sep 17 00:00:00 2001 From: David Bailes Date: Sat, 14 May 2016 15:43:28 +0100 Subject: [PATCH] Fix bug associated with the Cursor short/long jump left/right commands Fixes the following bug: if the zoom level is such that the jump does not cause a visible change in display of the tracks, then the selection in the selection bar, and the play region are not updated. The fix simply moves the location of the call to TP_DisplaySelection(), so that it is called irrespective of whether the tracks are redrawn. --- src/Menus.cpp | 3 +++ src/tracks/ui/EditCursorOverlay.cpp | 3 --- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/src/Menus.cpp b/src/Menus.cpp index 58d4571f6..909332752 100644 --- a/src/Menus.cpp +++ b/src/Menus.cpp @@ -7072,6 +7072,9 @@ void AudacityProject::SeekLeftOrRight // Move the visual cursor, avoiding an unnecessary complete redraw GetTrackPanel()->DrawOverlays(false); GetRulerPanel()->DrawOverlays(false); + + // This updates the selection shown on the selection bar, and the play region + TP_DisplaySelection(); } else { diff --git a/src/tracks/ui/EditCursorOverlay.cpp b/src/tracks/ui/EditCursorOverlay.cpp index 1bee3bdb0..813067fa4 100644 --- a/src/tracks/ui/EditCursorOverlay.cpp +++ b/src/tracks/ui/EditCursorOverlay.cpp @@ -127,9 +127,6 @@ void EditCursorOverlay::Draw(OverlayPanel &panel, wxDC &dc) } } - - // This updates related displays such as numbers on the status bar - mProject->TP_DisplaySelection(); } else if (auto ruler = dynamic_cast(&panel)) { wxASSERT(!mIsMaster);