search

rmariuzzo / react-new-window

🔲 Pop new windows in React, using `window.open`.
https://rmariuzzo.github.io/react-new-window/
MIT License
448 stars 110 forks source link

Fix for onUnload triggering as soon as window is opened (Safari) #135

Closed ShotSkydiver closed 2 years ago

ShotSkydiver commented 2 years ago

Proper fix for #57 . This uses on unload instead. Tested in latest Safari 16 and works fine.

ilhamgum commented 2 years ago

hey this should be pulled i think @rmariuzzo

rmariuzzo commented 2 years ago

I recently merged and published a related fix: https://github.com/rmariuzzo/react-new-window/pull/134

Note: if this doesn't fix issue, feel free to reopen.