Skip to content

fix(web): increase tooltip z-index to overlay popovers and menus - #5326

Merged
maria-rcks merged 1 commit into
pingdotgg:mainfrom
naMqe-h:fix/tooltip-zindex
Aug 4, 2026
Merged

fix(web): increase tooltip z-index to overlay popovers and menus#5326
maria-rcks merged 1 commit into
pingdotgg:mainfrom
naMqe-h:fix/tooltip-zindex

Conversation

@naMqe-h

@naMqe-h naMqe-h commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

What Changed

Increased the TooltipPositioner z-index from z-50 to z-70 in apps/web/src/components/ui/tooltip.tsx.

Why

Tooltips rendered inside popovers or dropdown menus (such as the "Add to favorites" star button inside model list items) were being clipped and rendered behind the popover container.

This occurred because PopoverPositioner (apps/web/src/components/ui/popover.tsx) and MenuPositioner (apps/web/src/components/ui/menu.tsx) use z-[60], whereas TooltipPositioner was set to z-50. Since both elements render via React Portals as siblings at the document root, the lower z-index caused tooltips to render underneath popover surfaces.

Updating TooltipPositioner to z-70 ensures tooltips always correctly overlay popovers and menus.

UI Changes

Before:
before
After:
after

Checklist

  • This PR is small and focused
  • I explained what changed and why
  • I included before/after screenshots for any UI changes
  • I included a video for animation/interaction changes

Note

Low Risk
Single CSS stacking-order tweak in the shared tooltip component; no auth, data, or behavioral logic changes.

Overview
Raises TooltipPositioner stacking from z-50 to z-70 so portal-rendered tooltips sit above popovers and menus (which use z-[60]), fixing clipping when tooltips appear inside those surfaces (e.g. favorite actions in model lists).

Reviewed by Cursor Bugbot for commit c332573. Bugbot is set up for automated code reviews on this repo. Configure here.

Note

Increase tooltip z-index to overlay popovers and menus

Updates the TooltipPopup positioner in tooltip.tsx from z-50 to z-70 so tooltips render above popovers and menus.

Macroscope summarized c332573.

@github-actions github-actions Bot added vouch:unvouched PR author is not yet trusted in the VOUCHED list. size:XS 0-9 changed lines (additions + deletions). labels Aug 4, 2026
@coderabbitai

coderabbitai Bot commented Aug 4, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: f0dcba83-d625-4b25-8f6a-e7a7bb3fbedf

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@macroscopeapp

macroscopeapp Bot commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

Approvability

Verdict: Approved c332573

Single-line CSS z-index change (50 → 70) to fix tooltip layering over popovers/menus. Purely visual adjustment with no runtime logic impact.

You can customize Macroscope's approvability policy. Learn more.

@maria-rcks
maria-rcks merged commit 8a16cad into pingdotgg:main Aug 4, 2026
17 checks passed
github-actions Bot added a commit to omarcresp/t3code-flake that referenced this pull request Aug 4, 2026
## What's Changed
* fix(web): increase tooltip z-index to overlay popovers and menus by @naMqe-h in pingdotgg/t3code#5326
* fix(server): use a Cursor todo's title when its content is blank by @arhxam in pingdotgg/t3code#5073
* fix(ssh): isolate managed tunnel processes by @nateEc in pingdotgg/t3code#4347
* fix(server): scrub AppImage XDG_DATA_DIRS and GSETTINGS_SCHEMA_DIR from terminals by @arhxam in pingdotgg/t3code#5075

## New Contributors
* @naMqe-h made their first contribution in pingdotgg/t3code#5326

**Full Changelog**: pingdotgg/t3code@v0.0.32-nightly.20260804.997...v0.0.32-nightly.20260804.998

Upstream release: https://github.com/pingdotgg/t3code/releases/tag/v0.0.32-nightly.20260804.998
cursor Bot pushed a commit to aaditagrawal/t3code that referenced this pull request Aug 6, 2026
cursor Bot pushed a commit to aaditagrawal/t3code that referenced this pull request Aug 6, 2026
cursor Bot pushed a commit to aaditagrawal/t3code that referenced this pull request Aug 6, 2026
cursor Bot pushed a commit to aaditagrawal/t3code that referenced this pull request Aug 6, 2026
cursor Bot pushed a commit to aaditagrawal/t3code that referenced this pull request Aug 6, 2026
vortechron pushed a commit to vortechron/void that referenced this pull request Aug 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XS 0-9 changed lines (additions + deletions). vouch:unvouched PR author is not yet trusted in the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants