-
Notifications
You must be signed in to change notification settings - Fork 7.7k
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(editor): Remove additional margin on tooltip #6802
fix(editor): Remove additional margin on tooltip #6802
Conversation
… has a default 12px offset, the old had 0)
Great PR! Please pay attention to the following items before merging: Files matching
Make sure to check off this list before asking for review. |
Codecov ReportPatch coverage has no change and project coverage change:
Additional details and impacted files@@ Coverage Diff @@
## master #6802 +/- ##
==========================================
- Coverage 24.62% 24.62% -0.01%
==========================================
Files 3122 3121 -1
Lines 190461 190417 -44
Branches 20930 20921 -9
==========================================
- Hits 46903 46885 -18
+ Misses 142630 142608 -22
+ Partials 928 924 -4 ☔ View full report in Codecov by Sentry. |
|
1 flaky tests on run #1687 ↗︎
Details:
cypress/e2e/2-credentials.cy.ts • 1 flaky test
This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. |
…re-to-far-from-triggering-content
✅ All Cypress E2E specs passed |
Got released with |
Element UI+ tooltip has a default 12px offset, the old had 0