Merge pull request cuke-ui#5 from cc616/master

fix: tooltip position left
mallex · Nov 29, 2018 · baae6d9 · baae6d9
2 parents 2d88a23 + c8adb29
commit baae6d9
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/components/tooltip/index.js b/components/tooltip/index.js
@@ -44,7 +44,7 @@ export default class Tooltip extends PureComponent {
   static propTypes = {
     prefixCls: PropTypes.string.isRequired,
     onVisibleChange: PropTypes.func,
-    title: PropTypes.oneOfType([PropTypes.string, PropTypes.number]),
+    title: PropTypes.node,
     position: PropTypes.oneOf(["top", "right", "left", "bottom"]),
     theme: PropTypes.oneOf(themes)
   };
@@ -92,7 +92,7 @@ export default class Tooltip extends PureComponent {
       },
       left: {
         top: top + scrollY + height / 2 - wrapperHeight / 2,
-        left: left + scrollX - width
+        left: left + scrollX - wrapperWidth
       },
       right: {
         top: top + scrollY + height / 2 - wrapperHeight / 2,

diff --git a/components/tooltip/styles.less b/components/tooltip/styles.less
@@ -112,7 +112,7 @@
       }
     }
     &.@{prefixCls}-position-left {
-      margin-left: -8px;
+      margin-left: -@tooltip-offset;
       &::before, &::after {
         transform: translateY(-50%);
         top: 50%;