Refactor placeholder color handling and add ColorExtensions utility methods

Author: IvanMurzak

Assets/_PackageRoot/Runtime/Future/Extensions/FutureEx.Placeholder.Color.cs

@@ -23,7 +23,7 @@ public static IFuture<Texture2D> SetPlaceholder(this IFuture<Texture2D> future,
         /// <param name="color">hex color to fill solid color</param>
         /// <returns>Returns the Future instance</returns>
         public static IFuture<Texture2D> SetPlaceholder(this IFuture<Texture2D> future, Color color, params PlaceholderTrigger[] triggers)
-            => future.SetPlaceholder(ColorUtility.ToHtmlStringRGBA(color), triggers);
+            => future.SetPlaceholder(color.ToHexRGBA(), triggers);
 
         /// <summary>
         /// Set a placeholder in all conditions for this Future instance
@@ -42,7 +42,7 @@ public static IFuture<Sprite> SetPlaceholder(this IFuture<Sprite> future, Color
         /// <param name="hexColor">hex color to fill solid color</param>
         /// <returns>Returns the Future instance</returns>
         public static IFuture<Sprite> SetPlaceholder(this IFuture<Sprite> future, Color color, params PlaceholderTrigger[] triggers)
-            => future.SetPlaceholder(ColorUtility.ToHtmlStringRGBA(color), triggers);
+            => future.SetPlaceholder(color.ToHexRGBA(), triggers);
 
         /// <summary>
         /// Set a placeholder in all conditions for this Future instance
@@ -61,7 +61,7 @@ public static IFuture<Reference<Texture2D>> SetPlaceholder(this IFuture<Referenc
         /// <param name="hexColor">hex color to fill solid color</param>
         /// <returns>Returns the Future instance</returns>
         public static IFuture<Reference<Texture2D>> SetPlaceholder(this IFuture<Reference<Texture2D>> future, Color color, params PlaceholderTrigger[] triggers)
-            => future.SetPlaceholder(ColorUtility.ToHtmlStringRGBA(color), triggers);
+            => future.SetPlaceholder(color.ToHexRGBA(), triggers);
 
         /// <summary>
         /// Set a placeholder in all conditions for this Future instance
@@ -80,6 +80,6 @@ public static IFuture<Reference<Sprite>> SetPlaceholder(this IFuture<Reference<S
         /// <param name="hexColor">hex color to fill solid color</param>
         /// <returns>Returns the Future instance</returns>
         public static IFuture<Reference<Sprite>> SetPlaceholder(this IFuture<Reference<Sprite>> future, Color color, params PlaceholderTrigger[] triggers)
-            => future.SetPlaceholder(ColorUtility.ToHtmlStringRGBA(color), triggers);
+            => future.SetPlaceholder(color.ToHexRGBA(), triggers);
     }
 }

Assets/_PackageRoot/Runtime/Future/Future.Placeholder.cs

@@ -19,6 +19,16 @@ public IFuture<T> SetPlaceholder(T placeholder, params PlaceholderTrigger[] trig
                 return this;
             }
 
+            if (triggers == null || triggers.Length == 0)
+            {
+                if (LogLevel.IsActive(DebugLevel.Error))
+                    Debug.LogError($"[ImageLoader] Future[id={Id}] SetPlaceholder: triggers are not specified\n{Url}");
+                return this;
+            }
+
+            if (LogLevel.IsActive(DebugLevel.Trace))
+                Debug.Log($"[ImageLoader] Future[id={Id}] SetPlaceholder\n{Url}");
+
             foreach (var trigger in triggers)
             {
                 if (trigger.IsEqual(Status))
@@ -28,7 +38,7 @@ public IFuture<T> SetPlaceholder(T placeholder, params PlaceholderTrigger[] trig
                         foreach (var setter in consumers)
                             Safe.Run(setter, placeholder, LogLevel);
                     }
-                    continue;
+                    // continue;
                 }
                 lock (placeholders)
                     placeholders[trigger.AsFutureStatus()] = placeholder;

Assets/_PackageRoot/Runtime/Future/Future.cs

@@ -225,7 +225,11 @@ void FeedConsumers(T value)
             lock (consumers)
             {
                 foreach (var setter in consumers)
+                {
+                    if (LogLevel.IsActive(DebugLevel.Trace))
+                        Debug.Log($"[ImageLoader] Future[id={Id}] Feed consumer\n{Url}");
                     Safe.Run(setter, value, LogLevel);
+                }
             }
         }
 

Assets/_PackageRoot/Runtime/Utils/ColorExtensions.cs

@@ -0,0 +1,48 @@
+using UnityEngine;
+
+namespace Extensions.Unity.ImageLoader
+{
+    public static class ColorExtensions
+    {
+        public static string ToHexRGB(this Color color, bool hexSymbol = true) => hexSymbol
+            ? $"#{ColorUtility.ToHtmlStringRGB(color)}"
+            : ColorUtility.ToHtmlStringRGB(color);
+
+        public static string ToHexRGBA(this Color color, bool hexSymbol = true) => hexSymbol
+            ? $"#{ColorUtility.ToHtmlStringRGBA(color)}"
+            : ColorUtility.ToHtmlStringRGBA(color);
+
+        public static Color SetR(this Color color, float r)
+        {
+            color.r = r;
+            return color;
+        }
+        public static Color SetG(this Color color, float g)
+        {
+            color.g = g;
+            return color;
+        }
+        public static Color SetB(this Color color, float b)
+        {
+            color.b = b;
+            return color;
+        }
+        public static Color SetA(this Color color, float a)
+        {
+            color.a = a;
+            return color;
+        }
+        public static bool HexToColor(this string hex, out Color color)
+            => ColorUtility.TryParseHtmlString(hex, out color);
+
+        public static Color HexToColor(this string hex)
+        {
+            if (!ColorUtility.TryParseHtmlString(hex, out var color))
+            {
+                if (ImageLoader.settings.debugLevel.IsActive(DebugLevel.Error))
+                    Debug.LogError($"[ImageLoader] Invalid hex color: {hex}");
+            }
+            return color;
+        }
+    }
+}

Assets/_PackageRoot/Runtime/Utils/ColorExtensions.cs.meta

@@ -6,19 +6,19 @@ public class SamplePlaceholder : MonoBehaviour
 {
     [SerializeField] string imageURL; // URL of the image to be loaded
     [SerializeField] Image image; // UI Image component to display the loaded sprite
-    [SerializeField] Sprite placeholder1;
-    [SerializeField] Sprite placeholder2;
-    [SerializeField] Sprite placeholder3;
+    [SerializeField] Sprite placeholderAny;
+    [SerializeField] Sprite placeholderLoadingFromSource;
+    [SerializeField] Color placeholderFailedToLoad = Color.red;
 
     void Start()
     {
         ImageLoader.LoadSprite(imageURL)
             // set placeholder in all conditions
-            .SetPlaceholder(placeholder1)
+            .SetPlaceholder(placeholderAny)
 
             // set placeholder in a specific conditions
-            .SetPlaceholder(placeholder2, PlaceholderTrigger.LoadingFromSource)
-            .SetPlaceholder(placeholder3, PlaceholderTrigger.FailedToLoad)
+            .SetPlaceholder(placeholderLoadingFromSource, PlaceholderTrigger.LoadingFromSource)
+            .SetPlaceholder(placeholderFailedToLoad, PlaceholderTrigger.FailedToLoad)
 
             // set consumer
             .Consume(image)