[Tooltip] When wrapping an input which is disabled while focused an error is thrown

[DavidJohnWilliams]

Steps to reproduce

Steps:

1. Open this link to live example: Code Sandbox
2. Click in the input as soon as the preview loads (if it is disabled already, refresh the preview and try again)
3. Wait a few seconds

Current behavior

A Javascript error is thrown

Expected behavior

The input should be disabled as usual

Context

Input has a useEffect which calls onBlur without an event. This is sort of documented i.e. a comment on the onBlur of Input however the Tooltip does not take account of this and tries to access event.target.

Your environment

It happens in any browser

Search keywords: tooltip target

[lukamandic]

Hi @DavidJohnWilliams, following your steps I can not reproduce this behavior

[DavidJohnWilliams]

@lukamandic Hmmm, I'm not really sure why? You need to click in the input once the sandbox loads (before the input is disabled), then wait. That's it. If it is disabled too fast for you, increase the timeout or refresh the preview.

20250220-2346-39.9438120.mp4

[siriwatknp]

I can reproduce the issue. I think this is a bug.

[sarathkumarsasi]

I will work on this

[mj12albert]

Quickly tried something like this should work:

diff --git a/packages/mui-material/src/Tooltip/Tooltip.js b/packages/mui-material/src/Tooltip/Tooltip.js
index c7479ac521..b1129eb799 100644
--- a/packages/mui-material/src/Tooltip/Tooltip.js
+++ b/packages/mui-material/src/Tooltip/Tooltip.js
@@ -478,8 +478,8 @@ const Tooltip = React.forwardRef(function Tooltip(inProps, ref) {
 
   const [, setChildIsFocusVisible] = React.useState(false);
   const handleBlur = (event) => {
-    if (!isFocusVisible(event.target)) {
-      setChildIsFocusVisible(false);
+    setChildIsFocusVisible(false);
+    if (event && !isFocusVisible(event.target)) {
       handleMouseLeave(event);
     }
   };

@sarathkumarsasi I've assigned you ~

[sarathkumarsasi]

mj12albert

The setChildNode function also utilizes the event object, so I added a null check at the beginning of the handleFocus function. Is this acceptable?

[mj12albert]

Is this acceptable?

@sarathkumarsasi If it's not directly related to the issue, I would suggest leaving it alone 🤔