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

[Select 选择器] Select Props 属性中 使用onRemove 事件失效 #2789

Closed
zedwei2 opened this issue Mar 5, 2024 · 10 comments
Closed

[Select 选择器] Select Props 属性中 使用onRemove 事件失效 #2789

zedwei2 opened this issue Mar 5, 2024 · 10 comments
Labels
🐞 bug Something isn't working to be published fixed, not be published

Comments

@zedwei2
Copy link

zedwei2 commented Mar 5, 2024

tdesign-react 版本

1.5.2

重现链接

https://codesandbox.io/s/tdesign-react-demo-forked-jpz77g?file=/src/demo.jsx

重现步骤

1.打开【重现链接】的地址
2.移除第一个select中某一项数据
3.查看控制台打印情况

期望结果

可以触发onRemove事件

实际结果

无法触发onRemove事件

框架版本

No response

浏览器版本

No response

系统版本

No response

Node版本

No response

补充说明

No response

Copy link
Contributor

github-actions bot commented Mar 5, 2024

👋 @zedwei2,感谢给 TDesign 提出了 issue。
请根据 issue 模版确保背景信息的完善,我们将调查并尽快回复你。

@HaixingOoO HaixingOoO added the 🐞 bug Something isn't working label Mar 5, 2024
@Ali-ovo
Copy link
Contributor

Ali-ovo commented Mar 8, 2024

看了一下代码这个改的话 onRmove 事件得改一下类型了,只能传出当前选中的key或者当前删除的节点出来,而不是现在定义的 SelectRemoveContext 类型
@HaixingOoO 需要改吗 我这边可以提个mr

@HaixingOoO
Copy link
Collaborator

看了一下代码这个改的话 onRmove 事件得改一下类型了,只能传出当前选中的key或者当前删除的节点出来,而不是现在定义的 SelectRemoveContext 类型

@HaixingOoO 需要改吗 我这边可以提个mr

我们已经在修复了,谢谢!!!

@HaixingOoO HaixingOoO added the in progress in progress label Mar 8, 2024
@Ali-ovo
Copy link
Contributor

Ali-ovo commented Mar 8, 2024

看了一下代码这个改的话 onRmove 事件得改一下类型了,只能传出当前选中的key或者当前删除的节点出来,而不是现在定义的 SelectRemoveContext 类型
@HaixingOoO 需要改吗 我这边可以提个mr

我们已经在修复了,谢谢!!!

好的

@HaixingOoO
Copy link
Collaborator

好的

欢迎后续更多的参与和贡献!😁

@uyarn
Copy link
Collaborator

uyarn commented Mar 8, 2024

看了一下代码这个改的话 onRmove 事件得改一下类型了,只能传出当前选中的key或者当前删除的节点出来,而不是现在定义的 SelectRemoveContext 类型
@HaixingOoO 需要改吗 我这边可以提个mr

我们已经在修复了,谢谢!!!

好的

@Ali-ovo 才看到 本来是我修复 那你提个PR吧 需要你的热情~

@Ali-ovo
Copy link
Contributor

Ali-ovo commented Mar 8, 2024

看了一下代码这个改的话 onRmove 事件得改一下类型了,只能传出当前选中的key或者当前删除的节点出来,而不是现在定义的 SelectRemoveContext 类型
@HaixingOoO 需要改吗 我这边可以提个mr

我们已经在修复了,谢谢!!!

好的

@Ali-ovo 才看到 本来是我修复 那你提个PR吧 需要你的热情~

哈哈,感谢 之前我也给 tdesign 做过很多贡献

这个bug看我说的 我想的是 onRmove 的参数是当前剩下选中的key或者是删除的节点 这样可以吗
目前是无法按照 定义的 SelectRemoveContext 类型 来返回值

或者是我的代码地方理解错了吗?

@uyarn
Copy link
Collaborator

uyarn commented Mar 14, 2024

看了一下代码这个改的话 onRmove 事件得改一下类型了,只能传出当前选中的key或者当前删除的节点出来,而不是现在定义的 SelectRemoveContext 类型
@HaixingOoO 需要改吗 我这边可以提个mr

我们已经在修复了,谢谢!!!

好的

@Ali-ovo 才看到 本来是我修复 那你提个PR吧 需要你的热情~

哈哈,感谢 之前我也给 tdesign 做过很多贡献

这个bug看我说的 我想的是 onRmove 的参数是当前剩下选中的key或者是删除的节点 这样可以吗 目前是无法按照 定义的 SelectRemoveContext 类型 来返回值

或者是我的代码地方理解错了吗?

不好意思 忘记回消息了 @Ali-ovo 我理解现在的参数类型是可以满足的 里面的value data都是指这个被删除的选项

@Ali-ovo
Copy link
Contributor

Ali-ovo commented Mar 15, 2024

看了一下代码这个改的话 onRmove 事件得改一下类型了,只能传出当前选中的key或者当前删除的节点出来,而不是现在定义的 SelectRemoveContext 类型
@HaixingOoO 需要改吗 我这边可以提个mr

我们已经在修复了,谢谢!!!

好的

@Ali-ovo 才看到 本来是我修复 那你提个PR吧 需要你的热情~

哈哈,感谢 之前我也给 tdesign 做过很多贡献
这个bug看我说的 我想的是 onRmove 的参数是当前剩下选中的key或者是删除的节点 这样可以吗 目前是无法按照 定义的 SelectRemoveContext 类型 来返回值
或者是我的代码地方理解错了吗?

不好意思 忘记回消息了 @Ali-ovo 我理解现在的参数类型是可以满足的 里面的value data都是指这个被删除的选项

好的 这个ts类型貌似一直报错 我该了一版 请review一下

Ali-ovo added a commit to Ali-ovo/tdesign-react that referenced this issue Mar 15, 2024
@uyarn uyarn added to be published fixed, not be published and removed in progress in progress labels Mar 15, 2024
@uyarn uyarn closed this as completed Mar 28, 2024
@uyarn
Copy link
Collaborator

uyarn commented Mar 28, 2024

fixed 1.5.4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐞 bug Something isn't working to be published fixed, not be published
Projects
None yet
Development

No branches or pull requests

4 participants