Skip to content

Make dropdown menus opaque so background no longer shows through#248

Merged
d3mocide merged 1 commit into
mainfrom
claude/ecstatic-galileo-lr0ghj
Jun 23, 2026
Merged

Make dropdown menus opaque so background no longer shows through#248
d3mocide merged 1 commit into
mainfrom
claude/ecstatic-galileo-lr0ghj

Conversation

@d3mocide

Copy link
Copy Markdown
Owner

Problem

The favorites selector, food-search results (budget check), and hydration goal/glass-size dropdowns let the UI behind them bleed through — you could read card text and other content under the open menu.

Root cause: those menus used background: var(--glass-2) (only 6% opaque white in dark mode) and leaned entirely on backdrop-filter: blur(16px) to obscure what was behind. Blur alone can't mask bright content, so it showed through. The favorites-page sort dropdown looked fine only because it opens over the empty dark page area.

Fix

  • Add a theme-aware --glass-popover token — near-opaque (rgba(16,24,44,0.94) dark / rgba(255,255,255,0.96) light) — for floating menus.
  • Point the favorites selector, both food-search result menus, and both hydration dropdowns at it.

The frosted blur, glass border, and drop shadow are unchanged, so the glass feel is preserved while the background is properly masked. The sort dropdown is intentionally left as-is (it was already perfect).

Verification

  • pnpm type-check
  • pnpm lint — 0 errors (2 pre-existing warnings, untouched code)
  • pnpm test — 114/114 passing ✅

🤖 Generated with Claude Code


Generated by Claude Code

The favorites selector, food-search results, and hydration goal/glass
dropdowns used --glass-2 (6% opaque white) and relied solely on
backdrop-filter to hide what was behind them. Over bright card content
and text, the underlying UI bled through.

Add a --glass-popover token (near-opaque, theme-aware) and point the
floating menus at it. The frosted blur, glass border, and shadow are
unchanged, so the glass feel is preserved while the background is masked.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01VDWtPAnqG2XzZB3rSnGb9J
@d3mocide d3mocide marked this pull request as ready for review June 23, 2026 00:18
@d3mocide d3mocide merged commit ea25eb5 into main Jun 23, 2026
4 checks passed
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.

2 participants