Merge pull request #935 from ReactTooltip/fix/wrapper-behavior

fix: wrapper element changing after first render

Author: danielbarion

.npmignore

@@ -21,7 +21,15 @@ coverage/
 .travis.yml
 .editorconfig
 
-// others
+// misc folders
 .vscode/
 .husky/
 .github/
+
+// misc files
+bundlesize.config.json
+
+// bundler - rollup
+rollup.config.dev.js
+rollup.config.prod.js
+rollup.config.types.js

src/components/Tooltip/Tooltip.tsx

@@ -19,7 +19,7 @@ const Tooltip = ({
   events = ['hover'],
   positionStrategy = 'absolute',
   middlewares,
-  wrapper: WrapperElement = 'div',
+  wrapper: WrapperElement,
   children = null,
   delayShow = 0,
   delayHide = 0,
@@ -38,7 +38,7 @@ const Tooltip = ({
   setIsOpen,
 }: ITooltip) => {
   const tooltipRef = useRef<HTMLElement>(null)
-  const tooltipArrowRef = useRef<HTMLDivElement>(null)
+  const tooltipArrowRef = useRef<HTMLElement>(null)
   const tooltipShowDelayTimerRef = useRef<NodeJS.Timeout | null>(null)
   const tooltipHideDelayTimerRef = useRef<NodeJS.Timeout | null>(null)
   const [inlineStyles, setInlineStyles] = useState({})

src/components/Tooltip/TooltipTypes.d.ts

@@ -43,7 +43,7 @@ export interface ITooltip {
   id?: string
   variant?: VariantType
   anchorId?: string
-  wrapper?: WrapperType
+  wrapper: WrapperType
   children?: ChildrenType
   events?: EventsType[]
   positionStrategy?: PositionStrategy

src/components/TooltipController/TooltipController.tsx

@@ -85,7 +85,7 @@ const TooltipController = ({
         setTooltipOffset(value === null ? offset : Number(value))
       },
       wrapper: (value) => {
-        setTooltipWrapper((value as WrapperType) ?? 'div')
+        setTooltipWrapper((value as WrapperType) ?? wrapper)
       },
       events: (value) => {
         const parsed = value?.split(' ') as EventsType[]