@sanity/ui 2.8.1 → 2.8.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sanity/ui",
-  "version": "2.8.1",
+  "version": "2.8.2",
   "keywords": [
     "sanity",
     "ui",
@@ -161,7 +161,7 @@
     "eslint-plugin-boundaries": "^4.2.2",
     "eslint-plugin-import": "^2.29.1",
     "eslint-plugin-jsx-a11y": "^6.9.0",
-    "eslint-plugin-prettier": "^5.1.3",
+    "eslint-plugin-prettier": "^5.2.1",
     "eslint-plugin-react": "^7.34.4",
     "eslint-plugin-react-compiler": "0.0.0-experimental-a97cca1-20240529",
     "eslint-plugin-react-hooks": "^4.6.2",
@@ -183,7 +183,7 @@
     "semantic-release": "^24.0.0",
     "start-server-and-test": "^2.0.4",
     "storybook": "^8.2.4",
-    "styled-components": "^6.1.11",
+    "styled-components": "^6.1.12",
     "tsconfig-paths": "^4.2.0",
     "typescript": "5.5.3",
     "vite": "^5.3.4",

package/src/core/components/dialog/dialog.tsx

@@ -216,18 +216,21 @@ const DialogCard = forwardRef(function DialogCard(
   )
 
   useClickOutside(
-    (event: MouseEvent) => {
-      if (!isTopLayer || !onClickOutside) return
+    useCallback(
+      (event) => {
+        if (!isTopLayer || !onClickOutside) return
 
-      const target = event.target as Node | null
+        const target = event.target as Node | null
 
-      if (target && !isTargetWithinScope(boundaryElement, portalElement, target)) {
-        // Ignore clicks outside of the scope
-        return
-      }
+        if (target && !isTargetWithinScope(boundaryElement, portalElement, target)) {
+          // Ignore clicks outside of the scope
+          return
+        }
 
-      onClickOutside()
-    },
+        onClickOutside()
+      },
+      [boundaryElement, isTopLayer, onClickOutside, portalElement],
+    ),
     [rootElement],
   )
 

package/src/core/components/menu/menu.tsx

@@ -93,7 +93,13 @@ export const Menu = forwardRef(function Menu(
   }, [activeIndex, onItemSelect])
 
   // Close menu when clicking outside
-  useClickOutside((event) => isTopLayer && onClickOutside && onClickOutside(event), [rootElement])
+  useClickOutside(
+    useCallback(
+      (event) => isTopLayer && onClickOutside && onClickOutside(event),
+      [isTopLayer, onClickOutside],
+    ),
+    [rootElement],
+  )
 
   // Close menu when pressing Escape
   useGlobalKeyDown(