antvis / S2

⚡️ A practical visualization library for tabular analysis.
https://s2.antv.antgroup.com
MIT License
1.49k stars 196 forks source link

fix: 修复排序菜单没有子节点时图标展示错误 & 调整默认样式 #2932

Closed lijinke666 closed 1 month ago

lijinke666 commented 1 month ago

👀 PR includes

🐛 Bugfix

📝 Description

🖼️ Screenshot

Before After
image image
image image

🔗 Related issue link

🔍 Self-Check before the merge

vercel[bot] commented 1 month ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
antvis-s2 ❌ Failed (Inspect) Oct 22, 2024 8:55am
github-actions[bot] commented 1 month ago

Size Change: -207 B (-0.06%)

Total Size: 329 kB

Filename Size Change
./packages/s2-react/dist/index.min.js 70.9 kB +9 B (+0.01%)
./packages/s2-react/dist/style.min.css 3.77 kB -106 B (-2.73%)
./packages/s2-vue/dist/style.min.css 1.85 kB -110 B (-5.62%)
ℹ️ View Unchanged | Filename | Size | | :--- | :---: | | `./packages/s2-core/dist/index.min.js` | 229 kB | | `./packages/s2-core/dist/style.min.css` | 388 B | | `./packages/s2-vue/dist/index.min.js` | 22.9 kB |

compressed-size-action

github-actions[bot] commented 1 month ago

你好, @lijinke666 CI 执行失败, 请点击 [Details] 按钮查看, 并根据日志修复。

Hello, @lijinke666 CI run failed, please click the [Details] button for detailed log information and fix it.

codecov-commenter commented 1 month ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 80.83%. Comparing base (ecdc9c8) to head (8a4ad11). Report is 453 commits behind head on next.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## next #2932 +/- ## ========================================== + Coverage 75.77% 80.83% +5.06% ========================================== Files 257 180 -77 Lines 11994 10426 -1568 Branches 2464 2365 -99 ========================================== - Hits 9088 8428 -660 + Misses 1398 554 -844 + Partials 1508 1444 -64 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

lijinke666 commented 4 weeks ago

:tada: This PR is included in version @antv/s2-react-v2.0.0-next.29 :tada:

The release is available on:

Your semantic-release bot :package::rocket: