Fix pointer keys drag config (#553)

* fix: correctly resolve pointer.keys

* tests: add test for drag and keydown

* ci: changeset

Author: dbismut

.changeset/great-carrots-grow.md

@@ -0,0 +1,5 @@
+---
+'@use-gesture/core': patch
+---
+
+fix: properly resolve pointer.keys config

packages/core/src/config/dragConfigResolver.ts

@@ -37,13 +37,16 @@ export const dragConfigResolver = {
     if (!SUPPORT.touchscreen || preventScroll === false) return undefined
     return value ? value : preventScroll !== undefined ? 'y' : undefined
   },
-  pointerCapture(this: InternalDragOptions, _v: any, _k: string, { pointer: { capture = true, buttons = 1 } = {} }) {
+  pointerCapture(
+    this: InternalDragOptions,
+    _v: any,
+    _k: string,
+    { pointer: { capture = true, buttons = 1, keys = true } = {} }
+  ) {
     this.pointerButtons = buttons
+    this.keys = keys
     return !this.pointerLock && this.device === 'pointer' && capture
   },
-  keys(value = true) {
-    return value
-  },
   threshold(
     this: InternalDragOptions,
     value: number | Vector2,

test/drag.test.tsx

@@ -390,4 +390,21 @@ describe.each([
     expect(getByTestId(`${prefix}drag-active`)).toHaveTextContent(`false`)
     fireEvent.pointerUp(element, { pointerId: 26 })
   })
+
+  test(`drag should react to key press`, () => {
+    rerender(<Component gestures={['Drag']} />)
+    fireEvent.keyDown(element, { key: 'ArrowDown' })
+    expect(getByTestId(`${prefix}drag-active`)).toHaveTextContent(`true`)
+    expect(getByTestId(`${prefix}drag-offset`)).toHaveTextContent(`15,40`)
+    fireEvent.keyDown(element, { key: 'ArrowLeft', shiftKey: true })
+    expect(getByTestId(`${prefix}drag-offset`)).toHaveTextContent(`-85,40`)
+    fireEvent.keyUp(element, { key: 'ArrowDown' })
+    expect(getByTestId(`${prefix}drag-active`)).toHaveTextContent(`false`)
+  })
+
+  test(`drag should not react to key press when pointer.key = false`, () => {
+    rerender(<Component gestures={['Drag']} config={{ drag: { pointer: { keys: false } } }} />)
+    fireEvent.keyDown(element, { key: 'ArrowDown' })
+    expect(getByTestId(`${prefix}drag-active`)).toHaveTextContent(`false`)
+  })
 })