Skip to content
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.

Sign up for GitHub

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

Jump to bottom

useHeights内htmlElement为null兼容 #259

Open
snowing-x opened this issue Mar 25, 2024 · 4 comments · May be fixed by #282
Open

useHeights内htmlElement为null兼容 #259

snowing-x opened this issue Mar 25, 2024 · 4 comments · May be fixed by #282

Comments

@snowing-x
Copy link

image htmlElement为null情况下，获取offsetHeight会报错，希望可以兼容一下
@yoyo837
Copy link
Member

yoyo837 commented Mar 26, 2024

来个复现到antd上提交个问题？

@lift46252
Copy link

image
image
using of deprecated findDOMNode throws error on component
and as i see findDOMNode is not needed here
image
Снимок экрана от 2024-07-03 09-58-15

lift46252 added a commit to bagrinsergiu/virtual-list that referenced this issue Jul 3, 2024
@lift46252
fix: offsetHeight null error (react-component#259)
6f9d98a
lift46252 added a commit to bagrinsergiu/virtual-list that referenced this issue Jul 3, 2024
@lift46252
Merge pull request #1 from bagrinsergiu/fix-offset_height_error
f1cecb5 
fix: offsetHeight null error (react-component#259)
@lift46252 lift46252 linked a pull request Jul 3, 2024 that will close this issue
Open
@lift46252
Copy link

lift46252 commented Jul 3, 2024 •
edited by yoyo837
Loading

PR #282 to fix #259

lift46252 added a commit to bagrinsergiu/virtual-list that referenced this issue Jul 4, 2024
@lift46252
fix: offsetHeight null error (react-component#259)
388d009
lift46252 added a commit to bagrinsergiu/virtual-list that referenced this issue Jul 4, 2024
@lift46252
fix: offsetHeight null error (react-component#259)
598fbd6
@lift46252
Copy link

@zombieJ #282 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: offsetHeight null error (#259) bagrinsergiu/virtual-list
3 participants
@yoyo837 @snowing-x @lift46252