Skip to content

fix menu popuptrigger flicker #497

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Nov 10, 2022
Merged

Conversation

JarvisArt
Copy link
Contributor

@JarvisArt JarvisArt commented Jul 26, 2022

切换 inline mode 后菜单栏点击展开最深层会出现上个 mode 的弹窗
ant-design 相关 issues ant-design/ant-design#36636

Rec.0001.mp4

@codecov
Copy link

codecov bot commented Jul 26, 2022

Codecov Report

Merging #497 (e5d4847) into master (b296551) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #497   +/-   ##
=======================================
  Coverage   99.85%   99.85%           
=======================================
  Files          26       26           
  Lines         696      698    +2     
  Branches      188      188           
=======================================
+ Hits          695      697    +2     
  Misses          1        1           
Impacted Files Coverage Δ
src/SubMenu/index.tsx 100.00% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@JarvisArt
Copy link
Contributor Author

@zombieJ @afc163

@zhengweipx
Copy link

zhengweipx commented Aug 29, 2022

这个问题还没修复吗? @JarvisArt

@JarvisArt JarvisArt force-pushed the master branch 2 times, most recently from 56ca54a to eacff32 Compare September 11, 2022 16:55
@JarvisArt JarvisArt requested a review from zombieJ September 11, 2022 16:58
@JarvisArt JarvisArt force-pushed the master branch 3 times, most recently from fde0081 to efc121d Compare October 24, 2022 16:09
@JarvisArt JarvisArt requested review from zombieJ October 25, 2022 15:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants