-
Notifications
You must be signed in to change notification settings - Fork 52
feat: add heightAsTableHeight prop #75
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
base: master
Are you sure you want to change the base?
Conversation
为什么要将scrollY(传入的height)和tableHeight中的最大值作为table的高度数值使用 |
|
加个demo例子把 |
popover会有这么离谱的吗? 这么长的话是不是应该搞个详情按钮给用户看才对. |
我记得scrollY就是table区域的内容高度啊, popover这么离谱的逻辑感觉不应该写在插件里啊 |
这个只是写个demo可能多了点,正常没这个长但是如果只有一行数据的时候也会有问题,另外就是样式适配,比如我例子中的pagination需要置底,在columns数量多的事情横向滚动条位置也不好看,这种情况下【将scrollY(传入的height)和tableHeight中的最大值作为table的高度数值使用】在样式上会好看点。 |
是想数据少的时候, 能按具体scrollY的多少展示多少对吧? 比如只有一条数据, 但是scrollY是500, 表格要按500撑开? |
是的,我们的运用场景确实相对而言复杂了点,是需要有一个可控的table高度撑起的 |
我记得antd scrollY本来的行为就是自适应撑开的吧, 插件不合适去弄这个逻辑. 但是像复杂的例子那样扩展应该是可以的, 我再想一下 |
新增heightAsTableHeight参数(默认关闭),heightAsTableHeight: true时会将scrollY(传入的height)和tableHeight中的最大值作为table的高度数值使用