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

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants