Skip to content

fix(tooltip): improve SSR compatibility for FloatingPortal#722

Open
sentry[bot] wants to merge 1 commit intomainfrom
seer/fix/tooltip-ssr-portal
Open

fix(tooltip): improve SSR compatibility for FloatingPortal#722
sentry[bot] wants to merge 1 commit intomainfrom
seer/fix/tooltip-ssr-portal

Conversation

@sentry
Copy link
Contributor

@sentry sentry bot commented Feb 18, 2026

Fixes TANSTACK-COM-22S. The issue was that: Floating-UI's FloatingPortal accesses ownerDocument on null container during iOS Safari SPA navigation.

  • Introduce isMounted state and portalRoot ref to manage portal rendering.
  • Use useEffect to safely set portalRoot to document.body only in a browser environment, preventing SSR errors.
  • Conditionally render FloatingPortal and explicitly pass document.body as its root.

This fix was generated by Seer in Sentry, triggered automatically. 👁️ Run ID: 10361036

Not quite right? Click here to continue debugging with Seer.

@netlify
Copy link

netlify bot commented Feb 18, 2026

Deploy Preview for tanstack ready!

Name Link
🔨 Latest commit 892027d
🔍 Latest deploy log https://app.netlify.com/projects/tanstack/deploys/69960a90a799880008cf0e65
😎 Deploy Preview https://deploy-preview-722--tanstack.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 27 (🟢 up 2 from production)
Accessibility: 90 (no change from production)
Best Practices: 75 (🔴 down 8 from production)
SEO: 97 (no change from production)
PWA: 70 (no change from production)
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify project configuration.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0 participants

Comments