-
-
Notifications
You must be signed in to change notification settings - Fork 319
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
fix: panel flick in some edge case #439
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
(direction === 'rtl' || arrowRef.current.offsetLeft > arrowLeft | ||
? 0 | ||
: arrowRef.current.offsetLeft) | ||
(direction === 'rtl' ? 0 : arrowMarginLeft) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
解释一下?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这里是暂时没考虑 rtl 的 case,所以先忽略成 0,marginRight 相对来说算起来麻烦一些
Codecov Report
@@ Coverage Diff @@
## master #439 +/- ##
=======================================
Coverage 99.53% 99.53%
=======================================
Files 46 46
Lines 2148 2149 +1
Branches 633 634 +1
=======================================
+ Hits 2138 2139 +1
Misses 8 8
Partials 2 2
📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more |
@afc163 could you please bump a new package version? |
close ant-design/ant-design#36507