[Dialog][Modal] Remove disableBackdropClick

[eps1lon]

Breaking changes

• [Dialog] Remove the disableBackdropClick prop. It's redundant with the reason argument.

  <Dialog
  - disableBackdropClick
  - onClose={handleClose}
  + onClose={(event, reason) => {
  +   if (reason !== 'backdropClick') {
  +     handleClose(event, reason);
  +   }
  + }}
  />

• [Modal] Remove the disableBackdropClick prop. It's redundant with the reason argument.

  <Modal
  - disableBackdropClick
  - onClose={handleClose}
  + onClose={(event, reason) => {
  +   if (reason !== 'backdropClick') {
  +     handleClose(event, reason);
  +   }
  + }}
  />

---

disableEscapeKeyDown is on the chopping block as well but will be handed separately.

Also includes onEscapeKeydown migration guide as a follow-up to #23571

Affected demos:

• https://deploy-preview-23607--material-ui.netlify.app/components/dialogs/#confirmation-dialogs
• https://deploy-preview-23607--material-ui.netlify.app/components/selects/#with-a-dialog

[mui-pr-bot]

Details of bundle changes

Generated by 🚫 dangerJS against 1cff479

[eps1lon]

They don't have any effect on next either because we already have a fully custom onClose in ConfirmationDialogRaw

[shiraze]

Shouldn't the example code still call handleClose(event)?

[oliviertassinari]

@shiraze Please be specific.

[shiraze]

I think initial code examples by @eps1lon should be:

Breaking changes

• [Dialog] Remove the disableBackdropClick prop. It's redundant with the reason argument.

<Dialog
- disableBackdropClick
- onClose={handleClose}
+ onClose={(event, reason) => {
+   if (reason !== 'backdropClick') {
+     handleClose(event);
+   }
+ }}
/>

• [Modal] Remove the disableBackdropClick prop. It's redundant with the reason argument.

<Modal
- disableBackdropClick
- onClose={handleClose}
+ onClose={(event, reason) => {
+   if (reason !== 'backdropClick') {
+     handleClose(event);
+   }
+ }}
/>

[oliviertassinari]

Yes, updated, thanks.