edc-hui comments

Results 25 comments of


                                            edc-hui

Table onRow -> onClick wrongly fired when clicking on the Checkbox (only when unselecting)

经过测试antd 5.x 和 antd 4.x 均有此问题

fix(Checkbox): clicking on the label tag will cause the event to bubble, causing the parent div click event to be triggered.

@zombieJ hello ，帮忙看下这个ci 未通过是啥原因啊

fix(Checkbox): clicking on the label tag will cause the event to bubble, causing the parent div click event to be triggered.

> > > 点击 Detail 可以看到报错的具体 test case 哈： > > ``` > 2022-12-14T02:08:31.5301287Z FAIL components/table/__tests__/Table.rowSelection.test.tsx (29.28 s) > 2022-12-14T02:08:31.5302140Z ● Table.rowSelection › should keep item even if in filter...

fix(Checkbox): clicking on the label tag will cause the event to bubble, causing the parent div click event to be triggered.

@zombieJ #38926所描述的问题在Transfer组件中也是可以复现出来。想要彻底解决这个问题，我目前想到的方案是设置Checkbox组件中label标签的line-height为0，目的是让label标签和Checkbox组件中的input等高。这样在Table中选中Checkbox，点击label标签事件冒泡触发Table的click事件的问题就不会出现。但是line-height为0这个样式的变更会导致Transfer组件，Checkbox与文字对不齐的问题，故我这边修改了Transfer组件的样式。 Checkbox中原先设置给.antd-checkbox的top: 0.2em样式，现移除，并给其父元素.antd-checkbox-wrapper设置transform: translateY(0.2em)。这种解决方案您认可么？

fix(Checkbox): clicking on the label tag will cause the event to bubble, causing the parent div click event to be triggered.

另外我还有个疑问就是，这次github ci未通过，看着像快照需要被更新，但是我此次修改的变动自认为可能会导致UI有细微的变化。 “ Checkbox中原先设置给.antd-checkbox的top: 0.2em样式被移除了，其父元素.antd-checkbox-wrapper设置transform: translateY(0.2em)” ，我觉得我的这个更改应该会导致UI变化。所以我不知道是应该执行 npm test -- -u 还是应该执行 npm run test-image -- -u