fix(Table): style confusion caused by vertical scrollbar, close #473

alibaba-fusion · Mar 22, 2019 · 9149879 · 9149879
1 parent 70c1bc4
commit 9149879
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/src/table/fixed.jsx b/src/table/fixed.jsx
@@ -70,14 +70,16 @@ export default function fixed(BaseComponent) {
         }
 
         adjustFixedHeaderSize() {
-            const { hasHeader, fixedHeader, maxBodyHeight } = this.props;
+            const { hasHeader, fixedHeader, maxBodyHeight, rtl } = this.props;
+            const paddingName = rtl ? 'paddingLeft' : 'paddingRight';
+
             if (hasHeader && fixedHeader && !this.props.lockType) {
                 if (this.bodyNode.scrollHeight <= maxBodyHeight) {
-                    dom.setStyle(this.headerNode, 'paddingRight', 0);
+                    dom.setStyle(this.headerNode, paddingName, 0);
                 } else {
                     dom.setStyle(
                         this.headerNode,
-                        'paddingRight',
+                        paddingName,
                         dom.scrollbar().width
                     );
                 }

diff --git a/src/table/lock.jsx b/src/table/lock.jsx
@@ -441,7 +441,7 @@ export default function lock(BaseComponent) {
                     dom.setStyle(
                         lockRightBodyWrapper,
                         rtl ? 'left' : 'right',
-                        width
+                        `${width}px`
                     );
             }
         }