fix(EditInPlace): removes focus when pressing esc or enter key (#5943)

* fix(EditInPlace): removes focus when pressing esc or enter key * fix: remove focus state on functions
carbon-design-system · Aug 28, 2024 · 5eff024 · 5eff024
1 parent 7ab472a
commit 5eff024
Showing 1 changed file with 7 additions and 4 deletions.
diff --git a/packages/ibm-products/src/components/EditInPlace/EditInPlace.tsx b/packages/ibm-products/src/components/EditInPlace/EditInPlace.tsx
@@ -207,13 +207,11 @@ export let EditInPlace = forwardRef<HTMLDivElement, EditInplaceProps>(
 
     const onSaveHandler = () => {
       setInitialValue(value);
-      setFocused(false);
       setDirtyInput(false);
       onSave();
     };
 
     const onCancelHandler = () => {
-      setFocused(false);
       setDirtyInput(false);
       onCancel(initialValue);
     };
@@ -248,16 +246,21 @@ export let EditInPlace = forwardRef<HTMLDivElement, EditInplaceProps>(
       onCancelHandler();
     };
 
+    const removeFocus = () => {
+      inputRef.current?.blur();
+      setFocused(false);
+    };
+
     const onKeyHandler = (e) => {
       // to prevent blur handler from being called twice add additional state to check if escape is being used
       escaping.current = true;
       switch (e.key) {
         case 'Escape':
-          inputRef.current?.blur();
+          removeFocus();
           escapeHandler();
           break;
         case 'Enter':
-          inputRef.current?.blur();
+          removeFocus();
           returnHandler();
           break;
         default: