-
-
Notifications
You must be signed in to change notification settings - Fork 32.3k
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
[TreeView] Update firstCharMap when a TreeItem is removed #20085
Conversation
Details of bundle changes.Comparing: c253b28...cdf4eb7 Details of page changes
|
Some observations after looking at the current tests. None of the tests test that the tabindex is set appropriately. |
closes #20074 |
Context is currently private implementation. The tests try to test features without touching implementation at all. |
@tonyhallett Please rebase with master |
…t/material-ui into clean-up-firstcharmap
Please can you get this through - it is a bug that was identified ( by @eps1lon #19882 (comment) ) https://codesandbox.io/s/goofy-buck-6f40o Type o in search Do the same with packages from this pull request resolved. |
@tonyhallett Thanks! |
Closes #20074