-
Notifications
You must be signed in to change notification settings - Fork 205
fix: 修复移动端点击事件触发多次 #3141
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: 修复移动端点击事件触发多次 #3141
Conversation
Refactored line break escaping logic to ensure consistent CSV export formatting, specifically handling \r\n and \n conversions more robustly.
你好 @Create-Peace,非常感谢你的贡献. Hello, @Create-Peace, Thanks for your contribution. In order to make the code more robust, please add the corresponding unit tests, and update the docs if there are API changes. |
Walkthrough此PR修复了移动端点击事件触发多次的问题。通过在事件控制器中根据设备类型选择合适的点击事件(TOUCH_END或POINTER_UP),避免了移动端点击单元格时事件回调被触发两次的情况。 Changes
|
@Create-Peace 单元测试没有过 |
This reverts commit e942d04.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## next #3141 +/- ##
==========================================
+ Coverage 75.77% 80.48% +4.70%
==========================================
Files 257 166 -91
Lines 11994 10400 -1594
Branches 2464 2377 -87
==========================================
- Hits 9088 8370 -718
+ Misses 1398 579 -819
+ Partials 1508 1451 -57 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
感谢你的贡献 |
👀 PR includes
🐛 Bugfix
📝 Description
移动端点击单元格触发两次事件回调
🖼️ Screenshot
🔗 Related issue link
🔍 Self-Check before the merge