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: prevent autocomplete from jumping to next autocomplete and opening… #5684

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

martinszeltins
Copy link

When an option is focused while the dropdown is open and pressing the tab key it jumps to the next autocomplete input component and opens the options dropdown and then jumps back to the previous autocomplete input leaving the other one haning.

This fix prevents jumping to the next input when an option is focused, it selects the value instead like a native dropdown and then pressing tab again jumps to the next input. Fixes dangling dropdown issue.

… the options dropdown

When an option is focused while the dropdown is open and pressing the tab key it jumps to the next autocomplete input component and opens the options dropdown and then jumps back to the previous autocomplete input leaving the other one haning. 

This fix prevents jumping to the next input when an option is focused, it selects the value instead like a native dropdown and then pressing tab again jumps to the next input. Fixes dangling dropdown issue.
Copy link

vercel bot commented May 3, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

2 Ignored Deployments
Name Status Preview Comments Updated (UTC)
primevue ⬜️ Ignored (Inspect) Visit Preview May 3, 2024 11:07am
primevue-v4 ⬜️ Ignored (Inspect) Visit Preview May 3, 2024 11:07am

@tugcekucukoglu
Copy link
Member

Please create an issue with a reproducer link.

@tugcekucukoglu tugcekucukoglu added the Resolution: Needs Revision The pull request can't be merged. Conflicts need to be corrected or documentation and code updated. label May 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Resolution: Needs Revision The pull request can't be merged. Conflicts need to be corrected or documentation and code updated.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants