-
Notifications
You must be signed in to change notification settings - Fork 199
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
refactor: 趋势表tooltip类名不再使用css-modules #1501
Conversation
Size Change: -68 B (0%) Total Size: 253 kB
ℹ️ View Unchanged
|
你好, @lcx-seima CI 执行失败, 请点击 [Details] 按钮查看, 并根据日志修复。 Hello, @lcx-seima CI run failed, please click the [Details] button for detailed log information and fix it. |
Codecov Report
@@ Coverage Diff @@
## master #1501 +/- ##
==========================================
+ Coverage 76.27% 76.33% +0.05%
==========================================
Files 227 228 +1
Lines 10847 10866 +19
Branches 2244 2248 +4
==========================================
+ Hits 8274 8295 +21
Misses 1313 1313
+ Partials 1260 1258 -2
Continue to review full report at Codecov.
|
rebase 下 master, 有个测试挂了, master 修复了 |
packages/s2-react/src/components/sheets/strategy-sheet/custom-tooltip/index.less
Outdated
Show resolved
Hide resolved
2c4252d
to
528e564
Compare
packages/s2-react/src/components/sheets/strategy-sheet/utils/index.ts
Outdated
Show resolved
Hide resolved
u1s1, 这个动图属实是没看出来差别在哪里🧐 |
This reverts commit 85d3637.
要的就是没差别呀,有问题就出事情了。LOL |
🎉 This PR is included in version @antv/s2-react-v1.21.0-alpha.1 🎉 The release is available on: Your semantic-release bot 📦🚀 |
* refactor: 趋势表tooltip类名不再使用css-modules * refactor: 统一css类名前缀变量到shared
🎉 This PR is included in version @antv/s2-react-v1.21.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
🎉 This PR is included in version 1.0.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
* refactor: 趋势表tooltip类名不再使用css-modules * refactor: 统一css类名前缀变量到shared
👀 PR includes
🎨 Enhance
📝 Description
css-modules
🖼️ Screenshot
🔗 Related issue link
🔍 Self-Check before the merge