Skip to content

tab keyup behavior -- issue #114 #128

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

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

machinehum
Copy link

Change to not break standard HTML form tab key navigation behavior. DOES NOT affect using tab key to confirm selections in dropdown (that happens in the keydown event starting at line 209), only prevents tabbing between fields from firing search events via this switch's default clause.

Justification: The user does not expect a search to happen and a dropdown to show up for a field they've already tabbed away from, or for a search to happen on the pre-existing value for a field that they tab into. This fundamentally breaks standard browser behavior, and causes accessibility issues.

… to confirm selections in dropdown (that happens in the keydown event prior), only prevents tabbing between fields from firing search events via this switch's default clause .

The user does not expect a search to happen and a dropdown to show up for a field they've already tabbed away from, or for a search to happen on the pre-existing value for a field that they tab into.
@xcash
Copy link
Owner

xcash commented Aug 27, 2021

Hi @machinehum and thanks for the PR.
I'm currently on holiday so I can't check that everything is working correctly. All the new work I'm doing is on branch xcash-v300 with BS5 support.
I think your PR will still work on that branch, would you please test it?

Thanks a lot :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants