Skip to content
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

透视表列多时,使用触摸板左右快速滑动,会报栈溢出错误 #2627

Closed
1 of 5 tasks
Create-Peace opened this issue Mar 25, 2024 · 8 comments · Fixed by #2643
Closed
1 of 5 tasks

透视表列多时,使用触摸板左右快速滑动,会报栈溢出错误 #2627

Create-Peace opened this issue Mar 25, 2024 · 8 comments · Fixed by #2643
Assignees
Labels
next 2.0-next 版本的问题 released on @next

Comments

@Create-Peace
Copy link

Create-Peace commented Mar 25, 2024

🏷 Version

Package Version
@antv/s2 next
@antv/s2-react
@antv/s2-vue

Sheet Type

  • PivotSheet
  • TableSheet
  • GridAnalysisSheet
  • StrategySheet
  • EditableSheet

🖋 Description

透视表列比较多,使用触摸板左右滑动快时会出现栈溢出错误, 看着像是@antv/g引擎报错了。

Uncaught RangeError: Maximum call stack size exceeded
    at @antv_s2.js?v=75bfa16a:39272:55
    at @antv_s2.js?v=75bfa16a:10425:18
    at Array.forEach (<anonymous>)
    at SyncHook2.call (@antv_s2.js?v=75bfa16a:10424:22)
    at RenderingService2.render (@antv_s2.js?v=75bfa16a:12154:29)
    at Canvas2.render (@antv_s2.js?v=75bfa16a:16213:24)
    at RootInteraction2.draw (@antv_s2.js?v=75bfa16a:43746:34)
    at RootInteraction2.clearState (@antv_s2.js?v=75bfa16a:43750:14)
    at RootInteraction2.changeState (@antv_s2.js?v=75bfa16a:43769:12)
    at PivotSheet2.<anonymous> (@antv_s2.js?v=75bfa16a:41177:21)

⌨️ Code Snapshots

官网demo向右滑动快的时候就能复现这个问题
https://s2.antv.antgroup.com/examples/case/performance-compare/#pivot

🔗 Reproduce Link

image

🤔 Steps to Reproduce

😊 Expected Behavior

😅 Current Behavior

💻 System information

Environment Info
System
Browser
@Create-Peace Create-Peace changed the title 透视表列多时,左右滑动,会报栈溢出错误 透视表列多时,使用触摸板左右快速滑动,会报栈溢出错误 Mar 26, 2024
@wuhaiyang wuhaiyang assigned wuhaiyang and lijinke666 and unassigned wuhaiyang Mar 27, 2024
@lijinke666
Copy link
Member

未复现

@Create-Peace
Copy link
Author

我使用的是Windows 10, 录了个视频。https://pan.baidu.com/s/141qet3k0MmyH2DeHVIp_wA 提取码: 3wir

@lijinke666
Copy link
Member

没有 windows 电脑, 无法复现, 你可以尝试定位下问题, 来个 PR

@Create-Peace
Copy link
Author

我尝试修复了,没有定位到问题,Mac 也能复现,向右滑动最后。

@DavisYao88 DavisYao88 mentioned this issue Apr 1, 2024
10 tasks
@lijinke666 lijinke666 added the next 2.0-next 版本的问题 label Apr 30, 2024
@EliazTray
Copy link

image
同样也是很多透视列,横向快速滚动时,会循环调用,导致栈溢出。

SpreadSheet 中 emit GLOBAL_MOUSE_MOVE
image

@lijinke666 lijinke666 linked a pull request May 27, 2024 that will close this issue
10 tasks
@lijinke666
Copy link
Member

下周发布

@lijinke666
Copy link
Member

🎉 This issue has been resolved in version @antv/s2-v2.0.0-next.23 🎉

The release is available on:

Your semantic-release bot 📦🚀

@lijinke666
Copy link
Member

🎉 This issue has been resolved in version @antv/s2-react-v2.0.0-next.22 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
next 2.0-next 版本的问题 released on @next
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants