Shabix

Results 11 comments of Shabix

@YyumeiZhang 您好,可以麻烦看看这个pr嘛,如果有需要改进的地方可以指点下嘛

![image](https://github.com/user-attachments/assets/de6f916a-de06-46b4-bfd3-a8444df2baa1) ![image](https://github.com/user-attachments/assets/675d6db1-9de4-4746-be73-37c6ff872e7d) 现在的表现是,同时设置的话现在虚拟化是生效的,分组的数据被合并了

@pointhalo 麻烦您了 请问这样解决这个issue可以嘛 希望得到您的批评指点~

> 抱歉,最近事情比较多。处理速度没跟上,预计下周整体看一下 > > * 另外可以先补充一下对应逻辑的测试用例,jest unit test 或者 cypress e2e test都可以 好的,我看一下哈

> 新增的测试用例通过,但arrayfield.test.js的一项遇到了问题,我会排查一下错误根源 之前的做法是希望用户使用add时默认在最后一行插入新行,用户可以传入索引在任意位置插入,未传入索引,默认插在最后一行。由于我的测试代码都是箭头函数调用,没有出现问题。之前的demo出现了如下的问题: 调整add方法后,支持传入索引 ```` add(index?: number) { const { keys } = this.state; const { field } = this.props; const updater = this.context; const newKey = getUuidv4(); if (index...

> > 新增的测试用例通过,但arrayfield.test.js的一项遇到了问题,我会排查一下错误根源 > > 之前的做法是希望用户使用add时默认在最后一行插入新行,用户可以传入索引在任意位置插入,未传入索引,默认插在最后一行。由于我的测试代码都是箭头函数调用,没有出现问题。之前的demo出现了如下的问题: > > 调整add方法后,支持传入索引 > > ``` > add(index?: number) { > const { keys } = this.state; > const { field } = this.props;...

> > > 新增的测试用例通过,但arrayfield.test.js的一项遇到了问题,我会排查一下错误根源 > > > > > > 之前的做法是希望用户使用add时默认在最后一行插入新行,用户可以传入索引在任意位置插入,未传入索引,默认插在最后一行。由于我的测试代码都是箭头函数调用,没有出现问题。之前的demo出现了如下的问题: > > 调整add方法后,支持传入索引 > > ``` > > add(index?: number) { > > const { keys } = this.state;...

在理解你的意思后我做了一些尝试,希望可以满足这个feature

我的看法是用户height设置超出默认的270px的情况下,应该是修改semi-select-option-list这个容器的max-height更合理些 ![image](https://github.com/user-attachments/assets/32a11d0d-89d0-4c9a-9fb3-fa8a2cef0f17) ![image](https://github.com/user-attachments/assets/b47ea3ee-5eaf-4ddc-b3d7-ce75fe8dc31b)