Skip to content
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

fix: Tooltip not work correctly on disabled Switch #17372

Merged
merged 3 commits into from Jun 29, 2019
Merged

Conversation

zombieJ
Copy link
Member

@zombieJ zombieJ commented Jun 29, 2019

🤔 This is a ...

  • New feature
  • Bug fix
  • Site / document update
  • Component style update
  • TypeScript definition update
  • Refactoring
  • Code style optimization
  • Test Case
  • Branch merge
  • Other (about what?)

👻 What's the background?

fix #17357

💡 Solution

Add check logic

📝 Changelog

Language Changelog
🇺🇸 English Fix Tooltip not work correctly on disabled Switch
🇨🇳 Chinese 修复 Tooltip 在 disabled Switch 中工作不正确的问题

☑️ Self Check before Merge

  • Doc is updated/provided or not needed
  • Demo is updated/provided or not needed
  • TypeScript definition is updated/provided or not needed
  • Changelog is provided or not needed

Copy link

@tests-checker tests-checker bot left a comment

Choose a reason for hiding this comment

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

Could you please add tests to make sure this change works as expected?

@netlify
Copy link

netlify bot commented Jun 29, 2019

Deploy preview for ant-design processing.

Building with commit 0b385f7

https://app.netlify.com/sites/ant-design/deploys/5d178e4d2c7df10008e2fadb

@zombieJ zombieJ merged commit a5f4956 into master Jun 29, 2019
@delete-merged-branch delete-merged-branch bot deleted the fix-toolyip branch June 29, 2019 16:29
@afc163 afc163 mentioned this pull request Jun 30, 2019
14 tasks
@arfeo
Copy link

arfeo commented Nov 20, 2019

Hey there,

What is the status of this merge? Looks like it hasn't appeared in any stable antd release...

Version 3.25.2: the bug is still there.

@damiangreen
Copy link
Contributor

This is still a bug in the latest 4.2.5 version. This case needs re-opening

@MYKEU
Copy link
Contributor

MYKEU commented May 28, 2020

Experiencing the same as above^

@Homingciu
Copy link

antd: 4.16.9, 依旧有问题!

@clintmilner
Copy link

My app is using 4.15.1 and I'm still seeing this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Tooltip still visible after mouseLeave in disabled Switch
6 participants