Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix picker issue when long press (47) #796

Conversation

YoussefHenna
Copy link
Collaborator

  • Picker had some inconsistent behavior when triggering it using a long press on Android. Explicitly adding the onLongPress prop addresses the issue.

@YoussefHenna YoussefHenna added the legacy For a legacy branch using an older Expo version label Oct 4, 2023
@linear
Copy link

linear bot commented Oct 4, 2023

P-4110 Picker Dropdown Issue with Touch Delay on Android

A user has reported an issue related to the touch sensitivity or delay for triggering the picker dropdown on Android device. Specifically, the problem arises when the picker is clicked a tad bit slower, making it appear as if it's stuck. This behavior is not replicated when the picker is clicked rapidly.

Observation on Live Preview for Android device:

  • When the picker is clicked rapidly, the dropdown behaves as expected.
  • Slower clicks seem to initiate some long-press behavior, causing the picker dropdown to get stuck.
  • If another picker is clicked after the first one appears stuck, both dropdowns open simultaneously.

Potential Cause: The issue might be related to a long-press delay on Android that's too short, causing the dropdown to misbehave.

App Link: https://build.draftbit.com/apps/tZbmlHTe/screens/us9saIbZ?b=ConsoleLogs&c=u1jw3uTY&p=Interactions&t=ON_SCREEN_FOCUS&v=AndroidPreview

User Contact: joannejoannecooper.co.za

@github-actions
Copy link

github-actions bot commented Oct 4, 2023

Published version: @draftbit/[email protected]

@YoussefHenna YoussefHenna merged commit c44a0bc into 47 Oct 4, 2023
1 check passed
@YoussefHenna YoussefHenna deleted the youssef/p-4110-picker-dropdown-issue-with-touch-delay-on-android47 branch October 4, 2023 17:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
legacy For a legacy branch using an older Expo version
Development

Successfully merging this pull request may close these issues.

2 participants