fix: should not render when showProp is false

[paranoidjk]

No description provided.

[yesmeck]

Add a test.

[paranoidjk]

https://github.com/react-component/m-dialog/blob/master/src/DialogWrap.tsx#L27 rmc-dialog 会在 unmount 的时候尝试关闭 modal, 反而触发 animate 渲染流程。

导致的 bug 是在单页应用，B 页面的 Modal 在返回到 A 页面时候显示出来

[paranoidjk]

cc @zhang740

[coveralls]

Coverage remained the same at 70.0% when pulling c830ccb on fix-showProps into 528f278 on master.

[yesmeck]

rmc-dialog 会在 unmount 的时候尝试关闭 modal, 反而触发 animate 渲染流程。

这本来应该是 feature，unmount 的时候也会触发动画。

[paranoidjk]

对，但 showProp 的值此时是 false, 内容不应该渲染出来。

[yesmeck]

https://github.com/react-component/dialog/blob/master/src/DialogWrap.tsx#L48

要在动画结束的时候移除 container。

[yesmeck]

https://github.com/react-component/dialog/blob/master/src/DialogWrap.tsx#L46 这里也要判断 visible。

[paranoidjk]

在 rmc-dialog 层面也能修复 @silentcloud 看下

我先写个只用 rc-animate 复现的情况

[silentcloud]

要在动画结束的时候移除 container。

这个是做了的 https://github.com/react-component/m-dialog/blob/master/src/DialogWrap.tsx#L41

[silentcloud]

https://github.com/react-component/dialog/blob/master/src/DialogWrap.tsx#L46 这里也要判断 visible。

原来的逻辑，如果直接 component unmount 就 remove 的话 preact 下有问题

[yesmeck]

@silentcloud 意思是 rc-dialog 的做法在 preact 下会有问题吗？

[silentcloud]

@yesmeck 嗯

[coveralls]

Coverage increased (+1.3%) to 71.25% when pulling d098f2c on fix-showProps into 528f278 on master.

[zhang740]

@yesmeck review一下？

[paranoidjk]

再多测一下这个情况 https://github.com/react-component/m-dialog/blob/master/src/DialogWrap.tsx#L27

[zhang740]

跟测试用例的流程是一样的

[yesmeck]

感觉是 breaking change。现在 rc-dialog 就需要在 visible=false 的时候也需要渲染的。

[paranoidjk]

那就在 rmc-dialog 层面修复，state 里面记录当前的 visible 状态，componentWillUnmount 的时候不要盲目尝试 renderComponent(false) 条件判断一下 cc @zhang740 @silentcloud

[silentcloud]

@paranoidjk componentWillUnmount 如果不 render false，preact 下关闭的时候动画就没了

[paranoidjk]

@silentcloud 是条件判断。如果当前就是 visible: false 的，componentWillUnmount 为啥还要 render false？

[silentcloud]

@paranoidjk 哦，理解错误，这个情况是👌 的