Skip to content

Redeign tooltip for different intents usage#1754

Open
Douglasymlai wants to merge 1 commit into
mainfrom
fix/tooltips
Open

Redeign tooltip for different intents usage#1754
Douglasymlai wants to merge 1 commit into
mainfrom
fix/tooltips

Conversation

@Douglasymlai

Copy link
Copy Markdown
Contributor

Related Issue

Description

Redesigns the tooltip system so behavior matches intent instead of using one global style everywhere.

Core changes (tooltip.tsx)

  • Introduces two tooltip variants:
    • default — 300ms hover delay, fade/zoom animation. For informational content (truncated text, hints, reconnect messages, pin/unpin actions).
    • instant — 0ms delay, no animation. For navigation and layout controls where the tooltip is just a label (icon buttons, folded sidebar items, top bar actions).
  • Removes per-tooltip nested TooltipProvider wrappers from TooltipSimple, so all tooltips share the app-level provider and the Radix “warm cursor” skip-delay window.
  • Updates default provider timing from 100ms to 300ms for both delayDuration and skipDelayDuration.

Call-site updates (19 files)

  • Applies variant="instant" to navigation/layout tooltips across the project sidebar, top bar, session panel, home hub toolbar, search input, triggers panel, and shadcn sidebar menu buttons.
  • Keeps variant="default" (implicit) for informational tooltips such as agent names and reconnect hints.
  • Removes ad-hoc delayDuration overrides where the new variants replace them.
  • Removes the sidebar’s nested TooltipProvider delayDuration={0} in favor of explicit instant tooltips on collapsed menu items.

This makes icon-only controls feel snappy while preserving a calmer, animated experience for content-heavy tooltips.

Testing Evidence (REQUIRED)

  • I have included human-verified testing evidence in this PR.
  • This PR includes frontend/UI changes, and I attached screenshot(s) or screen recording(s).
  • No frontend/UI changes in this PR.

Suggested manual checks:

  1. Instant tooltips — Hover folded sidebar nav items, top bar icon buttons (support, settings, fold/expand), and home hub view toggles. Tooltips should appear immediately with no animation.
  2. Default tooltips — Hover workforce agent cards (icon-only mode) and reconnect hints. Tooltips should appear after a short delay with fade/zoom animation.
  3. Warm cursor — Move quickly between adjacent icon buttons; subsequent tooltips should open without re-waiting the full delay.
  4. Collapsed sidebar — Confirm shadcn sidebar menu tooltips still show labels when collapsed.

What is the purpose of this pull request?

  • Bug fix
  • New Feature
  • Documentation update
  • Other

Contribution Guidelines Acknowledgement

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant