fix(Table): only show shadow when table need scroll, #201

ksc-fe · Jan 22, 2019 · 2fabae7 · 2fabae7
1 parent f52987c
commit 2fabae7
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/components/table/index.js b/components/table/index.js
@@ -40,6 +40,7 @@ export default class Table extends Intact {
 
             _padding: 0,
             _paddingBottom: 0,
+            _isShowLeftRightMiddle: false,
             _disabledAmount: 0,
             _isSticky: false,
             _leftWidth: 0,
@@ -409,11 +410,14 @@ export default class Table extends Intact {
                 }
 
                 // calculate the horizontal scroll bar
-                let paddingBottom = 0
+                let paddingBottom = 0;
+                let isShowLeftRightMiddle = false;
                 if (this.element.offsetWidth < tableWidth + this.get('_padding')) {
                     paddingBottom = this.get('_scrollBarWidth');
+                    isShowLeftRightMiddle = true;
                 }
                 data._paddingBottom = paddingBottom;
+                data._isShowLeftRightMiddle = isShowLeftRightMiddle;
 
                 this.set(data);
             }

diff --git a/components/table/index.vdt b/components/table/index.vdt
@@ -12,7 +12,7 @@ const {
     resizable, expandedKeys, type, _isSticky,
     style, rowClassName, children, sort, group, _scrollBarWidth,
     _scrollLeft, _scrollTop, _hoverIndex, _scrollPosition, 
-    _paddingBottom, _isStickyScrollbar, loading
+    _paddingBottom, _isStickyScrollbar, loading, _isShowLeftRightMiddle
 } = self.get();
 
 let colSpan = checkType === 'checkbox' || checkType === 'radio' ? 1 : 0;
@@ -344,7 +344,7 @@ const classNameObj = {
     [className]: className,
     [`k-${type}`]: type !== 'default',
     'k-sticky': _isSticky, 
-    [`k-scroll-${_scrollPosition}`]: hasFixedLeft || hasFixedRight,
+    [`k-scroll-${_scrollPosition}`]: _isShowLeftRightMiddle,
     'k-sticky-scrollbar': _isStickyScrollbar,
 };