[Toast] Ability to remove `document` event listeners

Feature request

Overview

Currently, the Toast package adds an event listener to the document for managing hotkeys internally.

https://github.com/radix-ui/primitives/blob/3e5644edadcded918db177368c3fe9654ce9885f/packages/react/toast/src/Toast.tsx#L169-L170

I’d like the ability to disable this so that I have full control over Toast hotkey handling. One option would be simply passing an empty array to the hotkeys prop. Ideally, Toast would check the length of the hotkey prop before adding an event handler to document and, if hotkey was length 0, then no handler would be added.

Note: the hotkey listeners were implemented using Array#every() which returns true for an empty array.

Who does this impact? Who is this for?

Users of the Toast package.

Additional context

In my application, I have a sophisticated, custom service which is responsible for handling all hotkeys. I’d like to manage toast hotkeys that way.

[benoitgrelard]

Oh ya, just tested. It works the way I thought it did (i.e. it affects other listeners). Your comment made me think that maybe I’ve had a huge misunderstanding of javascript event listeners for several years now 😅.

No you’re right, it’s just that in other places some of these cancellable events we have are custom events we raise, so those wouldn’t affect any other events. In this case we are piggy-backing on the keyboard event so it would.

[jjenzz]

In this case we are piggy-backing on the keyboard event so it would.

oh, was that a mistake on our part? seems it should be a CustomEvent no?