-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
refactor/findDomNode #6805
refactor/findDomNode #6805
Conversation
Deploying ant-design-mobile with Cloudflare Pages
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #6805 +/- ##
==========================================
+ Coverage 92.72% 92.74% +0.02%
==========================================
Files 335 335
Lines 7198 7193 -5
Branches 1767 1802 +35
==========================================
- Hits 6674 6671 -3
+ Misses 516 514 -2
Partials 8 8 ☔ View full report in Codecov by Sentry. |
Size Change: -101 B (-0.03%) Total Size: 370 kB
ℹ️ View Unchanged
|
size-limit report 📦
|
@@ -1,29 +1,29 @@ | |||
import React from 'react' | |||
import type { ReactNode } from 'react' | |||
import { findDOMNode } from 'react-dom' |
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.
findDOMNode
是没有替代物的,如果换掉的话是会 break change 的
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.
findDOMNode
是没有替代物的,如果换掉的话是会 break change 的
好, 那我等大版本规划在看看
为了 react19 的适配做准备.
因发现
findDomNode
将不再支持, 所以换了种获取实例的方式.通过增加一个同层级的
span
获取兄弟节点赋值, 并将span
设为display: none
.