search

minhtranite / react-notifications

Notification component for ReactJS
https://www.npmjs.com/package/react-notifications
287 stars 64 forks source link

Refactor <Notifications /> with hooks #57

Open edumoreira1506 opened 4 years ago

edumoreira1506 commented 4 years ago

Related to issue #54

chitgoks commented 4 years ago

is the author still active? this is the best design of notifications i have come across and hopefully these pull requests will be merged. those warning messages take up a huge chunk in the console.

edumoreira1506 commented 4 years ago

I don't know if actor is active, but I hope that yes.

chitgoks commented 4 years ago

or you could fork it?

tjrexer commented 4 years ago

I forked this repo and have taken ownership from the original owner in npm. I've published an update to remove the unsafe components. If you would like this to be reviewed, please submit a PR into tjrexer/react-notifications:next.

chitgoks commented 4 years ago

I forked this repo and have taken ownership from the original owner in npm. I've published an update to remove the unsafe components. If you would like this to be reviewed, please submit a PR into tjrexer/react-notifications:next.

thank you for keeping this alive. i tried your new lib. but i still get the same unsafe warnings in dev console. seems there is still one remaining called TransitionGroup.