Skip to content

Add z-index theme token and visibility to Popover placementStyle#716

Merged
adamkudrna merged 1 commit into
masterfrom
task/popover-z-index-token
Jul 1, 2026
Merged

Add z-index theme token and visibility to Popover placementStyle#716
adamkudrna merged 1 commit into
masterfrom
task/popover-z-index-token

Conversation

@adamkudrna

@adamkudrna adamkudrna commented Jun 29, 2026

Copy link
Copy Markdown
Member
  • Add --rui-Popover__z-index token (default: auto) so consumers can control the Popover's stacking context without needing attribute selectors or class injection.
  • Allow visibility through cleanPlacementStyle so callers can hide the Popover before its initial position is computed (flash guard).
  • Document both additions in README.

Replaces #670.

- Add `--rui-Popover__z-index` token (default: `auto`) so consumers can
  control the Popover's stacking context without needing attribute selectors
  or class injection.
- Allow `visibility` through `cleanPlacementStyle` so callers can hide the
  Popover before its initial position is computed (flash guard).
- Document both additions in README Theming table and placementStyle prop JSDoc.
@github-actions

github-actions Bot commented Jun 29, 2026

Copy link
Copy Markdown
Contributor
PR Preview Action v1.8.1
Preview removed because the pull request was closed.
2026-07-01 11:33 UTC

@adamkudrna adamkudrna moved this to 👀 In review in The Board Jun 29, 2026
@adamkudrna
adamkudrna merged commit 3cb7a8a into master Jul 1, 2026
16 checks passed
@adamkudrna
adamkudrna deleted the task/popover-z-index-token branch July 1, 2026 11:33
@github-project-automation github-project-automation Bot moved this from 👀 In review to ✅ Done in The Board Jul 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

3 participants