找回密码
 立即注册

QQ登录

只需一步,快速开始

spreadjs666
金牌服务用户   /  发表于:2024-2-27 12:04  /   查看:1095  /  回复:10
本帖最后由 Ellia.Duan 于 2024-3-19 09:31 编辑

调研编号:SJS-23024
产品:SpreadJS
版本:V17
LastReview:2024-3-19
该问题计划在v17.0.5版本修复。



v17删除行再撤销,RowChanged中返回的count与v16不一致是什么原因呢?
image.png312244872.png

image.png797456054.png

评分

参与人数 1金币 +200 收起 理由
Joestar.Xu + 200 赞一个!

查看全部评分

10 个回复

倒序浏览
Joestar.XuSpreadJS 开发认证
超级版主   /  发表于:2024-2-27 12:15:31
沙发
您好,我在16.2.6和17.0.3分别测试了一下,删除第十行后又撤销,结果都是一样的,没有复现出您这个问题。

image.png296594847.png

image.png489591674.png
SpreadJS 17.0.9 | GcExcel 7.1.2 已发布~
回复 使用道具 举报
spreadjs666
金牌服务用户   /  发表于:2024-2-27 14:10:26
板凳
Joestar.Xu 发表于 2024-2-27 12:15
您好,我在16.2.6和17.0.3分别测试了一下,删除第十行后又撤销,结果都是一样的,没有复现出您这个问题。
...

我这里使用的是16.1.0和17.0.2,另外count这个字段是什么含义呢?不是表示被操作的行数吗?
回复 使用道具 举报
Joestar.XuSpreadJS 开发认证
超级版主   /  发表于:2024-2-27 14:28:11
地板
本帖最后由 Joestar.Xu 于 2024-2-27 14:30 编辑

您好,V16.1.0和V17的表现也是一致的:

image.png172361449.png

在目前的版本中,count属性在插入时表示插入的行数,在撤销时表示撤销后sheet的行数。


另外,RowChanged事件中公开的属性只有如下几个:

image.png506389770.png

count属性暂未公开,属于事件内部的属性,在将来的版本中可能会发生更改,不建议您使用,您这边为什么需要获取这个count呢?
SpreadJS 17.0.9 | GcExcel 7.1.2 已发布~
回复 使用道具 举报
Joestar.XuSpreadJS 开发认证
超级版主   /  发表于:2024-2-28 13:46:31
5#
您好,请问您的问题是否已经解决,如果仍未解决,欢迎继续回帖,我们来协助调研。
SpreadJS 17.0.9 | GcExcel 7.1.2 已发布~
回复 使用道具 举报
spreadjs666
金牌服务用户   /  发表于:2024-2-29 11:53:11
6#
Joestar.Xu 发表于 2024-2-28 13:46
您好,请问您的问题是否已经解决,如果仍未解决,欢迎继续回帖,我们来协助调研。

您好,我们重写了右键删除行的逻辑,但是点击撤销的时候V17的 rowchange中的isUndo是false,但是同样的代码在V16的是正常的,辛苦看下什么原因呢?
已上传附件及录屏
V17
image.png797123718.png
V16
image.png953291625.png

屏幕录制2024-02-29 上午11.45.20.mov.zip

2.54 MB, 下载次数: 44

excel.zip

6.31 MB, 下载次数: 41

回复 使用道具 举报
Joestar.XuSpreadJS 开发认证
超级版主   /  发表于:2024-2-29 12:15:57
7#
您好,您描述的情况我了解了,这边调研一下,后续有进展我会在本贴中回复您,另外想问一下您为什么要关注这个属性呢?
SpreadJS 17.0.9 | GcExcel 7.1.2 已发布~
回复 使用道具 举报
spreadjs666
金牌服务用户   /  发表于:2024-2-29 14:06:47
8#
Joestar.Xu 发表于 2024-2-29 12:15
您好,您描述的情况我了解了,这边调研一下,后续有进展我会在本贴中回复您,另外想问一下您为什么要关注这 ...

我们是在做在线协同相关的内容会用到这个属性,用来判断是否是撤销操作从而进行一些数据的同步
回复 使用道具 举报
Joestar.XuSpreadJS 开发认证
超级版主   /  发表于:2024-2-29 14:17:50
9#
好的,了解了,这边调研一下,后续有进展我会在本贴中回复您。
SpreadJS 17.0.9 | GcExcel 7.1.2 已发布~
回复 使用道具 举报
Joestar.XuSpreadJS 开发认证
超级版主   /  发表于:2024-3-4 15:03:25
10#
您好,您描述的情况初步认定为是一个Bug,目前正在修复中,等修复完毕我会在帖子下回复您。
SpreadJS 17.0.9 | GcExcel 7.1.2 已发布~
回复 使用道具 举报
12下一页
您需要登录后才可以回帖 登录 | 立即注册
返回顶部