From da78fad79e8161c8983ecb3a82fd38a25cc0d916 Mon Sep 17 00:00:00 2001
From: Rian8337 <52914632+Rian8337@users.noreply.github.com>
Date: Fri, 8 Sep 2023 16:30:44 +0700
Subject: [PATCH] Split color pickers into single and multiple

The single color picker binds to the user input unlike the multiple one.
---
 ...olorEditor.css => MultipleColorEditor.css} |  0
 ...olorEditor.tsx => MultipleColorEditor.tsx} | 37 +++++----
 src/components/editors/SingleColorEditor.tsx  | 80 +++++++++++++++++++
 .../skin/Color/MenuItemDefaultColor.tsx       |  4 +-
 .../skin/Color/MenuItemDefaultTextColor.tsx   |  4 +-
 .../skin/Color/MenuItemOnTouchColor.tsx       |  4 +-
 .../skin/Color/MenuItemSelectedTextColor.tsx  |  4 +-
 .../Color/MenuItemVersionsDefaultColor.tsx    |  4 +-
 .../Color/MenuItemVersionsSelectedColor.tsx   |  4 +-
 .../groups/skin/ComboColor/ColorList.tsx      | 34 +-------
 .../groups/skin/Slider/SliderBodyColor.tsx    |  4 +-
 .../groups/skin/Slider/SliderBorderColor.tsx  |  4 +-
 .../groups/skin/Slider/SliderHintColor.tsx    |  4 +-
 13 files changed, 125 insertions(+), 62 deletions(-)
 rename src/components/editors/{ColorEditor.css => MultipleColorEditor.css} (100%)
 rename src/components/editors/{ColorEditor.tsx => MultipleColorEditor.tsx} (72%)
 create mode 100644 src/components/editors/SingleColorEditor.tsx

diff --git a/src/components/editors/ColorEditor.css b/src/components/editors/MultipleColorEditor.css
similarity index 100%
rename from src/components/editors/ColorEditor.css
rename to src/components/editors/MultipleColorEditor.css
diff --git a/src/components/editors/ColorEditor.tsx b/src/components/editors/MultipleColorEditor.tsx
similarity index 72%
rename from src/components/editors/ColorEditor.tsx
rename to src/components/editors/MultipleColorEditor.tsx
index 47026ec..4cba9f9 100644
--- a/src/components/editors/ColorEditor.tsx
+++ b/src/components/editors/MultipleColorEditor.tsx
@@ -1,6 +1,6 @@
 import { useState } from "react";
 import { Resettable } from "../../structures/resettable/Resettable";
-import "./ColorEditor.css";
+import "./MultipleColorEditor.css";
 
 interface Props {
     /**
@@ -16,24 +16,34 @@ interface Props {
     /**
      * The configuration that the input is responsible for.
      */
-    resettable: Resettable<string | undefined>;
+    resettable: Resettable<string[]>;
 
     /**
      * The label beside the color input box.
      */
     inputLabel?: string;
-
-    /**
-     * Whether to accept multiple colors. Defaults to `false`.
-     */
-    acceptMultipleColors?: boolean;
 }
 
-export default function ColorEditor(props: Props) {
-    const { title, description, resettable, inputLabel, acceptMultipleColors } =
-        props;
+export default function MultipleColorEditor(props: Props) {
+    const { title, description, resettable, inputLabel } = props;
     const [hexCode, setHexCode] = useState("#FFFFFF");
 
+    // Introduce additional hooks to convert the value from an array.
+    const defaultValue = resettable.defaultValue[0];
+    const [value, setValue] = useState(defaultValue);
+
+    const modifyValue = (value = defaultValue) => {
+        setValue(value);
+
+        resettable.setValue(value.split(",").map((v) => v.trim()));
+    };
+
+    const resetValue = () => {
+        setValue(defaultValue);
+
+        resettable.reset();
+    };
+
     return (
         <div className="json-item-editor">
             <div className="json-item-editor-title">{title}</div>
@@ -51,10 +61,9 @@ export default function ColorEditor(props: Props) {
                 <input
                     className="json-item-editor-input"
                     type="text"
-                    value={resettable.value}
-                    maxLength={acceptMultipleColors ? undefined : 7}
+                    value={value}
                     onChange={(event) => {
-                        resettable.setValue(event.target.value || undefined);
+                        modifyValue(event.target.value || undefined);
                     }}
                 />
 
@@ -62,7 +71,7 @@ export default function ColorEditor(props: Props) {
                     className="json-item-editor-input"
                     type="reset"
                     onClick={() => {
-                        resettable.reset();
+                        resetValue();
                     }}
                 />
             </div>
diff --git a/src/components/editors/SingleColorEditor.tsx b/src/components/editors/SingleColorEditor.tsx
new file mode 100644
index 0000000..1dec693
--- /dev/null
+++ b/src/components/editors/SingleColorEditor.tsx
@@ -0,0 +1,80 @@
+import { Resettable } from "../../structures/resettable/Resettable";
+import { validateColor } from "../../utils/validators";
+
+interface Props {
+    /**
+     * The title of the input.
+     */
+    title: string;
+
+    /**
+     * The description of the input.
+     */
+    description?: string;
+
+    /**
+     * The configuration that the input is responsible for.
+     */
+    resettable: Resettable<string | undefined>;
+}
+
+export default function SingleColorEditor(props: Props) {
+    const { title, description, resettable } = props;
+
+    return (
+        <div className="json-item-editor">
+            <div className="json-item-editor-title">{title}</div>
+            {description ? (
+                <div className="json-item-editor-description">
+                    {description}
+                </div>
+            ) : null}
+
+            <div className="json-item-editor-flex-container">
+                <input
+                    type="color"
+                    // The default color of a color input is black.
+                    // (ref: https://developer.mozilla.org/en-US/docs/Web/HTML/Element/input/color#providing_a_default_color)
+                    value={
+                        resettable.value && validateColor(resettable.value)
+                            ? resettable.value
+                            : "#000000"
+                    }
+                    onChange={(event) => {
+                        resettable.setValue(
+                            event.target.value.toUpperCase() || undefined
+                        );
+                    }}
+                />
+
+                <input
+                    className="json-item-editor-input"
+                    type="text"
+                    value={resettable.value ?? ""}
+                    maxLength={7}
+                    onChange={(event) => {
+                        const value = event.target.value;
+
+                        if (value) {
+                            resettable.setValue(
+                                validateColor(value)
+                                    ? value.toUpperCase()
+                                    : value
+                            );
+                        } else {
+                            resettable.reset();
+                        }
+                    }}
+                />
+
+                <input
+                    className="json-item-editor-input"
+                    type="reset"
+                    onClick={() => {
+                        resettable.reset();
+                    }}
+                />
+            </div>
+        </div>
+    );
+}
diff --git a/src/components/groups/skin/Color/MenuItemDefaultColor.tsx b/src/components/groups/skin/Color/MenuItemDefaultColor.tsx
index b8520a8..deb8bd6 100644
--- a/src/components/groups/skin/Color/MenuItemDefaultColor.tsx
+++ b/src/components/groups/skin/Color/MenuItemDefaultColor.tsx
@@ -1,12 +1,12 @@
 import { useContext } from "react";
 import { MenuItemDefaultColorContext } from "../../../../hooks/Color/MenuItemDefaultColorContext";
-import ColorEditor from "../../../editors/ColorEditor";
+import SingleColorEditor from "../../../editors/SingleColorEditor";
 
 export default function MenuItemDefaultColor() {
     const ctx = useContext(MenuItemDefaultColorContext);
 
     return (
-        <ColorEditor
+        <SingleColorEditor
             title="Unselected Color"
             description="The color of a beatmapset card when it is not selected."
             resettable={ctx}
diff --git a/src/components/groups/skin/Color/MenuItemDefaultTextColor.tsx b/src/components/groups/skin/Color/MenuItemDefaultTextColor.tsx
index a667a70..2566bdf 100644
--- a/src/components/groups/skin/Color/MenuItemDefaultTextColor.tsx
+++ b/src/components/groups/skin/Color/MenuItemDefaultTextColor.tsx
@@ -1,12 +1,12 @@
 import { useContext } from "react";
 import { MenuItemDefaultTextColorContext } from "../../../../hooks/Color/MenuItemDefaultTextColorContext";
-import ColorEditor from "../../../editors/ColorEditor";
+import SingleColorEditor from "../../../editors/SingleColorEditor";
 
 export default function MenuItemDefaultTextColor() {
     const ctx = useContext(MenuItemDefaultTextColorContext);
 
     return (
-        <ColorEditor
+        <SingleColorEditor
             title="Unselected Text Color"
             description="The color of the text inside a beatmap card when it is unselected."
             resettable={ctx}
diff --git a/src/components/groups/skin/Color/MenuItemOnTouchColor.tsx b/src/components/groups/skin/Color/MenuItemOnTouchColor.tsx
index 8532ffc..b42a94a 100644
--- a/src/components/groups/skin/Color/MenuItemOnTouchColor.tsx
+++ b/src/components/groups/skin/Color/MenuItemOnTouchColor.tsx
@@ -1,12 +1,12 @@
 import { useContext } from "react";
 import { MenuItemOnTouchColorContext } from "../../../../hooks/Color/MenuItemOnTouchColorContext";
-import ColorEditor from "../../../editors/ColorEditor";
+import SingleColorEditor from "../../../editors/SingleColorEditor";
 
 export default function MenuItemOnTouchColor() {
     const ctx = useContext(MenuItemOnTouchColorContext);
 
     return (
-        <ColorEditor
+        <SingleColorEditor
             title="Selected Color"
             description="The color of a beatmapset card when it is selected."
             resettable={ctx}
diff --git a/src/components/groups/skin/Color/MenuItemSelectedTextColor.tsx b/src/components/groups/skin/Color/MenuItemSelectedTextColor.tsx
index 0ffa43b..f8ba0a2 100644
--- a/src/components/groups/skin/Color/MenuItemSelectedTextColor.tsx
+++ b/src/components/groups/skin/Color/MenuItemSelectedTextColor.tsx
@@ -1,12 +1,12 @@
 import { useContext } from "react";
-import ColorEditor from "../../../editors/ColorEditor";
+import SingleColorEditor from "../../../editors/SingleColorEditor";
 import { MenuItemSelectedTextColorContext } from "../../../../hooks/Color/MenuItemSelectedTextColorContext";
 
 export default function MenuItemSelectedTextColor() {
     const ctx = useContext(MenuItemSelectedTextColorContext);
 
     return (
-        <ColorEditor
+        <SingleColorEditor
             title="Selected Text Color"
             description="The color of the text inside a beatmap card when it is selected."
             resettable={ctx}
diff --git a/src/components/groups/skin/Color/MenuItemVersionsDefaultColor.tsx b/src/components/groups/skin/Color/MenuItemVersionsDefaultColor.tsx
index 50a2e0c..d43ad6b 100644
--- a/src/components/groups/skin/Color/MenuItemVersionsDefaultColor.tsx
+++ b/src/components/groups/skin/Color/MenuItemVersionsDefaultColor.tsx
@@ -1,12 +1,12 @@
 import { useContext } from "react";
 import { MenuItemVersionsDefaultColorContext } from "../../../../hooks/Color/MenuItemVersionsDefaultColorContext";
-import ColorEditor from "../../../editors/ColorEditor";
+import SingleColorEditor from "../../../editors/SingleColorEditor";
 
 export default function MenuItemVersionsDefaultColor() {
     const ctx = useContext(MenuItemVersionsDefaultColorContext);
 
     return (
-        <ColorEditor
+        <SingleColorEditor
             title="Unselected Card Color"
             description="The color of a beatmap card when it is unselected."
             resettable={ctx}
diff --git a/src/components/groups/skin/Color/MenuItemVersionsSelectedColor.tsx b/src/components/groups/skin/Color/MenuItemVersionsSelectedColor.tsx
index 91afc72..deaea94 100644
--- a/src/components/groups/skin/Color/MenuItemVersionsSelectedColor.tsx
+++ b/src/components/groups/skin/Color/MenuItemVersionsSelectedColor.tsx
@@ -1,12 +1,12 @@
 import { useContext } from "react";
 import { MenuItemVersionsSelectedColorContext } from "../../../../hooks/Color/MenuItemVersionsSelectedColorContext";
-import ColorEditor from "../../../editors/ColorEditor";
+import SingleColorEditor from "../../../editors/SingleColorEditor";
 
 export default function MenuItemVersionsSelectedColor() {
     const ctx = useContext(MenuItemVersionsSelectedColorContext);
 
     return (
-        <ColorEditor
+        <SingleColorEditor
             title="Selected Card Color"
             description="The color of a beatmap card when it is selected."
             resettable={ctx}
diff --git a/src/components/groups/skin/ComboColor/ColorList.tsx b/src/components/groups/skin/ComboColor/ColorList.tsx
index 620b71c..78d5ff5 100644
--- a/src/components/groups/skin/ComboColor/ColorList.tsx
+++ b/src/components/groups/skin/ComboColor/ColorList.tsx
@@ -1,42 +1,16 @@
-import { useContext, useState } from "react";
+import { useContext } from "react";
 import { ComboColorsContext } from "../../../../hooks/ComboColor/ComboColorsContext";
-import ColorEditor from "../../../editors/ColorEditor";
+import MultipleColorEditor from "../../../editors/MultipleColorEditor";
 
 export default function ColorList() {
     const ctx = useContext(ComboColorsContext);
 
-    // Introduce additional hooks to convert the value from an array.
-    const defaultValue = ctx.defaultValue[0];
-    const [value, setValue] = useState(defaultValue);
-
-    const modifyValue = (value = defaultValue) => {
-        setValue(value);
-
-        ctx.setValue(value.split(",").map((v) => v.trim()));
-    };
-
-    const resetValue = () => {
-        setValue(defaultValue);
-
-        ctx.reset();
-    };
-
     return (
-        <ColorEditor
+        <MultipleColorEditor
             title="Combo Colors"
             description="The color order will determine the color appearance order in-game."
             inputLabel="Colors"
-            acceptMultipleColors={true}
-            resettable={{
-                defaultValue,
-                value,
-                reset: resetValue,
-                isDefault: ctx.isDefault,
-                setValue: modifyValue,
-                saveToJSON: (json) => {
-                    ctx.saveToJSON(json);
-                },
-            }}
+            resettable={ctx}
         />
     );
 }
diff --git a/src/components/groups/skin/Slider/SliderBodyColor.tsx b/src/components/groups/skin/Slider/SliderBodyColor.tsx
index f93b5cd..8203702 100644
--- a/src/components/groups/skin/Slider/SliderBodyColor.tsx
+++ b/src/components/groups/skin/Slider/SliderBodyColor.tsx
@@ -1,9 +1,9 @@
 import { useContext } from "react";
 import { SliderBodyColorContext } from "../../../../hooks/Slider/SliderBodyColorContext";
-import ColorEditor from "../../../editors/ColorEditor";
+import SingleColorEditor from "../../../editors/SingleColorEditor";
 
 export default function SliderBodyColor() {
     const ctx = useContext(SliderBodyColorContext);
 
-    return <ColorEditor title="Color" resettable={ctx} />;
+    return <SingleColorEditor title="Color" resettable={ctx} />;
 }
diff --git a/src/components/groups/skin/Slider/SliderBorderColor.tsx b/src/components/groups/skin/Slider/SliderBorderColor.tsx
index 9210303..b3be5f5 100644
--- a/src/components/groups/skin/Slider/SliderBorderColor.tsx
+++ b/src/components/groups/skin/Slider/SliderBorderColor.tsx
@@ -1,9 +1,9 @@
 import { useContext } from "react";
 import { SliderBorderColorContext } from "../../../../hooks/Slider/SliderBorderColorContext";
-import ColorEditor from "../../../editors/ColorEditor";
+import SingleColorEditor from "../../../editors/SingleColorEditor";
 
 export default function SliderBorderColor() {
     const ctx = useContext(SliderBorderColorContext);
 
-    return <ColorEditor title="Color" resettable={ctx} />;
+    return <SingleColorEditor title="Color" resettable={ctx} />;
 }
diff --git a/src/components/groups/skin/Slider/SliderHintColor.tsx b/src/components/groups/skin/Slider/SliderHintColor.tsx
index 36ab0bc..373388b 100644
--- a/src/components/groups/skin/Slider/SliderHintColor.tsx
+++ b/src/components/groups/skin/Slider/SliderHintColor.tsx
@@ -1,9 +1,9 @@
 import { useContext } from "react";
 import { SliderHintColorContext } from "../../../../hooks/Slider/SliderHintColorContext";
-import ColorEditor from "../../../editors/ColorEditor";
+import SingleColorEditor from "../../../editors/SingleColorEditor";
 
 export default function SliderHintColor() {
     const ctx = useContext(SliderHintColorContext);
 
-    return <ColorEditor title="Color" resettable={ctx} />;
+    return <SingleColorEditor title="Color" resettable={ctx} />;
 }