Merge pull request #159 from element-hq/florianduros/fix-tooltip-appe…

…arance Fix tooltip display
element-hq · Apr 18, 2024 · d3ade46 · d3ade46
2 parents 4c194b6 + 686b695
commit d3ade46
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/src/components/Form/Controls/Password/__snapshots__/Password.test.tsx.snap b/src/components/Form/Controls/Password/__snapshots__/Password.test.tsx.snap
@@ -49,9 +49,10 @@ exports[`PasswordControl > switches the input type > invisible 2`] = `
     />
     <button
       aria-controls=":r0:"
+      aria-describedby=":r3:"
       aria-label="Show"
       class="_action_1689d3"
-      data-state="closed"
+      data-state="open"
       type="button"
     >
       <svg
@@ -85,9 +86,10 @@ exports[`PasswordControl > switches the input type > visible 1`] = `
     />
     <button
       aria-controls=":r0:"
+      aria-describedby=":r3:"
       aria-label="Hide"
       class="_action_1689d3"
-      data-state="closed"
+      data-state="open"
       type="button"
     >
       <svg

diff --git a/src/components/Tooltip/useTooltip.ts b/src/components/Tooltip/useTooltip.ts
@@ -122,6 +122,7 @@ export function useTooltip({
   });
   const focus = useFocus(context, {
     enabled: controlledOpen === undefined,
+    visibleOnly: false,
   });
   const dismiss = useDismiss(context);
   const role = useRole(context, { role: "tooltip" });