Skip to content

Commit 90719ab

Browse files
committed
fix isOver on nested drop targets
1 parent fbb6ce1 commit 90719ab

1 file changed

Lines changed: 4 additions & 10 deletions

File tree

src/MouseBackend.js

Lines changed: 4 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -175,16 +175,10 @@ export default class MouseBackend {
175175

176176
e.preventDefault()
177177

178-
const matchingTargetIds = Object.keys(this.targetNodes)
179-
.filter((targetId) =>
180-
{
181-
const boundingRect =
182-
this.targetNodes[targetId].getBoundingClientRect()
183-
return clientOffset.x >= boundingRect.left &&
184-
clientOffset.x <= boundingRect.right &&
185-
clientOffset.y >= boundingRect.top &&
186-
clientOffset.y <= boundingRect.bottom
187-
})
178+
const matchingTargetIds = Object.entries(this.targetNodes)
179+
.filter(([, node]) => node.contains(e.toElement))
180+
.sort(([, nodeA], [, nodeB]) => (nodeA.contains(nodeB) ? -1 : 1))
181+
.map(([id]) => id);
188182

189183
this.actions.hover(matchingTargetIds, {
190184
clientOffset

0 commit comments

Comments
 (0)