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 show no options #6731

Merged
merged 4 commits into from
May 16, 2023
Merged

Fix: picker show no options #6731

merged 4 commits into from
May 16, 2023

Conversation

scomea
Copy link
Collaborator

@scomea scomea commented May 11, 2023

📖 Description

Picker "no options" display shows up one keystroke too late because the current detection happens before invalid options based on the last char typed are removed from the dom.

🎫 Issues

ad-ho

✅ Checklist

General

  • I have included a change request file using $ yarn change
  • I have added tests for my changes.
  • I have tested my changes.
  • I have updated the project documentation to reflect my changes.
  • I have read the CONTRIBUTING documentation and followed the standards for this project.

Component-specific

@awentzel awentzel merged commit 00140c2 into master May 16, 2023
@awentzel awentzel deleted the users/scomea/picker-no-options branch May 16, 2023 17:38
KingOfTac pushed a commit that referenced this pull request Jun 17, 2023
* show no options

* Change files

* Update change/@microsoft-fast-foundation-4ed56949-7343-439d-9bd1-4f76b36c9d1d.json

---------

Co-authored-by: Aaron Wentzel <16669785+awentzel@users.noreply.github.com>
Co-authored-by: Nicholas Rice <3213292+nicholasrice@users.noreply.github.com>
janechu pushed a commit that referenced this pull request Jun 10, 2024
* show no options

* Change files

* Update change/@microsoft-fast-foundation-4ed56949-7343-439d-9bd1-4f76b36c9d1d.json

---------

Co-authored-by: Aaron Wentzel <16669785+awentzel@users.noreply.github.com>
Co-authored-by: Nicholas Rice <3213292+nicholasrice@users.noreply.github.com>
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.

4 participants