ColorPickerInput: Allow returning empty color value (#52650)

* ColorPickerInput: Allow returning empty color value * ColorPickerInput: Do not call onChange if the value didn't change
3 years ago · 1e3135b18a
parent f9d8b68543
commit 1e3135b18a
1 changed files with 8 additions and 0 deletions
--- a/packages/grafana-ui/src/components/ColorPicker/ColorPickerInput.tsx
+++ b/packages/grafana-ui/src/components/ColorPicker/ColorPickerInput.tsx
@ -29,6 +29,14 @@ export const ColorPickerInput = forwardRef<HTMLInputElement, ColorPickerInputPro

    useThrottleFn(
      (c) => {
+        if (c === value) {
+          return;
+        }
+        // Default to an empty string if no color value is available
+        if (!c) {
+          onChange('');
+          return;
+        }
        const color = theme.visualization.getColorByName(c);
        if (returnColorAs === 'rgb') {
          onChange(colorManipulator.asRgbString(color));