feat(tooltip): remove prop types

plouc · Nov 15, 2020 · b187c61 · b187c61
1 parent 3413142
commit b187c61
Show file tree

Hide file tree

Showing 6 changed files with 13 additions and 33 deletions.
diff --git a/packages/tooltip/package.json b/packages/tooltip/package.json
@@ -26,8 +26,7 @@
     "@nivo/core": "*"
   },
   "peerDependencies": {
-    "@nivo/core": "0.63.1",
-    "prop-types": ">= 15.5.10 < 16.0.0"
+    "@nivo/core": "0.63.1"
   },
   "publishConfig": {
     "access": "public"

diff --git a/packages/tooltip/src/Crosshair.js b/packages/tooltip/src/Crosshair.js
@@ -1,9 +1,7 @@
 import React, { memo } from 'react'
-import PropTypes from 'prop-types'
-import { crosshairPropTypes } from './props'
-import CrosshairLine from './CrosshairLine'
+import { CrosshairLine } from './CrosshairLine'
 
-const Crosshair = memo(({ width, height, type, x, y }) => {
+export const Crosshair = memo(({ width, height, type, x, y }) => {
     let xLine
     let yLine
     if (type === 'cross') {
@@ -43,6 +41,7 @@ const Crosshair = memo(({ width, height, type, x, y }) => {
     )
 })
 
+/*
 Crosshair.displayName = 'Crosshair'
 Crosshair.propTypes = {
     width: PropTypes.number.isRequired,
@@ -56,3 +55,4 @@ Crosshair.defaultProps = {
 }
 
 export default Crosshair
+*/
diff --git a/packages/tooltip/src/CrosshairLine.js b/packages/tooltip/src/CrosshairLine.js
@@ -1,9 +1,8 @@
 import React, { memo, useMemo } from 'react'
-import PropTypes from 'prop-types'
 import { useSpring, animated } from 'react-spring'
 import { useTheme, useMotionConfig } from '@nivo/core'
 
-const CrosshairLine = memo(({ x0, x1, y0, y1 }) => {
+export const CrosshairLine = memo(({ x0, x1, y0, y1 }) => {
     const theme = useTheme()
     const { animate, config: springConfig } = useMotionConfig()
     const style = useMemo(
@@ -26,6 +25,7 @@ const CrosshairLine = memo(({ x0, x1, y0, y1 }) => {
     return <animated.line {...animatedProps} fill="none" style={style} />
 })
 
+/*
 CrosshairLine.displayName = 'CrosshairLine'
 CrosshairLine.propTypes = {
     x0: PropTypes.number.isRequired,
@@ -35,3 +35,4 @@ CrosshairLine.propTypes = {
 }
 
 export default CrosshairLine
+*/
diff --git a/packages/tooltip/src/TableTooltip.js b/packages/tooltip/src/TableTooltip.js
@@ -1,13 +1,12 @@
 import React, { memo } from 'react'
-import PropTypes from 'prop-types'
 import { useTheme } from '@nivo/core'
 
 const tableStyle = {
     width: '100%',
     borderCollapse: 'collapse',
 }
 
-const TableTooltip = memo(({ title, rows, renderContent }) => {
+export const TableTooltip = memo(({ title, rows, renderContent }) => {
     const theme = useTheme()
 
     if (!rows.length) return null
@@ -38,7 +37,7 @@ const TableTooltip = memo(({ title, rows, renderContent }) => {
 
     return <div style={theme.tooltip.container}>{content}</div>
 })
-
+/*
 TableTooltip.propTypes = {
     title: PropTypes.node,
     rows: PropTypes.arrayOf(PropTypes.arrayOf(PropTypes.node)).isRequired,
@@ -48,3 +47,4 @@ TableTooltip.propTypes = {
 TableTooltip.displayName = 'TableTooltip'
 
 export default TableTooltip
+*/
diff --git a/packages/tooltip/src/index.ts b/packages/tooltip/src/index.ts
@@ -1,11 +1,10 @@
 export * from './TooltipWrapper'
 export * from './BasicTooltip'
 export * from './Chip'
-export { default as TableTooltip } from './TableTooltip'
-export { default as Crosshair } from './Crosshair'
+export * from './TableTooltip'
+export * from './Crosshair'
 export * from './Tooltip'
 export * from './TooltipProvider'
 export * from './context'
 export * from './hooks'
-export * from './props'
 export * from './types'
diff --git a/packages/tooltip/src/props.js b/packages/tooltip/src/props.js