From bf9c0d653be5af5473a7b616f2c91803db46beb3 Mon Sep 17 00:00:00 2001
From: Diana <diana.petcheva@sap.com>
Date: Thu, 14 Nov 2024 09:51:10 +0200
Subject: [PATCH] feat(ui5-color-picker): add simplified display mode (#10153)

Introducing a new property `displayMode` for the `ui5-color-picker` allowing users to select between the `default` (classic) color picker or the new `simplified` color picker.

The simplified picker does not include the RGB inputs and alpha slider.

You can create a color picker in simplified mode as follows:

<ui5-color-picker display-mode="Simplified"></ui5-color-picker>

Fixes: #9979
---
 packages/main/cypress/specs/ColorPicker.cy.ts |  27 ++++
 packages/main/src/ColorPicker.hbs             | 116 +++++++++---------
 packages/main/src/ColorPicker.ts              |  14 +++
 .../main/src/types/ColorPickerDisplayMode.ts  |  19 +++
 packages/main/test/pages/ColorPicker.html     |   8 ++
 .../_components_pages/main/ColorPicker.mdx    |   8 +-
 .../main/ColorPicker/Simplified/Simplified.md |   4 +
 .../main/ColorPicker/Simplified/main.js       |   1 +
 .../main/ColorPicker/Simplified/sample.html   |  20 +++
 9 files changed, 160 insertions(+), 57 deletions(-)
 create mode 100644 packages/main/cypress/specs/ColorPicker.cy.ts
 create mode 100644 packages/main/src/types/ColorPickerDisplayMode.ts
 create mode 100644 packages/website/docs/_samples/main/ColorPicker/Simplified/Simplified.md
 create mode 100644 packages/website/docs/_samples/main/ColorPicker/Simplified/main.js
 create mode 100644 packages/website/docs/_samples/main/ColorPicker/Simplified/sample.html

diff --git a/packages/main/cypress/specs/ColorPicker.cy.ts b/packages/main/cypress/specs/ColorPicker.cy.ts
new file mode 100644
index 000000000000..30805685506c
--- /dev/null
+++ b/packages/main/cypress/specs/ColorPicker.cy.ts
@@ -0,0 +1,27 @@
+import { html } from "lit";
+import "../../src/ColorPicker.js";
+import type ColorPicker from "../../src/ColorPicker.js";
+
+describe("Color Picker tests", () => {
+	it("should not display color channel inputs and alpha slider in simplified mode", () => {
+		cy.mount(html`<ui5-color-picker display-mode="Simplified"></ui5-color-picker>`);
+
+		cy.get("ui5-color-picker")
+			.as("colorPicker");
+
+		cy.get<ColorPicker>("@colorPicker")
+			.shadow()
+			.find(".ui5-color-picker-hex-input")
+			.should("exist");
+
+		cy.get<ColorPicker>("@colorPicker")
+			.shadow()
+			.find("#red")
+			.should("not.exist");
+
+		cy.get<ColorPicker>("@colorPicker")
+			.shadow()
+			.find("#alpha")
+			.should("not.exist");
+	});
+});
diff --git a/packages/main/src/ColorPicker.hbs b/packages/main/src/ColorPicker.hbs
index 73a256664477..76b5aad81cbe 100644
--- a/packages/main/src/ColorPicker.hbs
+++ b/packages/main/src/ColorPicker.hbs
@@ -23,16 +23,18 @@
             accessible-name="{{hueSliderLabel}}"
             @ui5-input="{{_handleHueInput}}"
         ></ui5-slider>
-        <ui5-slider
-            disabled="{{inputsDisabled}}"
-            class="ui5-color-picker-alpha-slider"
-            min="0"
-            max="1"
-            step="0.01"
-            value="{{_alpha}}"
-            accessible-name="{{alphaSliderLabel}}"
-            @ui5-input="{{_handleAlphaInput}}"
-        ></ui5-slider>
+        {{#if _isDefaultPickerMode}}
+            <ui5-slider
+                disabled="{{inputsDisabled}}"
+                class="ui5-color-picker-alpha-slider"
+                min="0"
+                max="1"
+                step="0.01"
+                value="{{_alpha}}"
+                accessible-name="{{alphaSliderLabel}}"
+                @ui5-input="{{_handleAlphaInput}}"
+            ></ui5-slider>
+        {{/if}}
     </div>
 
     <div class="ui5-color-picker-current-color">
@@ -56,51 +58,53 @@
         </div>
     </div>
 
-    <div
-        class="ui5-color-picker-rgb-wrapper"
-        @ui5-change="{{_handleRGBInputsChange}}"
-    >
-        <div class="ui5-color-picker-rgb">
-            <ui5-input
-                id="red"
-                class="ui5-color-picker-rgb-input"
-                disabled="{{inputsDisabled}}"
-                accessible-name="{{redInputLabel}}"
-                value="{{_value.r}}"
-            ></ui5-input>
-            <ui5-label>R</ui5-label>
-        </div>
-        <div class="ui5-color-picker-rgb">
-            <ui5-input
-                id="green"
-                class="ui5-color-picker-rgb-input"
-                disabled="{{inputsDisabled}}"
-                accessible-name="{{greenInputLabel}}"
-                value="{{_value.g}}"
-            ></ui5-input>
-            <ui5-label>G</ui5-label>
-        </div>
-        <div class="ui5-color-picker-rgb">
-            <ui5-input
-                id="blue"
-                class="ui5-color-picker-rgb-input"
-                disabled="{{inputsDisabled}}"
-                accessible-name="{{blueInputLabel}}"
-                value="{{_value.b}}"
-            ></ui5-input>
-            <ui5-label>B</ui5-label>
-        </div>
-        <div class="ui5-color-picker-rgb">
-            <ui5-input
-				id="alpha"
-                disabled="{{inputsDisabled}}"
-                class="ui5-color-picker-rgb-input"
-                value="{{_alpha}}"
-                accessible-name="{{alphaInputLabel}}"
-                @ui5-input="{{_handleAlphaInput}}"
-                @ui5-change="{{_handleAlphaChange}}"
-            ></ui5-input>
-            <ui5-label>A</ui5-label>
+    {{#if _isDefaultPickerMode}}
+        <div
+            class="ui5-color-picker-rgb-wrapper"
+            @ui5-change="{{_handleRGBInputsChange}}"
+        >
+            <div class="ui5-color-picker-rgb">
+                <ui5-input
+                    id="red"
+                    class="ui5-color-picker-rgb-input"
+                    disabled="{{inputsDisabled}}"
+                    accessible-name="{{redInputLabel}}"
+                    value="{{_value.r}}"
+                ></ui5-input>
+                <ui5-label>R</ui5-label>
+            </div>
+            <div class="ui5-color-picker-rgb">
+                <ui5-input
+                    id="green"
+                    class="ui5-color-picker-rgb-input"
+                    disabled="{{inputsDisabled}}"
+                    accessible-name="{{greenInputLabel}}"
+                    value="{{_value.g}}"
+                ></ui5-input>
+                <ui5-label>G</ui5-label>
+            </div>
+            <div class="ui5-color-picker-rgb">
+                <ui5-input
+                    id="blue"
+                    class="ui5-color-picker-rgb-input"
+                    disabled="{{inputsDisabled}}"
+                    accessible-name="{{blueInputLabel}}"
+                    value="{{_value.b}}"
+                ></ui5-input>
+                <ui5-label>B</ui5-label>
+            </div>
+            <div class="ui5-color-picker-rgb">
+                <ui5-input
+                    id="alpha"
+                    disabled="{{inputsDisabled}}"
+                    class="ui5-color-picker-rgb-input"
+                    value="{{_alpha}}"
+                    accessible-name="{{alphaInputLabel}}"
+                    @ui5-input="{{_handleAlphaInput}}"
+                    @ui5-change="{{_handleAlphaChange}}"
+                ></ui5-input>
+                <ui5-label>A</ui5-label>
+            </div>
         </div>
-    </div>
+    {{/if}}
 </section>
diff --git a/packages/main/src/ColorPicker.ts b/packages/main/src/ColorPicker.ts
index e2040b5fa32f..c75f5391d308 100644
--- a/packages/main/src/ColorPicker.ts
+++ b/packages/main/src/ColorPicker.ts
@@ -22,6 +22,7 @@ import ColorPickerTemplate from "./generated/templates/ColorPickerTemplate.lit.j
 import Input from "./Input.js";
 import Slider from "./Slider.js";
 import Label from "./Label.js";
+import ColorPickerDisplayMode from "./types/ColorPickerDisplayMode.js";
 
 import {
 	COLORPICKER_ALPHA_SLIDER,
@@ -111,6 +112,15 @@ class ColorPicker extends UI5Element implements IFormInputElement {
 	@property()
 	name?: string;
 
+	/**
+	 * Defines the display mode of the component.
+	 * @default "Default"
+	 * @public
+	 * @since 2.5.0
+	 */
+	@property()
+	displayMode: `${ColorPickerDisplayMode}` = "Default";
+
 	/**
 	 * Defines the HEX code of the currently selected color
 	 *
@@ -529,6 +539,10 @@ class ColorPicker extends UI5Element implements IFormInputElement {
 		return this._wrongHEX ? "Error" : undefined;
 	}
 
+	get _isDefaultPickerMode() {
+		return this.displayMode === ColorPickerDisplayMode.Default;
+	}
+
 	get styles() {
 		return {
 			mainColor: {
diff --git a/packages/main/src/types/ColorPickerDisplayMode.ts b/packages/main/src/types/ColorPickerDisplayMode.ts
new file mode 100644
index 000000000000..5914080b26d4
--- /dev/null
+++ b/packages/main/src/types/ColorPickerDisplayMode.ts
@@ -0,0 +1,19 @@
+/**
+ * ColorPicker display modes.
+ * @public
+ */
+enum ColorPickerDisplayMode {
+	/**
+	 * Default display mode. Includes all color picker features.
+	 * @public
+	 */
+	 Default = "Default",
+
+	/**
+	 * Simplified display mode. Doesn't include alpha slider and inputs for RGB values.
+	 * @public
+	 */
+	Simplified = "Simplified",
+}
+
+export default ColorPickerDisplayMode;
diff --git a/packages/main/test/pages/ColorPicker.html b/packages/main/test/pages/ColorPicker.html
index 3e2cac1f1e7d..7bf1cbf5b891 100644
--- a/packages/main/test/pages/ColorPicker.html
+++ b/packages/main/test/pages/ColorPicker.html
@@ -45,6 +45,14 @@
 
 	<ui5-color-picker id="cp3"></ui5-color-picker>
 
+	<br>
+	<br>
+	<br>
+
+	<ui5-title>Simplified Color Picker</ui5-title>
+	<br>
+	<ui5-color-picker id="cp-simplified" value="#dbf692" display-mode="Simplified"></ui5-color-picker>
+
 	<script>
 		var colorPicker = document.querySelector("#change-event"),
 			colorLabel = document.querySelector("#color-input"),
diff --git a/packages/website/docs/_components_pages/main/ColorPicker.mdx b/packages/website/docs/_components_pages/main/ColorPicker.mdx
index b62193f76f92..075124d82e11 100644
--- a/packages/website/docs/_components_pages/main/ColorPicker.mdx
+++ b/packages/website/docs/_components_pages/main/ColorPicker.mdx
@@ -3,10 +3,16 @@ slug: ../ColorPicker
 ---
 
 import Basic from "../../_samples/main/ColorPicker/Basic/Basic.md";
+import Simplified from "../../_samples/main/ColorPicker/Simplified/Simplified.md";
 
 <%COMPONENT_OVERVIEW%>
 
 ## Basic Sample
 <Basic />
 
-<%COMPONENT_METADATA%>
\ No newline at end of file
+<%COMPONENT_METADATA%>
+
+## More Samples
+
+### Simplified Picker
+<Simplified />
\ No newline at end of file
diff --git a/packages/website/docs/_samples/main/ColorPicker/Simplified/Simplified.md b/packages/website/docs/_samples/main/ColorPicker/Simplified/Simplified.md
new file mode 100644
index 000000000000..17798ecc59ab
--- /dev/null
+++ b/packages/website/docs/_samples/main/ColorPicker/Simplified/Simplified.md
@@ -0,0 +1,4 @@
+import html from '!!raw-loader!./sample.html';
+import js from '!!raw-loader!./main.js';
+
+<Editor html={html} js={js} />
diff --git a/packages/website/docs/_samples/main/ColorPicker/Simplified/main.js b/packages/website/docs/_samples/main/ColorPicker/Simplified/main.js
new file mode 100644
index 000000000000..cf4af63d9e43
--- /dev/null
+++ b/packages/website/docs/_samples/main/ColorPicker/Simplified/main.js
@@ -0,0 +1 @@
+import "@ui5/webcomponents/dist/ColorPicker.js";
\ No newline at end of file
diff --git a/packages/website/docs/_samples/main/ColorPicker/Simplified/sample.html b/packages/website/docs/_samples/main/ColorPicker/Simplified/sample.html
new file mode 100644
index 000000000000..1d89501b4cbf
--- /dev/null
+++ b/packages/website/docs/_samples/main/ColorPicker/Simplified/sample.html
@@ -0,0 +1,20 @@
+<!-- playground-fold -->
+<!DOCTYPE html>
+<html lang="en">
+
+<head>
+    <meta charset="UTF-8">
+    <meta name="viewport" content="width=device-width, initial-scale=1.0">
+    <title>Sample</title>
+</head>
+
+<body style="background-color: var(--sapBackgroundColor)">
+    <!-- playground-fold-end -->
+
+    <ui5-color-picker display-mode="Simplified" value="#F6A192">Picker</ui5-color-picker>
+    <!-- playground-fold -->
+    <script type="module" src="main.js"></script>
+</body>
+
+</html>
+<!-- playground-fold-end -->