-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
[DateTimePickers] Remove the calendar/clock tabs #7121
Comments
I'm not certain we should.
I would be curious to hear @joserodolfofreitas and/or @gerdadesign take on this case. 🤔 |
To test it: https://next.mui.com/x/react-date-pickers/date-time-picker/ My option was Maybe it's because I'm used to the previous behavior, but when pressing a Tab, I expect the associated view to switch to another one, and at the same place. |
To remove the tabs, we just have to move the definition of the I would prefer 3. over 2. (which will take time, if we decide to do it). |
The tabs should theoretically change the content above it. Right now, clicking the clock is closing the modal and focusing the HH input, which breaks expected behaviour and feel accidental since it closes the modal. The highlight is fairly small compared to the large pop up, so can be easily missed that it's highlighting. |
It is relatively easy to do |
@LukasTy is this still relevant now that we have the new UI ? |
@flaviendelangle I'm not sure if it is still an issue that matters. From the video/screenshot it seems that we might had a bug previously, where we always showed the tabs on the desktop, but if that was the case - we no longer do that. |
The problem
The new Date Time Picker component show the tabs calendar/clock. So when you click on it, you expect to see the clock which has been removed
Screencast.from.07-12-2022.10.17.07.mp4
The expected behavior
If there is no clock, it should not have a Tabs in the view
The text was updated successfully, but these errors were encountered: