Fixed Code review issues

Author: CoderGamester

Runtime/DelayUiPresenter.cs

@@ -12,7 +12,7 @@ public abstract class DelayUiPresenter : UiPresenter
 
 		private void OnValidate()
 		{
-			_delayer = _delayer ?? GetComponent<PresenterDelayerBase>();
+			_delayer = _delayer != null ? _delayer : GetComponent<PresenterDelayerBase>();
 
 			OnEditorValidate();
 		}

Runtime/Delayers/AnimationDelayer.cs

@@ -8,6 +8,7 @@ namespace GameLovers.UiService
 	/// <remarks>
 	/// Allows this Presenter to have an intro and outro animation when opened and closed to provide feedback and joy for players.
 	/// </remarks>
+	[RequireComponent(typeof(Animation))]
 	public class AnimationDelayer : PresenterDelayerBase
 	{
 		[SerializeField] protected Animation _animation;
@@ -20,6 +21,11 @@ public class AnimationDelayer : PresenterDelayerBase
 		/// <inheritdoc />
 		public override float CloseDelayInSeconds => _outroAnimationClip.length * 1000;
 
+		private void OnValidate()
+		{ 
+			_animation = _animation != null ? _animation : GetComponent<Animation>();
+		}
+
 		protected override void OnOpenStarted()
 		{
 			_animation.clip = _introAnimationClip;

Runtime/Delayers/TimeDelayer.cs

@@ -10,13 +10,13 @@ namespace GameLovers.UiService
 	/// </remarks>
 	public class TimeDelayer : PresenterDelayerBase
 	{
-		[SerializeField] protected float _openDelayInSeconds;
-		[SerializeField] protected float _closeDelayInSeconds;
+		[SerializeField, Range(0f, float.MaxValue)] protected float _openDelayInSeconds;
+		[SerializeField, Range(0f, float.MaxValue)] protected float _closeDelayInSeconds;
 
 		/// <inheritdoc />
 		public override float OpenDelayInSeconds => _openDelayInSeconds;
 
 		/// <inheritdoc />
-		public override float CloseDelayInSeconds => _openDelayInSeconds;
+		public override float CloseDelayInSeconds => _closeDelayInSeconds;
 	}
 }

Runtime/UiService.cs

@@ -275,7 +275,7 @@ public void CloseAllUi()
 		/// <inheritdoc />
 		public void CloseAllUi(int layer)
 		{
-			for (int i = 0; i < _visibleUiList.Count; i++)
+			for (int i = _visibleUiList.Count - 1; i >= 0; i--)
 			{
 				var type = _visibleUiList[i];