Is there a way to execute onOpenChange on modals without making it a controlled component?

[DarkstarXDD]

Initially, I thought the onOpenChange on modal overlay's get called when the overlay opens/closes. But looks like that only gets called if I make it a controlled component and pass in the isOpen state.

For example, this will not work.

<DialogTrigger>
	<Button/>
	<ModalOverlay onOpenChange={() => console.log("something")}>
		<Modal>
			<Dialog></Dialog>
		</Modal>
	</ModalOverlay>
</DilaogTrigger>

In order for onOpenChange to execute I need to pass the isOpen state myself and the code will look like below.

<DialogTrigger>
	<Button/>
	<ModalOverlay isOpen={isOpen} onOpenChange={(isOpen) => {
		console.log("something")
		setIsOpen(isOpen)
	}}>
		<Modal>
			<Dialog></Dialog>
		</Modal>
	</ModalOverlay>
</DilaogTrigger>

I want to know if there is a way I can execute some code when the overlay opens/closes without making the component a controlled component?
I want to reset some form fields when the dialog closes and I was hoping to execute form.reset(defaultValues) with onOpenChange().

[snowystinger]

DialogTrigger should be taking over and controlling the state, so you should move the onOpenChange up to there. https://react-spectrum.adobe.com/react-aria/Dialog.html#dialogtrigger
It looks like you should be seeing a console warning about it? #6547

[DarkstarXDD]

@snowystinger Hey, thanks for the response!

Yeah, if I add the onOpenChange to the <DialogTrigger> it does work without me needing to pass the isOpen state. So that should work for me.

I feel like it would be better if you add this in the docs, inisde the props tables for both DialogTrigger and Modal components.

I didn't see any console message most probably because I am still on version 1.10.1 of RAC. Seems like your PR has been merged with version 1.11.0.

Thanks again!