Docs: Notification doesn't close itself after 4.5s

[gudh]

First of all, thanks for building antd, I am amazed at how FANTASTIC it is! 🌷

Version

2.9.2

Environment

lastest MacOS, Chrome

Reproduction link

https://ant.design/components/notification/

Steps to reproduce

It's in the online docs

What is expected?

It should close itself after 4.5s

What is actually happening?

Notification doesn't close itself after 4.5s

Keep up great work! ❤️

[benjycui]

@afc163 close this issue?

[afc163]

Fixed in 2.9.3

[AsceticBoy]

online example, if you click several times frequently, after 4.5 secs, you will see after all Notifications close, have one Notification open again! I find component will in DidMount inject timer again @afc163 @benjycui

[afc163]

@AsceticBoy You can file a new issue about it.

[afc163]

@AsceticBoy I just revert react-component/notification#17 and released in rc-notification@2.0.2, you can try to reinstall node_modules.

cc @paranoidjk

[paranoidjk]

hmm... intersting bug 😅 I wonder why

[AsceticBoy]

A intersting bug, when set animateProps.component = '', it will find children[0] to return , so it will come DidMount to register Timer! Thx @afc163 resolve my problem

[paranoidjk]

The reason is animateProps.component changed when destroy the last but one notification instance, which caused the rc-animate root component tree rerendered, so the last notification actually is a totally new instance, so the timer is a restarted new timer too.

@afc163 But revert this will have influence on ant-design/ant-design-mobile#1389, please fix this or release rc-notication as a major upgrade.