Fix iPad pointer and trackpad input - #96
alantangok wants to merge 2 commits into
Conversation
|
@SandiyosDev |
I'll review your changes in the next two weeks, if you'd like to help me get your changes merged faster, please test these PRs on your personal device as well Thank you for your PRs |
|
Thanks for the update! I’ll test the relevant changes on my iPad and run regression checks on my iPhone, then post the device models, OS versions, and results on each PR to help streamline the review. |
|
thanks for the split, reviewing this one first as you asked. One thing from my own testing before the code, on my iPad (iPadOS 16.6.1) a left click under pointer lock comes in as a direct touch with an empty buttonMask, not as an indirect pionter touch, so in relative mode the new recognizer won't see the click at all and it falls through to the finger path, that's the jump #88 was about which iPadOS did you test on, and did you try a host that captures the cursor? If you can make the locked click case work, I'll test your builds on my iPad to see if the bug can be reproduced... |
|
And a code review, the gate/mapper enums just return their argument, inline them, and pointerButtonTouchSupported is always true at our deployment target so the GameController button handlers are dead code now |
Summary
UIHoverGestureRecognizeras the sole absolute movement producerVerification
.checkpoint, orscripts/changesDevice validation still required
Split from #93.