Skip to content

ref(🥞): remove portal z-index references (dropdown, tooltip, hovercard)#115957

Draft
natemoo-re wants to merge 6 commits into
nm/zindex/wire-portalsfrom
nm/zindex/remove-portal-zindex
Draft

ref(🥞): remove portal z-index references (dropdown, tooltip, hovercard)#115957
natemoo-re wants to merge 6 commits into
nm/zindex/wire-portalsfrom
nm/zindex/remove-portal-zindex

Conversation

@natemoo-re
Copy link
Copy Markdown
Member

@natemoo-re natemoo-re commented May 20, 2026

Summary

  • Remove theme.zIndex.dropdown, tooltip, and hovercard references
  • These are now handled by Layer portal outlets (each Layer creates a portal container at the top of its stacking context)
  • Remove deprecated zIndex prop usage on PositionWrapper

🥞 Layer Primitive Series

Test plan

  • Hover tooltips in content area — verify they appear above content but behind drawer
  • Open drawer, hover tooltips inside — verify they appear above drawer content
  • Open dropdown menus — verify they appear correctly positioned
  • Search query builder comboboxes render above surrounding content
  • Chart tooltips still visible on hover
  • pnpm run typecheck

@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label May 20, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 20, 2026

📊 Type Coverage Diff

✅ No new type safety issues introduced. Coverage: 93.58%

@natemoo-re natemoo-re changed the base branch from master to nm/zindex/wire-portals May 20, 2026 22:04
@natemoo-re natemoo-re force-pushed the nm/zindex/wire-portals branch from 0e6e40d to 579605a Compare May 21, 2026 13:59
@natemoo-re natemoo-re force-pushed the nm/zindex/remove-portal-zindex branch from 1048894 to dd19fd8 Compare May 21, 2026 14:00
@natemoo-re natemoo-re force-pushed the nm/zindex/remove-portal-zindex branch from dd19fd8 to 4056ee3 Compare May 21, 2026 14:14
@natemoo-re natemoo-re force-pushed the nm/zindex/remove-portal-zindex branch from 4056ee3 to a4ee0df Compare May 21, 2026 17:31
@natemoo-re natemoo-re force-pushed the nm/zindex/remove-portal-zindex branch from a4ee0df to 3057c93 Compare May 21, 2026 17:35
@natemoo-re natemoo-re force-pushed the nm/zindex/remove-portal-zindex branch from 3057c93 to dfc37fa Compare May 21, 2026 17:42
@natemoo-re natemoo-re force-pushed the nm/zindex/wire-portals branch from 255dc7c to 97da138 Compare May 26, 2026 17:16
@natemoo-re natemoo-re force-pushed the nm/zindex/remove-portal-zindex branch from dfc37fa to 75bb04e Compare May 26, 2026 17:20
@natemoo-re natemoo-re force-pushed the nm/zindex/wire-portals branch from 97da138 to f0adcb8 Compare May 26, 2026 21:18
@natemoo-re natemoo-re force-pushed the nm/zindex/remove-portal-zindex branch from 1344bfb to e19b859 Compare May 26, 2026 21:22
@natemoo-re natemoo-re force-pushed the nm/zindex/wire-portals branch from f0adcb8 to 1d3b325 Compare May 26, 2026 21:59
@natemoo-re natemoo-re force-pushed the nm/zindex/remove-portal-zindex branch from e19b859 to 0a3164f Compare May 26, 2026 21:59
@natemoo-re natemoo-re force-pushed the nm/zindex/wire-portals branch from 1d3b325 to 471b49d Compare May 26, 2026 22:26
@natemoo-re natemoo-re force-pushed the nm/zindex/remove-portal-zindex branch from 0a3164f to 5a6115e Compare May 26, 2026 22:26
natemoo-re and others added 6 commits May 28, 2026 12:34
Dropdown, tooltip, and hovercard z-index values are now redundant —
Layer portal outlets handle overlay stacking within each stacking
context.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
These files were deleted on master (refactored). The rebase conflict
resolution accidentally re-created them from stale versions.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant