make TooltipWithBounds API match that of Tooltip

airbnb · May 19, 2020 · 70be7a4 · 70be7a4
1 parent 806fbf2
commit 70be7a4
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/packages/vx-tooltip/Readme.md b/packages/vx-tooltip/Readme.md
@@ -86,7 +86,7 @@ component (i.e., ...restProps):
 | offsetRight | number | 10      | Vertical offset of the tooltip from the passed `top` value, functions as a vertical padding.                                                                                     |
 | style       | object | --      | Sets / overrides any styles on the tooltip container (including top and left)                                                                                                    |
 | children    | node   | --      | Sets the children of the tooltip, i.e., the actual content                                                                                                                       |
-| unstyled  | bool             | true    | Whether the tooltip should render with default styles or not                       |
+| unstyled  | bool             | true    | Whether the tooltip should use styles from the style prop or not              |
 
 Note that this component is positioned using a `transform`, so overriding `left` and `top` via
 styles may have no effect.

diff --git a/packages/vx-tooltip/src/tooltips/TooltipWithBounds.tsx b/packages/vx-tooltip/src/tooltips/TooltipWithBounds.tsx
@@ -33,7 +33,7 @@ function TooltipWithBounds({
   parentRect,
   getRects,
   children,
-  style,
+  style = defaultStyles,
   unstyled,
   ...otherProps
 }: Props) {
@@ -60,8 +60,7 @@ function TooltipWithBounds({
       style={{
         top: 0,
         transform: `translate(${left}px, ${top}px)`,
-        ...style,
-        ...(!unstyled && defaultStyles),
+        ...(!unstyled && style),
       }}
       {...otherProps}
     >