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

[Bug Report] el-table set max-height only doesn't work after release-2.8.0 #15281

Closed
chenweitech opened this issue Apr 26, 2019 · 16 comments
Closed

Comments

@chenweitech
Copy link

chenweitech commented Apr 26, 2019

Element UI version

2.8.0

OS/Browsers version

All

Vue version

2.6.10

Reproduction Link

#14660

Steps to reproduce

PR[#14660] caused anthor bug - el-table set max-height only doesn't work

What is Expected?

set max-height can work normally

What is actually happening?

tables-body height always be 60px

@GitHanDong
Copy link

我也遇到了同样的问题。
现在还是将版本锁定在2.8.0之前了。
去掉【max-height】会恢复正常。

@ziyoung
Copy link
Contributor

ziyoung commented Apr 26, 2019

这是我测试的 demo。没看出来哪里出问题了。https://jsfiddle.net/djmtapyh/

@fudax
Copy link

fudax commented Apr 26, 2019

同样的问题,刚想来提issue的就看到首页这个
问题版本:2.8.2
<el-table :data="filterResult" :max-height="baseHeight - 36" size="small" …… </el-table>

@fudax
Copy link

fudax commented Apr 26, 2019

<div class="el-table__body-wrapper" ref="bodyWrapper" :class="[layout.scrollX ?is-scrolling-${scrollPosition} : 'is-scrolling-none']" :style="[bodyHeight]">

@neostfox
Copy link

    "div",
    {
      ref: "bodyWrapper",
      staticClass: "el-table__body-wrapper",
      class: [
        _vm.layout.scrollX
          ? "is-scrolling-" + _vm.scrollPosition
          : "is-scrolling-none"
      ],
      style: [_vm.bodyHeight]
    }

I see style: [_vm.bodyHeight] in Webpack source

@chenweitech
Copy link
Author

@ziyoung 直接初始化table组件是正常的,但是如果是异步获取的数据则内容不会撑开父容器,这是我的例子 https://jsfiddle.net/atugsewp/2/

@ziyoung
Copy link
Contributor

ziyoung commented Apr 26, 2019

@ziyoung 直接初始化table组件是正常的,但是如果是异步获取的数据则内容不会撑开父容器,这是我的例子 https://jsfiddle.net/atugsewp/2/

是 bug。

@evansun1995
Copy link

也遇到了同样的问题,已经回退锁死版本了,希望尽快解决

@xiongchengqing
Copy link

什么时候发布新版本解决这个问题呀

@s12113122
Copy link

同样的问题+1 QQQ

@Air1996
Copy link

Air1996 commented May 5, 2019

还以为自己代码出问题了 =-=

@fangwenzheng88
Copy link

我也是,2.8.0版本表格优化还是很不错的,可是这个bug直接退到2.7.2了。

@ziyoung ziyoung modified the milestones: 2.8.3, 2.9.0 May 8, 2019
@shinchanZ
Copy link

我也是,2.8.0版本表格优化还是很不错的,可是这个bug直接退到2.7.2了。

我回退到2.7.2还是有这个问题,不知道为啥

@caoke
Copy link

caoke commented May 22, 2019

我也是,2.8.0版本表格优化还是很不错的,可是这个bug直接退到2.7.2了。

我回退到2.7.2还是有这个问题,不知道为啥
回退到2.7.2就没有这个问题了

@shinchanZ
Copy link

我也是,2.8.0版本表格优化还是很不错的,可是这个错误直接退到2.7.2了。

我回退到2.7.2还是有这个问题,知道不为啥
回退到2.7.2就没有这个问题了

我也不知道为啥,之前好好的,把node_modules删了重装就这样了,我的解决办法是把他的样式覆盖
.el-table__body-wrapper.is-scrolling-none{ max-height:440px; overflow-y: auto; }

@luckyCao
Copy link
Contributor

fixed in #15709

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.