window is not defined (SSR)

[nicksrandall]

Would it be possible to make this hook SSR compatible by wrapping this fn in a guard to only run on the client?

react-hotkeys-hook/src/isHotkeyPressed.ts

Line 42 in cc8aeed

window.addEventListener('DOMContentLoaded', () => {

[JohannesKlauss]

Yes, I'll fix that later this day.

[JohannesKlauss]

Fixed in 4.0.3

[nicksrandall]

@JohannesKlauss I'm still getting an issue when running server-side because of the usage of useLayoutEffect which is not supported in a server environment. I've seen many other libraries get around this issue by creating an abstraction called useSafeLayoutEffect which will run a regular useEffect on server and useLayoutEffect on client.

See: https://github.com/chakra-ui/chakra-ui/blob/260076e2ac6165c8079c800475b9d7a9ca0f29a2/packages/components/descendant/src/utils.ts#L53-L54

[JohannesKlauss]

This should be fixed with version 4.0.4