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

Column resizing followup - task: Fix focus marshalling to resizer #3027

Closed
snowystinger opened this issue Apr 11, 2022 · 0 comments · Fixed by #3258
Closed

Column resizing followup - task: Fix focus marshalling to resizer #3027

snowystinger opened this issue Apr 11, 2022 · 0 comments · Fixed by #3258

Comments

@snowystinger
Copy link
Member

🐛 Bug Report

Followup issue #3007 item:

Remove setTimeout that is compensating for restoreFocus so that selecting resize from the menu actually moves focus to the resizer

When using the Column Header Menu to initiate resizing, focus is not moved to the resizer. This is especially problematic when keyboard navigating because there is not other way for a keyboard user to initiate resizing.

🤔 Expected Behavior

😯 Current Behavior

💁 Possible Solution

We may be able to remove the setTimeout which tries to wait for the menu to close before moving focus to the resizer. If focus has been moved, restore focus should not try to move it again.

🔦 Context

💻 Code Sample

🌍 Your Environment

Software Version(s)
react-spectrum
Browser
Operating System

🧢 Your Company/Team

🕷 Tracking Issue (optional)

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

Successfully merging a pull request may close this issue.

2 participants