github
/
element
mirror of https://github.com/ElemeFE/element
1
0
Fork 0
Code Issues Releases Wiki Activity

Fix Table Bug: body-wrapper should not set style.height when table not set height. (#839)

pull/842/head
FuryBean 2016-11-05 07:31:18 +08:00 committed by cinwell.li
parent f013d6eb0b
commit 07cd230674
2 changed files with 6 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
packages/table/src/table-layout.js
View File

@ -9,6 +9,7 @@ class TableLayout {
this.columns = null;
this.fit = true;
this.height = null;
this.scrollX = false;
this.scrollY = false;
this.bodyWidth = null;
@ -53,6 +54,8 @@ class TableLayout {
height = Number(height);
}
this.height = height;
const el = this.table.$el;
if (!isNaN(height) && el) {
el.style.height = height + 'px';
@ -66,7 +69,8 @@ class TableLayout {
const { headerWrapper } = this.table.$refs;
if (!headerWrapper) return;
const headerHeight = this.headerHeight = headerWrapper.offsetHeight;
const bodyHeight = this.bodyHeight = height - headerHeight;
const bodyHeight = height - headerHeight;
if (this.height !== null && !isNaN(this.height)) this.bodyHeight = bodyHeight;
this.fixedBodyHeight = this.scrollX ? bodyHeight - this.gutterWidth : bodyHeight;
this.viewportHeight = this.scrollX ? height - this.gutterWidth : height;
}

1
test/unit/specs/table.spec.js
View File

@ -441,6 +441,7 @@ describe('Table', () => {
expect(toArray(vm.$el.querySelectorAll('.el-table__fixed-right th:not(.is-hidden)'))
.map(node => node.textContent))
.to.eql(['test2']);
expect(vm.$el.querySelector('.el-table__body-wrapper').style.height).to.equal('');
destroyVM(vm);
done();
}, DELAY);