-
Notifications
You must be signed in to change notification settings - Fork 57
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Tooltip hide on disabled input doesn't work #41
Comments
This is a known issue with React on Chrome and Safari, where Ref: facebook/react#4251 |
Seems it's going to be fixed in coming React 16.13.0 |
@denisborovikov yeah, I had asked here facebook/react#17675 (comment) just to be sure it's the exact same issue. |
They have now closed the issue. I guess we can close this one as well, as I can see that your sandbox is working as expected after the upgrade. |
Huh, now it means we can't even have a tooltip on a disabled button. onMouseEnter just doesn't fire. |
That's what should happen according to HTML spec. If you still want it, you can wrap the button with a div. |
Sure, or use a fake button as well. Did know it's according to spec. |
We had the same problem at hazelcast/hive#58. We ended up using a child component inside of the button as the tooltip trigger. |
If the trigger element is a disabled button, the tooltip isn't hiding on mouse leave.
Demo: https://codesandbox.io/s/61yv95m533
The text was updated successfully, but these errors were encountered: