Skip to content

Conversation

@atomiks
Copy link
Contributor

@atomiks atomiks commented Dec 7, 2025

To reproduce: https://deploy-preview-3452--base-ui.netlify.app/react/components/tooltip#animating-the-tooltip

  • Add gap: 30px (or whatever substantial value) between the triggers on the flex container
  • Hover first trigger and wait for tooltip to open
  • Hover to second trigger, then third
  • Previously the tooltip would close when reaching the last trigger because the delay got reset back to the initial (due to the timeout not being cleared in a shared scenario.)

@atomiks atomiks added type: bug It doesn't behave as expected. component: tooltip Changes related to the tooltip component. labels Dec 7, 2025
@pkg-pr-new
Copy link

pkg-pr-new bot commented Dec 7, 2025

  • vite-css-base-ui-example

    pnpm add https://pkg.pr.new/mui/base-ui/@base-ui/react@3452
    
    pnpm add https://pkg.pr.new/mui/base-ui/@base-ui/utils@3452
    

commit: ac4e5c8

@mui-bot
Copy link

mui-bot commented Dec 7, 2025

Bundle size report

Bundle Parsed size Gzip size
@base-ui/react 🔺+186B(+0.05%) 🔺+45B(+0.03%)

Details of bundle changes


Check out the code infra dashboard for more information about this PR.

@atomiks atomiks force-pushed the fix/tooltip-group-timeout-clear branch from a9be3a5 to 6677e5d Compare December 7, 2025 11:29
@netlify
Copy link

netlify bot commented Dec 7, 2025

Deploy Preview for base-ui ready!

Name Link
🔨 Latest commit a9be3a5
🔍 Latest deploy log https://app.netlify.com/projects/base-ui/deploys/693564ac0349b400088dd668
😎 Deploy Preview https://deploy-preview-3452--base-ui.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@netlify
Copy link

netlify bot commented Dec 7, 2025

Deploy Preview for base-ui ready!

Name Link
🔨 Latest commit ac4e5c8
🔍 Latest deploy log https://app.netlify.com/projects/base-ui/deploys/693972d788a8110009e40c50
😎 Deploy Preview https://deploy-preview-3452--base-ui.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@atomiks atomiks marked this pull request as ready for review December 7, 2025 11:32
@atomiks atomiks force-pushed the fix/tooltip-group-timeout-clear branch 4 times, most recently from d8f1cf1 to fa59ea8 Compare December 10, 2025 01:37
@github-actions github-actions bot added the PR: out-of-date The pull request has merge conflicts and can't be merged. label Dec 10, 2025
@atomiks atomiks force-pushed the fix/tooltip-group-timeout-clear branch from fa59ea8 to c53f6ac Compare December 10, 2025 11:23
@github-actions github-actions bot removed the PR: out-of-date The pull request has merge conflicts and can't be merged. label Dec 10, 2025
@atomiks atomiks force-pushed the fix/tooltip-group-timeout-clear branch from 02cf8d4 to 5c0b79f Compare December 10, 2025 12:57
@atomiks atomiks merged commit cffdde3 into mui:master Dec 10, 2025
21 of 22 checks passed
@atomiks atomiks deleted the fix/tooltip-group-timeout-clear branch December 10, 2025 13:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component: tooltip Changes related to the tooltip component. type: bug It doesn't behave as expected.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants