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

🐛version 1.33.0 版本详情表 鼠标滚动 报错,望尽快修复 #1868

Closed
1 of 4 tasks
guizhichao opened this issue Nov 2, 2022 · 13 comments · Fixed by #1874
Closed
1 of 4 tasks

🐛version 1.33.0 版本详情表 鼠标滚动 报错,望尽快修复 #1868

guizhichao opened this issue Nov 2, 2022 · 13 comments · Fixed by #1874
Assignees
Labels

Comments

@guizhichao
Copy link

guizhichao commented Nov 2, 2022

🏷 Version 1.33.0

image

1.33.0版本有问题,详情表报错,望尽快修复。。

1.31.0 版本没这个问题

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

Sheet Type

  • PivotSheet
  • TableSheet
  • GridAnalysisSheet
  • StrategySheet

🖋 Description

⌨️ Code Snapshots

🔗 Reproduce Link

🤔 Steps to Reproduce

😊 Expected Behavior

😅 Current Behavior

💻 System information

@lcx-seima lcx-seima added the 🤔 need reproduce 需要一个在线复现链接 label Nov 2, 2022
@github-actions
Copy link
Contributor

github-actions bot commented Nov 2, 2022

你好 @guizhichao,请提供一个可以在线访问的链接, 或者使用 codesandbox 提供示例并详细说明复现步骤 (查看更多模板), 15 天内未回复issue自动关闭。

Hello, @guizhichao, please provide an accessible link or usage codesandbox to describe the reproduction steps (more template). The issue will be closed without any replay within 15 days.

@serializedowen
Copy link
Member

辛苦提供一下复现途径

@guizhichao
Copy link
Author

guizhichao commented Nov 2, 2022

辛苦提供一下复现途径

https://codesandbox.io/s/priceless-pateu-5nrwmx?file=/src/index.js 换成这个路径了

@github-actions github-actions bot removed the 🤔 need reproduce 需要一个在线复现链接 label Nov 2, 2022
@stone-lyl
Copy link
Member

https://codesandbox.io/s/jolly-goodall-9hsw1q?file=/src/index.js

无法复现,麻烦提供一下复现步骤

image

@guizhichao
Copy link
Author

不好意思,我提了两个,链接被覆盖了,现在好了

https://codesandbox.io/s/priceless-pateu-5nrwmx?file=/src/index.js

@guizhichao guizhichao changed the title 🐛version 1.33.0 版本详情表 报错,望尽快修复 🐛version 1.33.0 版本详情表 鼠标滚动 报错,望尽快修复 Nov 3, 2022
@guizhichao
Copy link
Author

guizhichao commented Nov 3, 2022

再试下呢,1.33.0 版本 详情表格鼠标滚动的时候会报错, 1.32.0版本就是好的

@guizhichao
Copy link
Author

image

@stone-lyl
Copy link
Member

stone-lyl commented Nov 3, 2022

你多包了一层数组,导致的报错了。你的配置太混乱了,也很容易导致其他问题。
https://codesandbox.io/s/modest-goldberg-mlf6ky?file=/src/index.js 我改动后,就没有这个问题了

@stone-lyl stone-lyl self-assigned this Nov 3, 2022
@guizhichao
Copy link
Author

image

不是这个导致的啊,为啥1.32.0版本好的啊,到1.33.0就不行了呢。而且我们也是参考你们官方文档做的呀,是个二维数组

@guizhichao
Copy link
Author

image

@stone-lyl
Copy link
Member

不是这个导致的啊,为啥1.32.0版本好的啊,到1.33.0就不行了呢。而且我们也是参考你们官方文档做的呀,是个二维数组

我这边弄错了,以为是mergeCells 的格式问题。
是因为主题没有适配 mergeCells ,你使用 1.33.0 版本,可以先把 mergedCellInfo 的相关的配置去掉。
下个版本中会修复这个问题。

@guizhichao
Copy link
Author

不是这个导致的啊,为啥1.32.0版本好的啊,到1.33.0就不行了呢。而且我们也是参考你们官方文档做的呀,是个二维数组

我这边弄错了,以为是mergeCells 的格式问题。 是因为主题没有适配 mergeCells ,你使用 1.33.0 版本,可以先把 mergedCellInfo 的相关的配置去掉。 下个版本中会修复这个问题。
好的,感谢🙏,我本地回滚到1.32.0版本了

@lijinke666
Copy link
Member

🎉 This issue has been resolved in version @antv/s2-v1.33.1 🎉

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
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants