[Dialog] Consider adding `disableOutsidePointerEvents`

[Avizura]

Feature request

Overview

I need to remove disableOutsidePointerEvents feature in Dialog component but it seems impossible https://github.com/radix-ui/primitives/blob/main/packages/react/dialog/src/Dialog.tsx#L241.

Additional context

Such option is already exists in DropdownMenu.Content component, would be useful to have it in Dialog as well.

[benoitgrelard]

Hi @Avizura,

I don't think it would make sense to allow that for our Dialog primitive as it's a modal dialog.
What is your use case?

[Avizura]

Hi @benoitgrelard !
I need to able to click on other cards while dialog with details of current card is opened. But now I have doubts that sidebar on the right is actually a dialog 😅

[benoitgrelard]

Yeah so if you need to be able to interact with what's behind, that would probably be considered a non-modal dialog.
I don't think our Dialog primitive currently supports this well so there may be something there.

Popover lets you create non-modal things, but then here it's not really what you need as you don't need the whole positioning thing.

I'm currently re-writing bits of our focus scope logic so it's kinda good timing actually haha.
Let me consider your use case a little bit and discuss with @jjenzz as well and we'll figure something out.

[Avizura]

I see, thanks a lot for this lib guys

[benoitgrelard]

Hi @Avizura I just wanted to come back to you to say that we think we will be making changes to our components such as Dialog, Popover, etc to add the concept of modality. This should then resolve your issue here.

Not sure what timeframe that will be released but I will update you once that's done.

[Avizura]

@benoitgrelard Hi, cool, thanks!

[jjenzz]

@Avizura Sorry for the delay on an update here.

We're hoping to release this week with a new modal prop for Dialog. Setting it to false would allow you to achieve the UI you posted 🙂

[Avizura]

I see, thanks again for your amazing work @jjenzz

[jjenzz]

@Avizura thanks but it was @benoitgrelard and @andy-hook who did this work so I cannot take credit I'm afraid 🙂