-
-
Notifications
You must be signed in to change notification settings - Fork 1.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
[pickers] Update pickers when new value has a distinct timezone #7853
Conversation
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.
Is there a similar problem on the new fields or is it purely a mask issue ?
Good points. It does not seem to be a problem in v6 @stem-ui If you could check v6 is working nice. Here is a codesandbox where you can switch between UTC and LA time zone. The field is correctly responding https://codesandbox.io/s/friendly-wilbur-pjudi8?file=/demo.tsx |
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.
🙌 Could we maybe add a test case asserting this behaviour and potentially forward-porting it to next
as well? 🤔
@alexfauquette , confirmed not an issue in v6 https://codesandbox.io/s/elated-austin-pld75s?file=/demo.tsx |
Fix #7833