fix: Unmounting actions dispatches close-modal when modal is already closed

[wout-o]

Description

This addresses the issue described in #10581.

The modal's close() function already closes the current modal and it also dispatches the modal-closed event, which when caught will cause the action's unmount function to be called. This function, however, does not take this into account and will always dispatch the close-modal event again.

This PR introduces a new variable $shouldDispatchCloseModal for the actions' unmount function, which when set to false will prevent the close-modal event from being dispatched again. Since this variable has a default value of true it should not break any other uses of the functions.

Visual changes

Improved UX when rapidly closing and reopening modals on complex pages or when the user has a slower connection.

Actions

Delete button
Before: https://demo.filamentphp.com/shop/orders/1/edit
After: https://demo.filament.thenines.be/shop/orders/1/edit

Table actions

Edit button
Before: https://demo.filamentphp.com/blog/categories
After: https://demo.filament.thenines.be/blog/categories

Form actions

Reset button
Before: https://demo.filamentphp.com/shop/orders/1/edit
After: https://demo.filament.thenines.be/shop/orders/1/edit

Infolist Actions

Author's suffix action
Before: not present in the demo
After: https://demo.filament.thenines.be/blog/posts/1

Functional changes

• Code style has been fixed by running the composer cs command.
• Changes have been tested to not break existing functionality.
• Documentation is up-to-date.

[danharrin]

This looks great, thank you so much for taking the time!