-
Notifications
You must be signed in to change notification settings - Fork 933
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
[Bug] Fix the tooltip display misalignment issue #8202
Conversation
Signed-off-by: yubonluo <[email protected]>
Hi @AMoo-Miki , the PR is related to your PR(https://github.com/opensearch-project/OpenSearch-Dashboards/pull/7637/files#diff-2d652eece8cdb99548e2f74430a037f961e4525379c683e551118e378f70f0bfR13). If you have time, please take a look. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #8202 +/- ##
=======================================
Coverage 64.05% 64.05%
=======================================
Files 3741 3741
Lines 88629 88629
Branches 13801 13801
=======================================
Hits 56771 56771
Misses 31260 31260
Partials 598 598
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice fix!
@AMoo-Miki Would be nice if you could take a look at this PR :), the style change was introduced in your previous commit 0a3bcd8 |
The same fix was merged in https://github.com/opensearch-project/OpenSearch-Dashboards/pull/8223/files |
Closing this one as fixed in #8223 |
Description
There will be a display issue of content misalignming when showing tooltip on the page.
Before showing tooltip:
After showing tooltip:
Issues Resolved
Screenshot
After fixing the issue:
Testing the changes
Changelog
Check List
yarn test:jest
yarn test:jest_integration