mui / mui/base-ui

`AlertDialog`: add support for light dismiss?

Open
#3,339 3 comments 0 reactions 0 assignees View on GitHub
component: alert dialog support: question
Dominant language
TypeScript
Stars
10.9k
Forks
543
Avg merge
1d 20h
Merged PRs (30d)
101

Description

Base UI's `Dialog` has support for light dismiss (enabled by default and optionally disabled via `disablePointerDismissal`), whereas `AlertDialog` doesn't seem to support light dismiss.

I'm guessing this was done to match system OS behaviour and to force the user to make an explicit decision, but I don't think this will suit every use case.

I don't think it makes sense to switch to `Dialog` just for this because it has a different accessibility role.

I wonder if it would make sense to add a prop to `AlertDialog` to optionally enable light dismiss?

For context, at Unsplash we have this `AlertDialog` and we would like it to support light dismiss (speaking for the product team collectively here rather than my own personal opinion 😁).

Image

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.