Fix trigger repeat animation

[shaodahong]

Ref: ant-design/ant-design#19498 (comment)

[vercel]

This pull request is being automatically deployed with ZEIT Now (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://zeit.co/react-component/menu/8dkhh3j92
🌍 Preview: https://menu-git-fix-trigger-repeat-animation.react-component.now.sh

[shaodahong]

这样应该就可以了，popup 有自己的动画，rc-trigger 只是提供 trigger 功能就行

[codecov]

Codecov Report

Merging #297 into master will decrease coverage by 0.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #297      +/-   ##
==========================================
- Coverage   93.74%   93.72%   -0.02%     
==========================================
  Files          11       11              
  Lines         799      797       -2     
  Branches      229      229              
==========================================
- Hits          749      747       -2     
  Misses         50       50

Impacted Files	Coverage Δ
src/SubMenu.tsx	97.97% <100%> (-0.03%)	⬇️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2f6c3c6...f54ab5b. Read the comment docs.

[zombieJ]

看看是不是应该在 rc-dropdown 里调整一下，rc-menu 比较底层。这里兼容性方法删了容易让其他没有改用新版的挂掉。

[shaodahong]

看看是不是应该在 rc-dropdown 里调整一下，rc-menu 比较底层。这里兼容性方法删了容易让其他没有改用新版的挂掉。

和 rc-dropdown 没关系，现在的问题根源是 rc-menu 的重复动画，单独用 rc-menu 也是有问题的，这个删了应该问题不大，毕竟 subMenu 是有 cssMotion 控制的，你传入 motionName 也是可以定义动画的

[shaodahong]

V7 版本我刚才看了也是没有的，popupMotion 是 V8 刚加的

[zombieJ]

单独用 rc-menu 也是有问题的

能给个重现看看吗？这个 motion 做兼容特地加的,我看看是我哪里疏忽了。

[shaodahong]

单独用 rc-menu 也是有问题的

能给个重现看看吗？这个 motion 做兼容特地加的,我看看是我哪里疏忽了。

https://deploy-preview-19498--ant-design.netlify.com/components/menu-cn/#components-menu-demo-vertical

你看下这个 menu 的动画

ant-design/ant-design#19498 (comment) 这个得截图也是上面的例子

[zombieJ]

挺神奇，rc-menu 里动画是正常的。我看看为啥。
https://menu.react-component.now.sh/?selectedKind=rc-menu&selectedStory=antd&full=0&addons=1&stories=1&panelRight=0&addonPanel=storybook%2Factions%2Factions-panel

[zombieJ]

嗯，你是对的。我发个版看看效果。

[zombieJ]

+ rc-menu@8.0.0-alpha.3

[shaodahong]

还有个问题，v3 也是有的，隐藏的时候动画还没有结束再触发就会有这种

[zombieJ]

如果 v3 也有，可以先放放。升级优先级更高些。

[zombieJ]

上面的代码里把 popupClassName 删了，这里一直空的。

[shaodahong]

……我说怎么一直都是 undefined, 所以我换成这个写法了，没看到，我修复下

[shaodahong]

之前没用 classNames 所以我看到页面渲染出 undefined 就换成这个了

[zombieJ]

不用了，已经 patch 了

[afc163]

close ant-design/ant-design#19602