Skip to content

[2.x] fix(approval): apply opacity to whole unapproved post, not child elements#4453

Merged
imorland merged 1 commit into2.xfrom
im/fix-4212-unapproved-post-opacity
Mar 15, 2026
Merged

[2.x] fix(approval): apply opacity to whole unapproved post, not child elements#4453
imorland merged 1 commit into2.xfrom
im/fix-4212-unapproved-post-opacity

Conversation

@imorland
Copy link
Copy Markdown
Member

Summary

  • Applying opacity < 1 to .Post-header directly creates a new CSS stacking context, which confines the z-index of any absolutely-positioned descendants (e.g. the user-card dropdown on avatar hover) to that context — causing the dropdown to disappear when the user tries to navigate to it
  • Moving opacity: 0.5 up to .Post--unapproved itself removes the stacking context issue; the visual result is identical (whole post dimmed) and the per-child selector list is no longer needed

Fixes #4212

Test plan

  • Set a post to awaiting approval
  • Hover over the avatar — user card dropdown should appear and remain visible as the mouse moves into it
  • Visual: unapproved post still appears at 50% opacity
  • Event posts (renamed, hidden, etc.) with unapproved state still appear dimmed correctly

🤖 Generated with Claude Code

…ents

Applying opacity < 1 directly to .Post-header created a new CSS stacking
context, which caused the user-card dropdown (shown on avatar hover) to be
confined to that stacking context and disappear when the user tried to
navigate to it.

Moving opacity to the .Post--unapproved wrapper fixes the stacking context
issue and produces the same visual result — the whole post is dimmed at 50%.
The per-child selectors for .EventPost-icon/.EventPost-info are covered by
the parent rule and can be removed.

Fixes #4212
@imorland imorland changed the title fix(approval): apply opacity to whole unapproved post, not child elements [2.x] fix(approval): apply opacity to whole unapproved post, not child elements Mar 15, 2026
@imorland imorland added this to the 2.0.0-beta.8 milestone Mar 15, 2026
@imorland imorland marked this pull request as ready for review March 15, 2026 08:03
@imorland imorland requested a review from a team as a code owner March 15, 2026 08:03
@imorland imorland merged commit a5984ac into 2.x Mar 15, 2026
27 checks passed
@imorland imorland deleted the im/fix-4212-unapproved-post-opacity branch March 15, 2026 08:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Awaiting Approval Dropdown bug

1 participant