-
-
Notifications
You must be signed in to change notification settings - Fork 47.4k
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: typography double tooltip when hover operations #49008
base: master
Are you sure you want to change the base?
Conversation
Run & review this pull request in StackBlitz Codeflow. |
👁 Visual Regression Report for PR #49008 Failed ❌
Check Full Report for details If you think the visual diff is acceptable, please check:
|
This pull request is automatically built and testable in CodeSandbox. To see build info of the built libraries, click here or the icon next to each commit SHA. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #49008 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 750 749 -1
Lines 13129 13066 -63
Branches 3442 3431 -11
=========================================
- Hits 13129 13066 -63 ☔ View full report in Codecov by Sentry. |
Hi, @li-jia-nan, The action run says: This workflow is awaiting approval from a maintainer. Could you please help me to approval it? The runs link is here (updated) |
Hi,@zombieJ , I am sorry to trouble you. Could you please help me to approve the workflow? |
中文版模板 / Chinese template
🤔 This is a ...
🔗 Related issue link
💡 Background and solution
Background
Typography.Text with ellipsis and copyable trigger both tooltips instead of one, just like below:
Solution
Modify the structure of Typography component. When hovering operation like copy or edit, it only trigger it's own tooltip instead of trigger the text's tooltip.
📝 Changelog
☑️ Self-Check before Merge