fix: PlayOnEnable.Reverse in UIEffectTweener does not work

mob-sakai · mob-sakai · commit 3f5a7dcd2d82 · 2025-02-27T18:41:37.000+09:00
close #310
diff --git a/Packages/src/Runtime/UIEffectTweener.cs b/Packages/src/Runtime/UIEffectTweener.cs
@@ -345,7 +345,7 @@ public void Play(bool resetTime)
         {
             if (resetTime)
             {
-                ResetTime();
+                ResetTime(direction);
             }
 
             Play();
@@ -365,7 +365,7 @@ public void PlayForward(bool resetTime)
         {
             if (resetTime)
             {
-                ResetTime();
+                ResetTime(Direction.Forward);
             }
 
             PlayForward();
@@ -386,7 +386,7 @@ public void PlayReverse(bool resetTime)
         {
             if (resetTime)
             {
-                ResetTime();
+                ResetTime(Direction.Reverse);
             }
 
             PlayReverse();
@@ -419,6 +419,18 @@ public void ResetTime()
             SetTime(0);
         }
 
+        public void ResetTime(Direction dir)
+        {
+            if (dir == Direction.Forward)
+            {
+                SetTime(0);
+            }
+            else
+            {
+                SetTime(totalTime - 0.0001f);
+            }
+        }
+
         [Obsolete(
             "UIEffectTweener.Restart has been deprecated. Use UIEffectTweener.ResetTime instead (UnityUpgradable) -> ResetTime")]
         public void Restart()

Original file line number	Diff line number	Diff line change
`@@ -345,7 +345,7 @@ public void Play(bool resetTime)`
`345`	`345`	`{`
`346`	`346`	`if (resetTime)`
`347`	`347`	`{`
`348`		`- ResetTime();`
	`348`	`+ ResetTime(direction);`
`349`	`349`	`}`
`350`	`350`
`351`	`351`	`Play();`
`@@ -365,7 +365,7 @@ public void PlayForward(bool resetTime)`
`365`	`365`	`{`
`366`	`366`	`if (resetTime)`
`367`	`367`	`{`
`368`		`- ResetTime();`
	`368`	`+ ResetTime(Direction.Forward);`
`369`	`369`	`}`
`370`	`370`
`371`	`371`	`PlayForward();`
`@@ -386,7 +386,7 @@ public void PlayReverse(bool resetTime)`
`386`	`386`	`{`
`387`	`387`	`if (resetTime)`
`388`	`388`	`{`
`389`		`- ResetTime();`
	`389`	`+ ResetTime(Direction.Reverse);`
`390`	`390`	`}`
`391`	`391`
`392`	`392`	`PlayReverse();`
`@@ -419,6 +419,18 @@ public void ResetTime()`
`419`	`419`	`SetTime(0);`
`420`	`420`	`}`
`421`	`421`
	`422`	`+ public void ResetTime(Direction dir)`
	`423`	`+ {`
	`424`	`+ if (dir == Direction.Forward)`
	`425`	`+ {`
	`426`	`+ SetTime(0);`
	`427`	`+ }`
	`428`	`+ else`
	`429`	`+ {`
	`430`	`+ SetTime(totalTime - 0.0001f);`
	`431`	`+ }`
	`432`	`+ }`
	`433`	`+`
`422`	`434`	`[Obsolete(`
`423`	`435`	`"UIEffectTweener.Restart has been deprecated. Use UIEffectTweener.ResetTime instead (UnityUpgradable) -> ResetTime")]`
`424`	`436`	`public void Restart()`