Skip to content

Commit

Permalink
Do not clamp time when seeking to the screen boundaries
Browse files Browse the repository at this point in the history
MoveWhenAudioInactive calls ScrollIntoView internally, this results in weird results when snapping.
  • Loading branch information
crsib committed Sep 27, 2023
1 parent 64de5b0 commit 0f54c87
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions src/menus/SelectMenus.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -221,8 +221,6 @@ void MoveWhenAudioInactive
// If TIME_UNIT_SECONDS, snap-to will be off.
auto snapMode = settings.GetSnapMode();
const double t0 = viewInfo.selectedRegion.t0();
const double end = std::max(
tracks.GetEndTime(), viewInfo.GetScreenEndTime());

// Move the cursor
// Already in cursor mode?
Expand All @@ -232,7 +230,6 @@ void MoveWhenAudioInactive
t0, seekStep, timeUnit, snapMode);
// constrain.
newT = std::max(0.0, newT);
newT = std::min(newT, end);
// Move
viewInfo.selectedRegion.setT0(
newT,
Expand Down

0 comments on commit 0f54c87

Please sign in to comment.