Skip to content

Fix transparent tooltip background on max projects reached tooltip#471

Merged
AnthonyRonning merged 1 commit intomasterfrom
devin/1776189117-fix-tooltip-transparent-bg
Apr 14, 2026
Merged

Fix transparent tooltip background on max projects reached tooltip#471
AnthonyRonning merged 1 commit intomasterfrom
devin/1776189117-fix-tooltip-transparent-bg

Conversation

@devin-ai-integration
Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration Bot commented Apr 14, 2026

Summary

Fixes the tooltip for the "Maximum of 10 projects reached" message appearing transparent on desktop, making it hard to read against the project list behind it.

Two changes:

  1. Global tooltip z-index fix (tooltip.tsx): Changed z-1000 to z-[1000]. The bare z-1000 is not a valid Tailwind class (arbitrary values require bracket syntax), so it was silently ignored — meaning all tooltips had no explicit z-index.

  2. Opaque background for max-projects tooltip (ChatHistoryList.tsx): Added bg-muted class override to the tooltip content. The default bg-popover resolves to the same color as --background in dark mode, and the sidebar uses backdrop-blur-lg, which together caused the see-through appearance. bg-muted (0 0% 15% in dark mode) provides a visually distinct, opaque surface.

Review & Testing Checklist for Human

  • Verify the tooltip visually on desktop (dark mode): Hover over the disabled "New project" button when at 10 projects and confirm the tooltip has an opaque, readable background
  • Check that bg-muted looks right in both light and dark mode: The color choice may not match the intended design — compare against other tooltips/popovers in the app
  • Spot-check other tooltips in the app: The z-[1000] fix is global — verify no unintended z-index stacking issues appeared elsewhere (e.g., tooltips overlapping modals or dropdowns that they shouldn't)

Notes

  • The root cause (sidebar backdrop-blur-lg + bg-popover matching --background) could affect other tooltips rendered within the sidebar. If more tooltips are added to the sidebar in the future, they may need a similar bg-muted override — or the global bg-popover variable could be changed to a distinct color.

Link to Devin session: https://app.devin.ai/sessions/7da7b04fef0a43beaf1e35b14b79db8d


Open with Devin

- Fix z-index class in TooltipContent from invalid z-1000 to z-[1000]
- Add bg-muted to the max projects tooltip for an opaque, visually distinct background

Co-Authored-By: tony@opensecret.cloud <TonyGiorgio@protonmail.com>
@devin-ai-integration
Copy link
Copy Markdown
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@cloudflare-workers-and-pages
Copy link
Copy Markdown

Deploying maple with  Cloudflare Pages  Cloudflare Pages

Latest commit: 6591e7f
Status: ✅  Deploy successful!
Preview URL: https://d9ddda4c.maple-ca8.pages.dev
Branch Preview URL: https://devin-1776189117-fix-tooltip.maple-ca8.pages.dev

View logs

Copy link
Copy Markdown
Contributor Author

@devin-ai-integration devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 1 additional finding.

Open in Devin Review

@AnthonyRonning AnthonyRonning merged commit f5e6edd into master Apr 14, 2026
12 checks passed
@AnthonyRonning AnthonyRonning deleted the devin/1776189117-fix-tooltip-transparent-bg branch April 14, 2026 18:37
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.

1 participant