fix(app-admin): call onClose callback when dialog is closed

[swapnilmmane]

Changes

The useDialogs hook was not triggering the onClose callback when dialogs were dismissed. The closeDialog function in DialogsContext only removed the dialog from state without executing the user-provided onClose callback.

This fix retrieves the dialog from state before removal and calls the onClose callback if it exists, ensuring proper cleanup and event handling when dialogs are closed through any method (cancel button, accept button, or programmatic closure).

How Has This Been Tested?

I tested it manually using this code example.

import React, { useState } from "react";  
import { useDialogs } from "@webiny/app-admin";  
  
const DialogTest = () => {  
    const { showDialog } = useDialogs();  
    const [closeCount, setCloseCount] = useState(0);  
  
    const handleShowDialog = () => {  
        showDialog({  
            title: "Test Dialog",  
            content: "This is a test dialog to verify onClose callback works.",  
            acceptLabel: "Accept",  
            cancelLabel: "Cancel",  
            onAccept: (data) => {  
                console.log("Dialog accepted with data:", data);  
            },  
            onClose: () => {  
                console.log("Dialog closed!");  
                setCloseCount(prev => prev + 1);  
            }  
        });  
    };  
  
    return (  
        <div style={{ padding: "20px", border: "1px solid #ccc", margin: "10px" }}>  
            <h3>Dialog Test Component</h3>  
            <button onClick={handleShowDialog}>  
                Show Test Dialog  
            </button>  
            <p>Dialog closed {closeCount} times</p>  
        </div>  
    );  
};  
  
export default DialogTest;