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

Select.Trigger cannot focus back using keyboard #57

Closed
tienpv222 opened this issue Jan 19, 2023 · 3 comments · Fixed by #68
Closed

Select.Trigger cannot focus back using keyboard #57

tienpv222 opened this issue Jan 19, 2023 · 3 comments · Fixed by #68
Labels
bug Something isn't working WIP Work in progress

Comments

@tienpv222
Copy link

tienpv222 commented Jan 19, 2023

To Reproduce
https://stackblitz.com/edit/node-z2nn98?file=src%2Fmain.tsx

In the demo, there are 2 tabs containing a TextField and a Select. The one with TextField can tab back to Tabs.List, while the other cannot.

Additional context
There are two other issues that I'm not sure how they are related, so I'll list them here just in case.

  1. Using Select in dev mode complains Computations created outside a 'createRoot' or 'render' will never be disposed. This can be seen in the demo above too.

  2. Select.Item not showing data-hover on hover. The image below is from the docs site.

    chrome_KMHJisZiRN

@fabien-ml fabien-ml changed the title Select inside Tabs.Content cannot focus back to Tabs.List using keyboard Select.Trigger cannot focus back using keyboard Jan 19, 2023
@fabien-ml
Copy link
Collaborator

Hi, thanks for the report. It looks like it's not just in Tabs. i'm digging into the problem

@fabien-ml fabien-ml added bug Something isn't working WIP Work in progress labels Jan 19, 2023
@fabien-ml
Copy link
Collaborator

fixed in v0.4.0.

@tienpv222
Copy link
Author

fixed in v0.4.0.

That was super fast, thank you a lot.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working WIP Work in progress
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants