Skip to content

fix(ui): constrain tooltip width and remove question mark cursor#3752

Merged
waleedlatif1 merged 1 commit intostagingfrom
waleedlatif1/fix-tooltip-sizing
Mar 25, 2026
Merged

fix(ui): constrain tooltip width and remove question mark cursor#3752
waleedlatif1 merged 1 commit intostagingfrom
waleedlatif1/fix-tooltip-sizing

Conversation

@waleedlatif1
Copy link
Collaborator

Summary

  • Add max-w-[260px] to Tooltip.Content so tooltips with video previews stay a consistent size instead of blowing out
  • Replace cursor-help with cursor-default on info icons in settings to remove the browser question mark cursor

Type of Change

  • Bug fix

Testing

Tested manually

Checklist

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Tests added/updated and passing
  • No new warnings introduced
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

- Add max-w-[260px] to Tooltip.Content so video previews don't blow out the tooltip size
- Replace cursor-help with cursor-default on info icons in settings
@vercel
Copy link

vercel bot commented Mar 25, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
docs Skipped Skipped Mar 25, 2026 2:01am

Request Review

@cursor
Copy link

cursor bot commented Mar 25, 2026

PR Summary

Low Risk
Low risk styling-only changes that may slightly affect tooltip layout and cursor affordances but do not touch business logic or data flows.

Overview
Improves tooltip UX by constraining Tooltip.Content to max-w-[260px] so preview/tooltips don’t expand excessively.

Updates the settings info icons in general.tsx to use cursor-default instead of cursor-help, removing the browser help/question-mark cursor while retaining tooltip behavior.

Written by Cursor Bugbot for commit feac38f. Configure here.

@waleedlatif1 waleedlatif1 merged commit be64746 into staging Mar 25, 2026
7 checks passed
@waleedlatif1 waleedlatif1 deleted the waleedlatif1/fix-tooltip-sizing branch March 25, 2026 02:01
@greptile-apps
Copy link
Contributor

greptile-apps bot commented Mar 25, 2026

Greptile Summary

This is a small, focused UI polish PR that fixes two tooltip-related issues. A max-w-[260px] constraint is added to the shared Tooltip.Content base class to prevent video-preview tooltips from expanding uncontrollably, and cursor-help is replaced with cursor-default on the two Info icons in the General settings panel to remove the browser's question-mark pointer.

Key changes:

  • tooltip.tsx: max-w-[260px] added to the base cn(...) class; because className is merged last, per-call overrides still work fine.
  • general.tsx: Two Info icon triggers updated from cursor-help to cursor-default; no functional impact, purely cosmetic.

Both changes are minimal, well-scoped, and consistent with the existing code style.

Confidence Score: 5/5

  • This PR is safe to merge — both changes are minimal, isolated UI tweaks with no logic or data risk.
  • Both changes are single-line style adjustments. The max-w-[260px] addition is non-breaking because className overrides still compose correctly, and the Preview component's w-full / negative-margin layout is compatible with the new constraint. The cursor-default switch is purely cosmetic. No tests are needed for these visual-only changes.
  • No files require special attention.

Important Files Changed

Filename Overview
apps/sim/components/emcn/components/tooltip/tooltip.tsx Adds max-w-[260px] to the base Content class; callers can still override via className. No regressions — the Preview component already uses w-full and negative margins to fill the container, so the new constraint is compatible with all existing usages.
apps/sim/app/workspace/[workspaceId]/settings/components/general/general.tsx Changes cursor-helpcursor-default on two Info icon triggers in the General settings panel, removing the browser question-mark cursor as intended.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[Info icon hover] --> B[Tooltip.Root triggers]
    B --> C[Tooltip.Content rendered in Portal]
    C --> D{Has Preview?}
    D -- Yes --> E[Render text + Tooltip.Preview\n width capped at max-w-260px]
    D -- No --> F[Render text only\n width capped at max-w-260px]
    E --> G[Video/Image fills full\ntoolip width via w-full\nand neg. margins -mx-8px]
Loading

Reviews (1): Last reviewed commit: "fix(ui): constrain tooltip width and rem..." | Re-trigger Greptile

Sg312 pushed a commit that referenced this pull request Mar 25, 2026
- Add max-w-[260px] to Tooltip.Content so video previews don't blow out the tooltip size
- Replace cursor-help with cursor-default on info icons in settings
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