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: Tabs inkBar in StrictMode #39653
Conversation
@@ -263,6 +267,7 @@ exports[`Tabs tabBarGutter should work 2`] = ` | |||
> | |||
<div | |||
class="ant-tabs-tab ant-tabs-tab-active" | |||
data-node-key="null" |
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.
null 需要渲染出来么?
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.
像是个 bug,我瞅瞅
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.
像是个 bug,我瞅瞅
Update:发现是测试用例写的比较随便,没给 key。另外 data-node-key
是 querySelector
需要用,总是有的。
size-limit report 📦
|
Codecov ReportBase: 100.00% // Head: 100.00% // No change to project coverage 👍
Additional details and impacted files@@ Coverage Diff @@
## master #39653 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 546 546
Lines 9366 9366
Branches 2656 2656
=========================================
Hits 9366 9366 Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
On the latest versions of Chrome the problem is fixed, but it seems that there is a problem for the older ones. I've tested with Chrome 45 (Ant 5.12) and the inkBar is missing. |
[中文版模板 / Chinese template]
🤔 This is a ...
🔗 Related issue link
fix #39631
fix #39494
💡 Background and solution
📝 Changelog
☑️ Self-Check before Merge