From a9f96f9fc3a13a4af77095a04d7e85f758b32f97 Mon Sep 17 00:00:00 2001
From: Brian Ingles <brianingles@deephaven.io>
Date: Fri, 24 Feb 2023 11:10:01 -0600
Subject: [PATCH] fix: Disable applying "No formatting"

fixes #1106
---
 .../ColumnFormatEditor.tsx                    | 19 +++++++++++++------
 1 file changed, 13 insertions(+), 6 deletions(-)

diff --git a/packages/iris-grid/src/sidebar/conditional-formatting/ColumnFormatEditor.tsx b/packages/iris-grid/src/sidebar/conditional-formatting/ColumnFormatEditor.tsx
index bb7af1299e..997020e7cc 100644
--- a/packages/iris-grid/src/sidebar/conditional-formatting/ColumnFormatEditor.tsx
+++ b/packages/iris-grid/src/sidebar/conditional-formatting/ColumnFormatEditor.tsx
@@ -8,6 +8,7 @@ import {
   getConditionConfig,
   getDefaultStyleConfig,
   ModelColumn,
+  FormatStyleType,
 } from './ConditionalFormattingUtils';
 import ConditionEditor from './ConditionEditor';
 import StyleEditor from './StyleEditor';
@@ -83,14 +84,20 @@ function ColumnFormatEditor(props: ColumnFormatEditorProps): JSX.Element {
 
   useEffect(
     function updateColumnFormat() {
+      let isValid = conditionValid;
+
       if (selectedColumn === undefined) {
-        log.debug('Column is not selected, skip update.');
-        return;
+        log.debug('Column is not selected, invalidating update.');
+        isValid = false;
       }
-      if (selectedStyle === undefined) {
-        log.debug('Style is not selected, skip update.');
-        return;
+      if (
+        selectedStyle === undefined ||
+        selectedStyle.type === FormatStyleType.NO_FORMATTING
+      ) {
+        log.debug('Style is not selected, invalidating update.');
+        isValid = false;
       }
+
       const { type, name } = selectedColumn;
       const column = { type, name };
       onChange(
@@ -99,7 +106,7 @@ function ColumnFormatEditor(props: ColumnFormatEditorProps): JSX.Element {
           style: selectedStyle,
           ...conditionConfig,
         },
-        conditionValid
+        isValid
       );
     },
     [onChange, selectedColumn, selectedStyle, conditionConfig, conditionValid]