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

[DateTimeRangePicker] Fix validation behavior #12243

Merged
merged 19 commits into from
Feb 29, 2024
Merged

Conversation

LukasTy
Copy link
Member

@LukasTy LukasTy commented Feb 28, 2024

Fixes #12048

Follow-up on #9528

Closes #12101

  • initialize disableIgnoringDatePartForTimeValidation accordingly (the main issue fix) - same as in DateTimePicker
  • Initialize minDate and maxDate based on minDateTime and maxDateTime respectively
  • Add describes tests to DesktopDateTimeRangePicker, MobileDateTimeRangePicker, and DateTimeRangePicker
  • Add a few specific (validation) DateTimeRangePicker tests
  • Fix to not render DateTimeRangePickerToolbar (root element) when it is hidden
  • Fix DateTimeRangePickerToolbar titleId (aria-labelledby on dialog) behavior (for mobile)
  • Set aria-disabled on a disabled MultiDigitalClockSection item

@LukasTy LukasTy added test component: pickers This is the name of the generic UI component, not the React module! component: DateTimeRangePicker The React component labels Feb 28, 2024
@LukasTy LukasTy self-assigned this Feb 28, 2024
@mui-bot
Copy link

mui-bot commented Feb 28, 2024

Deploy preview: https://deploy-preview-12243--material-ui-x.netlify.app/

Generated by 🚫 dangerJS against 54ae977

Copy link
Member

@michelengelen michelengelen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

just a few details! looks good otherwise! 💪🏼

@LukasTy LukasTy merged commit a31fb9f into mui:next Feb 29, 2024
17 checks passed
@LukasTy LukasTy deleted the add-dtr-tests branch February 29, 2024 08:59
thomasmoon pushed a commit to thomasmoon/mui-x that referenced this pull request Sep 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: DateTimeRangePicker The React component component: pickers This is the name of the generic UI component, not the React module! test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[pickers] DateTimeRangePicker disableFuture disables Time in the Past
3 participants