From 411e4ce9b83534d10d4d1b692f63b2c582b2a18d Mon Sep 17 00:00:00 2001
From: Maximilian Seidler <paideia_dilemma@losfuzzys.net>
Date: Thu, 9 Jan 2025 08:17:45 +0100
Subject: [PATCH] animationmgr: don't warp based on POINTY value

---
 src/managers/AnimationManager.cpp | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/src/managers/AnimationManager.cpp b/src/managers/AnimationManager.cpp
index c1ea1d2769c..f0f6a980bdc 100644
--- a/src/managers/AnimationManager.cpp
+++ b/src/managers/AnimationManager.cpp
@@ -40,7 +40,7 @@ CHyprAnimationManager::CHyprAnimationManager() {
 
 template <Animable VarType>
 void updateVariable(CAnimatedVariable<VarType>& av, const float POINTY, bool warp = false) {
-    if (POINTY >= 1.f || warp || av.value() == av.goal()) {
+    if (warp || av.value() == av.goal()) {
         av.warp();
         return;
     }
@@ -50,7 +50,7 @@ void updateVariable(CAnimatedVariable<VarType>& av, const float POINTY, bool war
 }
 
 void updateColorVariable(CAnimatedVariable<CHyprColor>& av, const float POINTY, bool warp) {
-    if (POINTY >= 1.f || warp || av.value() == av.goal()) {
+    if (warp || av.value() == av.goal()) {
         av.warp();
         return;
     }
@@ -146,11 +146,12 @@ static void handleUpdate(CAnimatedVariable<VarType>& av, bool warp) {
     const auto SPENT   = av.getPercent();
     const auto PBEZIER = g_pAnimationManager->getBezier(av.getBezierName());
     const auto POINTY  = PBEZIER->getYForPoint(SPENT);
+    const bool WARP    = animationsDisabled || SPENT >= 1.f;
 
     if constexpr (std::same_as<VarType, CHyprColor>)
-        updateColorVariable(av, POINTY, animationsDisabled);
+        updateColorVariable(av, POINTY, WARP);
     else
-        updateVariable<VarType>(av, POINTY, animationsDisabled);
+        updateVariable<VarType>(av, POINTY, WARP);
 
     av.onUpdate();