-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
[pickers] Fix focus management #6914
Conversation
@@ -21,7 +21,6 @@ interface DateViewRendererProps<TDate> | |||
export const renderDateView = <TDate extends unknown>(props: DateViewRendererProps<TDate>) => ( | |||
<DateCalendar | |||
{...props} | |||
autoFocus |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
autoFocus
has no effect if focusedView
is controlled
These are the results for the performance tests:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The behavior make sense
If you could add a test case for the fix it does, that would be great
No description provided.