Notifications doesn't removes from DOM

puranjayjain / react-materialui-notifications

Spec compliant notifications for react and material ui users

https://puranjayjain.github.io/react-materialui-notifications/

MIT License

251 stars 36 forks source link

Notifications doesn't removes from DOM #98

Closed irrvrsl closed 7 years ago

irrvrsl commented 7 years ago

Hi! I've explored your project's demo site and saw, that currently notifications doesn't removes from DOM, when clicking on 'close' button. Is it desired behavior?

puranjayjain commented 7 years ago

It must be a bug introduced with the latest version of material ui. Are you using a local copy of the demo? Can you also tell me your browser and environment (desktop / mobile)

irrvrsl commented 7 years ago

Chrome 55, react: 15.4.1 material-ui: 0.16.5, desktop env. I saw that on your https://puranjayjain.github.io/react-materialui-notifications/

trelly commented 7 years ago

lastest version material-ui for paper component did't support key prop.

puranjayjain commented 7 years ago

@trelly thank you for pointing out the reason shall fix this.

puranjayjain commented 7 years ago

this has been fixed in #116