refactor(core/form/inputs): function can now be moved out of scope

sanity-io · Dec 20, 2022 · ccdc0fd · ccdc0fd
1 parent 3fd22cc
commit ccdc0fd
Showing 1 changed file with 8 additions and 5 deletions.
diff --git a/packages/sanity/src/core/form/inputs/PortableText/object/BlockObject.tsx b/packages/sanity/src/core/form/inputs/PortableText/object/BlockObject.tsx
@@ -97,18 +97,21 @@ export function BlockObject(props: BlockObjectProps) {
     setTimeout(() => PortableTextEditor.focus(editor))
   }, [editor, path])
 
-  const blockPreview = useMemo(() => {
-    const _handleDelete = (e: React.MouseEvent<Element, MouseEvent>) => {
+  const handleDelete = useCallback(
+    (e: React.MouseEvent<Element, MouseEvent>) => {
       e.preventDefault()
       e.stopPropagation()
       onRemove()
-    }
+    },
+    [onRemove]
+  )
+
+  const blockPreview = useMemo(() => {
     return (
       <BlockObjectPreview
         focused={focused}
         isActive={isActive}
-        // eslint-disable-next-line react/jsx-no-bind
-        onClickingDelete={_handleDelete}
+        onClickingDelete={handleDelete}
         onClickingEdit={openItem}
         readOnly={readOnly}
         renderPreview={renderPreview}