From 0bb5f7a8cc6cf3497a2f3fec92c14035cc11cb85 Mon Sep 17 00:00:00 2001
From: Ilza Medeiros <ilza.medeiros@liferay.com>
Date: Mon, 4 Sep 2023 16:13:00 -0300
Subject: [PATCH] refactor(@clayui/color-picker): Change Hue component visual

---
 packages/clay-color-picker/src/Hue.tsx | 122 +++++++++----------------
 1 file changed, 41 insertions(+), 81 deletions(-)

diff --git a/packages/clay-color-picker/src/Hue.tsx b/packages/clay-color-picker/src/Hue.tsx
index b8ef6e16ce..5eea768965 100644
--- a/packages/clay-color-picker/src/Hue.tsx
+++ b/packages/clay-color-picker/src/Hue.tsx
@@ -3,13 +3,11 @@
  * SPDX-License-Identifier: BSD-3-Clause
  */
 
-import {Keys} from '@clayui/shared';
-import React, {useCallback} from 'react';
+import {ClayInput} from '@clayui/form';
+import ClaySlider from '@clayui/slider';
+import React from 'react';
 
 import {LimitValue} from './Editor';
-import {usePointerPosition} from './hooks';
-import {hueToX, xToHue} from './util';
-
 type Props = {
 	/**
 	 * Callback function for when the hue value changes
@@ -22,90 +20,52 @@ type Props = {
 	value: number;
 };
 
-const useIsomorphicLayoutEffect =
-	typeof window === 'undefined' ? React.useEffect : React.useLayoutEffect;
-
 /**
  * Renders Hue component
  */
 const ClayColorPickerHue = ({value = 0, onChange = () => {}}: Props) => {
-	const containerRef = React.useRef<HTMLDivElement>(null);
-	const selectorActive = React.useRef<boolean>(false);
-
-	const {onPointerMove, setXY, x, y} = usePointerPosition(containerRef);
-
-	const removeListeners = useCallback(() => {
-		selectorActive.current = false;
-
-		window.removeEventListener('pointermove', onPointerMove);
-		window.removeEventListener('pointerup', removeListeners);
-	}, []);
-
-	const onKeyDown = (event: React.KeyboardEvent<HTMLDivElement>) => {
-		event.preventDefault();
-
-		switch (event.key) {
-			case Keys.Left:
-				onChange(value - 1);
-				break;
-			case Keys.Right:
-				onChange(value + 1);
-				break;
-			default:
-				return;
-		}
-	};
-
-	useIsomorphicLayoutEffect(() => {
-		if (containerRef.current && selectorActive.current) {
-			onChange(xToHue(x, containerRef.current));
-		}
-	}, [x]);
-
-	React.useEffect(() => {
-		if (containerRef.current) {
-			setXY({x: hueToX(value, containerRef.current), y});
-		}
-	}, [value]);
-
-	React.useEffect(() => removeListeners, []);
-
 	return (
-		<div
-			aria-labelledby="color-range"
-			aria-valuemax={LimitValue.maxHue}
-			aria-valuemin={LimitValue.min}
-			aria-valuenow={value}
-			className="clay-color-range clay-color-range-hue"
-			onKeyDown={(event) => onKeyDown(event)}
-			onPointerDown={(event) => {
-				event.preventDefault();
-
-				selectorActive.current = true;
-				onPointerMove(event);
-
-				(containerRef.current!.querySelector(
-					'.clay-color-range-pointer'
-				) as HTMLElement)!.focus();
-
-				window.addEventListener('pointermove', onPointerMove);
-				window.addEventListener('pointerup', removeListeners);
-			}}
-			ref={containerRef}
-			role="slider"
-			tabIndex={0}
-		>
-			<button
-				className="clay-color-pointer clay-color-range-pointer"
-				onBlur={() => {
-					selectorActive.current = false;
-				}}
+		<div className="clay-color-form-group">
+			<ClaySlider
+				className="clay-color-slider clay-color-slider-hue"
+				data-testid="hueSlider"
+				max={LimitValue.maxHue}
+				min={LimitValue.min}
+				onChange={onChange}
+				showTooltip={false}
+				step={1}
 				style={{
-					background: `hsl(${value}, 100%, 50%)`,
-					left: x - 7,
+					color: `hsl(${value}, 100%, 50%)`,
 				}}
-				type="button"
+				value={value}
 			/>
+			<ClayInput.Group>
+				<ClayInput.GroupItem>
+					<ClayInput
+						data-testid="hInput"
+						insetBefore
+						max="360"
+						min="0"
+						onChange={(event) => {
+							const value = event.target.value;
+
+							let newVal = Number(value);
+
+							if (newVal < 0) {
+								newVal = 0;
+							} else if (newVal > 360) {
+								newVal = 360;
+							}
+							onChange(newVal);
+						}}
+						type="number"
+						value={value}
+					/>
+					<ClayInput.GroupInsetItem before tag="label">
+						H
+					</ClayInput.GroupInsetItem>
+				</ClayInput.GroupItem>
+			</ClayInput.Group>
 		</div>
 	);
 };