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.

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

Table的columns属性发生变化时,设置scroll属性,页面报错 #19274

Closed
1 task
admire7444 opened this issue Oct 17, 2019 · 3 comments
Closed
1 task
Assignees
Labels
🐛 Bug Ant Design Team had proved that this is a bug.

Comments

@admire7444
Copy link

  • I have searched the issues of this repository and believe that this is not a duplicate.

Reproduction link

Edit on CodeSandbox

Steps to reproduce

please see the page link above
请看重现链接

What is expected?

show a normal table
显示正常表格

What is actually happening?

page error, according to the error log, i think the error was that the filterDropdown can not find the mount node
页面报错,根据排查,应该是filterDropdown渲染的节点找不到引起的问题

Environment Info
antd 3.24.1
React 16.9.0
System windows10
Browser Chrome 77.0.3865.120

i run 'yarn upgrade' in 2019.10.16, and i got the error, in the pervious version, every thing is ok
2019.10.16号执行yarn upgrade后,出现该问题,此前版本无该问题

@yoyo837
Copy link
Contributor

yoyo837 commented Oct 17, 2019

https://github.com/react-component/trigger/blob/master/src/index.tsx#L518

mountNode.appendChild(popupContainer);

TypeError: Cannot read property 'appendChild' of null.

@afc163
Copy link
Member

afc163 commented Oct 18, 2019

@zombieJ 应该和 #19205 有关系,我看不动了,你来看一下。

@afc163 afc163 assigned zombieJ and unassigned chenshuai2144 Oct 18, 2019
@afc163 afc163 added the 🐛 Bug Ant Design Team had proved that this is a bug. label Oct 18, 2019
@zombieJ
Copy link
Member

zombieJ commented Oct 18, 2019

https://github.com/ant-design/ant-design/pull/18726/files#diff-9e758b938531cabc07bf62e7314948fdR509

这里把 findDOMNode 换成了 ref.tableNode 而 tableNode 是空导致的。我看看怎么修。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 Bug Ant Design Team had proved that this is a bug.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants