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
docs: add drag column sorting demo #48434
Conversation
Run & review this pull request in StackBlitz Codeflow. |
👁 Visual Regression Report for PR #48434 Passed ✅
🎊 Congrats! No visual-regression diff found. |
No dependency changes detected. Learn more about Socket for GitHub ↗︎ 👍 No dependency changes detected in pull request |
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 #48434 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 744 744
Lines 12887 12887
Branches 3371 3371
=========================================
Hits 12887 12887 ☔ View full report in Codecov by Sentry. |
Address 列有点太宽了,反而不适合演示,多加一些等宽列? |
👌添加了两列 |
交互效果改成 https://juejin.cn/post/7118266614339862559 这样如何? 拖拽列原地占位,待拖拽位置出现竖线。 |
可以,黑色的拖拽状态感觉有点难看,选一个颜色淡一点的token怎么样 |
尝试了一下,拖拽叠加的单元格不太容易拿到对应列的宽高 |
可以的。 |
不建议将 columns 放在 useState 里,这样会引导用户写同样的代码。columns 里的函数(onCell、 render 等)包含外部变量时,外部变量改变,函数是不会执行的。比如 render: () => stateA,stateA 变化了,表格里不会改变。 |
分组列拖拽有问题啊,因为ID的重复导致的问题@GeorgeHcc |
|
[中文版模板 / Chinese template]
🤔 This is a ...
🔗 Related issue link
#48306
💡 Background and solution
📝 Changelog
☑️ Self-Check before Merge