Skip to content

Fix tooltip delay in Studio UI - #3942

Merged
shaunandrews merged 3 commits into
trunkfrom
stu-1883-enable-tooltip-delay-before-showing
Jun 24, 2026
Merged

Fix tooltip delay in Studio UI#3942
shaunandrews merged 3 commits into
trunkfrom
stu-1883-enable-tooltip-delay-before-showing

Conversation

@shaunandrews

Copy link
Copy Markdown
Contributor

Related issues

How AI was used in this PR

Codex helped inspect the tooltip usage, research the upstream @wordpress/ui behavior, implement the Studio-side fix, and run verification. I reviewed the resulting diff and test output.

Proposed Changes

Studio UI tooltips now respect the WPDS tooltip delay instead of appearing immediately on hover. This reduces noisy tooltip popups while users move across dense controls or click buttons.

The fix also patches the bundled @wordpress/ui IconButton output used by Studio so its built-in tooltips no longer force an immediate zero-delay hover behavior.

⚠️ Visual change: needs human review in light + dark mode.

Testing Instructions

  • Run npm test -- apps/ui/src/components/site-list/index.test.tsx apps/ui/src/components/site-preview/index.test.tsx
  • Run npm run typecheck
  • In Studio, hover icon-only controls in the agentic UI/sidebar and confirm tooltips wait briefly before appearing instead of showing immediately.

Pre-merge Checklist

  • Have you checked for TypeScript, React or other console errors?
@shaunandrews
shaunandrews marked this pull request as ready for review June 24, 2026 01:11
@shaunandrews
shaunandrews requested review from a team and bcotrim June 24, 2026 01:11

@katinthehatsite katinthehatsite left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Just a sidenote, I had to apply the patch first with npx patch-package --patch-dir apps/studio/patches to see the changes proposed in this PR to be able to test them (in case it is useful for the next person testing)

Comment thread apps/ui/src/components/site-preview/index.test.tsx

@katinthehatsite katinthehatsite left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The changes look good to me, I left a couple of minor comments but they are non blocking

@bcotrim bcotrim left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Works as described, a nice improvement to the user experience!
LGTM 👍

…tip-delay-before-showing

# Conflicts:
#	apps/ui/src/components/site-list/index.tsx
@wpmobilebot

Copy link
Copy Markdown
Collaborator

📊 Performance Test Results

Comparing d0afb4a vs trunk

app-size

Metric trunk d0afb4a Diff Change
App Size (Mac) 1311.88 MB 1311.88 MB 0.00 MB ⚪ 0.0%

site-editor

Metric trunk d0afb4a Diff Change
load 1093 ms 1070 ms 23 ms ⚪ 0.0%

site-startup

Metric trunk d0afb4a Diff Change
siteCreation 6507 ms 6518 ms +11 ms ⚪ 0.0%
siteStartup 6522 ms 6996 ms +474 ms 🔴 7.3%

Results are median values from multiple test runs.

Legend: 🟢 Improvement (faster) | 🔴 Regression (slower) | ⚪ No change (<50ms diff)

@shaunandrews
shaunandrews merged commit 939addf into trunk Jun 24, 2026
11 checks passed
@shaunandrews
shaunandrews deleted the stu-1883-enable-tooltip-delay-before-showing branch June 24, 2026 20:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

4 participants