feat(tooltip): ensure we only render the tooltip when the charts are …

…interactive
plouc · Nov 15, 2020 · 6623000 · 6623000
1 parent 8addc8d
commit 6623000
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/packages/core/src/components/Container.js b/packages/core/src/components/Container.js
@@ -21,6 +21,7 @@ export const Container = ({
     children,
     theme,
     renderWrapper = true,
+    isInteractive = true,
     animate,
     motionStiffness,
     motionDamping,
@@ -43,7 +44,7 @@ export const Container = ({
                         wrapper={<div style={containerStyle} ref={container} />}
                     >
                         {children}
-                        <Tooltip />
+                        {isInteractive && <Tooltip />}
                     </ConditionalWrapper>
                 </TooltipProvider>
             </MotionConfigProvider>

diff --git a/packages/core/src/hocs/withContainer.js b/packages/core/src/hocs/withContainer.js
@@ -28,6 +28,7 @@ export const withContainer = WrappedComponent => {
                 <Container
                     theme={theme}
                     renderWrapper={renderWrapper}
+                    isInteractive={childProps.isInteractive}
                     animate={animate}
                     motionStiffness={motionStiffness}
                     motionDamping={motionDamping}