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

[DateTimePickers] Remove the calendar/clock tabs #7121

Open
alexfauquette opened this issue Dec 7, 2022 · 7 comments
Open

[DateTimePickers] Remove the calendar/clock tabs #7121

alexfauquette opened this issue Dec 7, 2022 · 7 comments
Labels
component: DateTimePicker The React component. component: pickers This is the name of the generic UI component, not the React module! v6.x

Comments

@alexfauquette
Copy link
Member

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

@alexfauquette alexfauquette added v6.x component: pickers This is the name of the generic UI component, not the React module! component: DateTimePicker The React component. labels Dec 7, 2022
@LukasTy
Copy link
Member

LukasTy commented Dec 7, 2022

I'm not certain we should.

  1. Clicking on time tab does change the focused section to hours
  2. We are going to bring back the visual time selection behaviour eventually

I would be curious to hear @joserodolfofreitas and/or @gerdadesign take on this case. 🤔

@alexfauquette
Copy link
Member Author

To test it: https://next.mui.com/x/react-date-pickers/date-time-picker/

My option was
3. Remove the Tab

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.

@flaviendelangle
Copy link
Member

To remove the tabs, we just have to move the definition of the Tabs slot from NextDateTimePicker/shared.tsx to MobileNextDateTimePicker.tsx and StaticNextDateTimePicker.tsx.

I would prefer 3. over 2. (which will take time, if we decide to do it).
Between 1. and 3. I would tend toward 3. but no strong opinion.

@flaviendelangle flaviendelangle changed the title [DateTimePickers] Remove the claendar/clock tabs [DateTimePickers] Remove the calendar/clock tabs Dec 9, 2022
@gerdadesign
Copy link
Member

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.
Is it a pain to remove the bottom tab for now, and bring it back when we swap out the clock with a different selector?

@flaviendelangle
Copy link
Member

Is it a pain to remove the bottom tab for now, and bring it back when we swap out the clock with a different selector?

It is relatively easy to do

@flaviendelangle
Copy link
Member

@LukasTy is this still relevant now that we have the new UI ?

@LukasTy
Copy link
Member

LukasTy commented Nov 20, 2023

@flaviendelangle I'm not sure if it is still an issue that matters.
WDYT @alexfauquette?

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.
I've never seen a complaint from the community about this. Correct me if I'm wrong. 👌
To me it seems, that we could close this issue in it's current form, or we need to discuss what we actually need to do here. 🤔

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: DateTimePicker The React component. component: pickers This is the name of the generic UI component, not the React module! v6.x
Projects
None yet
Development

No branches or pull requests

4 participants