From 0f54c8731ec6edd4748e4b21a0c5551de8cc56df Mon Sep 17 00:00:00 2001
From: Dmitry Vedenko <dmitry@crsib.me>
Date: Tue, 12 Sep 2023 16:02:58 +0300
Subject: [PATCH] Do not clamp time when seeking to the screen boundaries

MoveWhenAudioInactive calls ScrollIntoView internally, this results in weird results when snapping.
---
 src/menus/SelectMenus.cpp | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/src/menus/SelectMenus.cpp b/src/menus/SelectMenus.cpp
index fee2c8882a03..34809377d373 100644
--- a/src/menus/SelectMenus.cpp
+++ b/src/menus/SelectMenus.cpp
@@ -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?
@@ -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,