Skip to content
This repository has been archived by the owner on Jun 21, 2023. It is now read-only.

Expose gesture protection area when tracking location #356

Merged
merged 3 commits into from
Apr 30, 2020

Conversation

LukasPaczos
Copy link

This PR brings 2 changes:

<changelog>Expose a `LocationComponentOptions#trackingMultiFingerProtectedMoveArea`. When the screen area is provided and `LocationComponentOptions#trackingGesturesManagement` is enabled, any gesture executed within this area will try avoiding breaking the tracking. </changelog>
<changelog>Fix an issue where the `onFling` listener was called even when the velocity of the gesture did not meet the threshold to actually animate the camera</changelog>

ezgif com-optimize (26)

Currently, the PR is based on the gesture's lib snapshot but whenever the changes are accepted, I'll bump the version of the library to a stable one.

@LukasPaczos LukasPaczos requested a review from a team April 29, 2020 14:27
This makes sure that we need to meet all of the thresholds and actually start the fling camera animation before dispatching the onFLing event. Previously, it would be dispatched as soon as the compat detector fired.
@LukasPaczos LukasPaczos force-pushed the lp-location-protected-gesture-area branch from 504c2ba to 7e9e734 Compare April 30, 2020 10:59
This option provides a way of defining a protected area on the screen which prevents breaking the camera location when executing a multi finger gesture as long as the focal point of the gesture is within this area.
@LukasPaczos LukasPaczos force-pushed the lp-location-protected-gesture-area branch from 5c8a961 to b581f3e Compare April 30, 2020 12:47
@LukasPaczos LukasPaczos force-pushed the lp-location-protected-gesture-area branch from b581f3e to d19b139 Compare April 30, 2020 13:02
@LukasPaczos LukasPaczos merged commit 77ea09e into master Apr 30, 2020
@LukasPaczos LukasPaczos deleted the lp-location-protected-gesture-area branch April 30, 2020 14:51
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants