Browse Source

ref(useOverlay): Use `HTMLElement` type for trigger element (#50353)

The current type, `HTMLButtonElement`, is more restrictive than it needs
to be. Various elements can be overlay triggers, e.g. an anchor `<a />`
could trigger a tooltip overlay.
Vu Luong 1 year ago
parent
commit
5790fadeeb
1 changed files with 1 additions and 1 deletions
  1. 1 1
      static/app/utils/useOverlay.tsx

+ 1 - 1
static/app/utils/useOverlay.tsx

@@ -109,7 +109,7 @@ function useOverlay({
   disableTrigger,
 }: UseOverlayProps = {}) {
   // Callback refs for react-popper
-  const [triggerElement, setTriggerElement] = useState<HTMLButtonElement | null>(null);
+  const [triggerElement, setTriggerElement] = useState<HTMLElement | null>(null);
   const [overlayElement, setOverlayElement] = useState<HTMLDivElement | null>(null);
   const [arrowElement, setArrowElement] = useState<HTMLDivElement | null>(null);