fix: include borders when testing insides of a shape (#7865)

10 months ago · 4987cc53d0
parent d917db438e
commit 4987cc53d0
1 changed files with 3 additions and 1 deletions
--- a/packages/excalidraw/element/collision.ts
+++ b/packages/excalidraw/element/collision.ts
@ -62,7 +62,9 @@ export const hitElementItself = ({
  frameNameBound = null,
 }: HitTestArgs) => {
  let hit = shouldTestInside(element)
-    ? isPointInShape([x, y], shape)
+    ? // Since `inShape` tests STRICTLY againt the insides of a shape
+      // we would need `onShape` as well to include the "borders"
+      isPointInShape([x, y], shape) || isPointOnShape([x, y], shape, threshold)
    : isPointOnShape([x, y], shape, threshold);

  // hit test against a frame's name