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 #795

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.

@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 857b490 into master Oct 4, 2023
@YoussefHenna YoussefHenna deleted the youssef/p-4110-picker-dropdown-issue-with-touch-delay-on-android 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
None yet
Development

Successfully merging this pull request may close these issues.

2 participants