@lumx/react 3.11.1 → 3.11.2

package/package.json

@@ -6,8 +6,8 @@
         "url": "https://github.com/lumapps/design-system/issues"
     },
     "dependencies": {
-        "@lumx/core": "^3.11.1",
-        "@lumx/icons": "^3.11.1",
+        "@lumx/core": "^3.11.2",
+        "@lumx/icons": "^3.11.2",
         "@popperjs/core": "^2.5.4",
         "body-scroll-lock": "^3.1.5",
         "classnames": "^2.3.2",
@@ -110,5 +110,5 @@
         "build:storybook": "storybook build"
     },
     "sideEffects": false,
-    "version": "3.11.1"
+    "version": "3.11.2"
 }

package/src/components/popover/Popover.tsx

@@ -6,7 +6,7 @@ import classNames from 'classnames';
 import { useCallbackOnEscape } from '@lumx/react/hooks/useCallbackOnEscape';
 import { useFocus } from '@lumx/react/hooks/useFocus';
 import { ClickAwayProvider } from '@lumx/react/utils/ClickAwayProvider';
-import { DOCUMENT, VISUALLY_HIDDEN } from '@lumx/react/constants';
+import { DOCUMENT } from '@lumx/react/constants';
 import { Comp, GenericProps, HasTheme } from '@lumx/react/utils/type';
 import { getRootClassName, handleBasicClasses } from '@lumx/react/utils/className';
 import { useMergeRefs } from '@lumx/react/utils/react/mergeRefs';
@@ -166,9 +166,8 @@ const _InnerPopover = forwardRef<PopoverProps, HTMLDivElement>((props, ref) => {
                           theme,
                           elevation: Math.min(elevation || 0, 5),
                           position,
+                          isInitializing: !styles.popover?.transform,
                       }),
-                      // Do not show the popover while it's not properly placed
-                      !styles.popover?.transform ? VISUALLY_HIDDEN : undefined,
                   )}
                   style={styles.popover}
                   {...attributes.popper}

package/src/components/tooltip/Tooltip.tsx

@@ -119,12 +119,13 @@ export const Tooltip = forwardRef<TooltipProps, HTMLDivElement>((props, ref) =>
 
     // Update on open
     React.useEffect(() => {
-        if (isOpen) update?.();
-    }, [isOpen, update]);
+        if (isOpen || popperElement) update?.();
+    }, [isOpen, update, popperElement]);
 
     const labelLines = label ? label.split('\n') : [];
 
     const tooltipRef = useMergeRefs(ref, setPopperElement, onPopperMount);
+
     return (
         <>
             <TooltipContextProvider>{wrappedChildren}</TooltipContextProvider>
@@ -140,6 +141,7 @@ export const Tooltip = forwardRef<TooltipProps, HTMLDivElement>((props, ref) =>
                             handleBasicClasses({
                                 prefix: CLASSNAME,
                                 position,
+                                isInitializing: !styles.popper?.transform,
                             }),
                             isHidden && VISUALLY_HIDDEN,
                         )}