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

Table: table add highlight selection row #22382

Merged
merged 2 commits into from
Aug 14, 2023

Conversation

wangdaodao
Copy link
Contributor

Please make sure these boxes are checked before submitting your PR, thank you!

对应issues:#22342

对应demo:https://ui.const.team/#/component/table#duo-xuan

  • Make sure you follow Element's contributing guide (中文 | English | Español | Français).
  • Make sure you are merging your commits to dev branch.
  • Add some descriptions and refer relative issues for you PR.

@xinguanhua
Copy link
Collaborator

这种场景是否可以取一个独立的类名,而不是复用 current-row ,毕竟 selected-row 和 current-row 是不一样的

@wangdaodao
Copy link
Contributor Author

好的,我去改一下类名吧,确实这样有点冲突。

@wangdaodao
Copy link
Contributor Author

@xinguanhua 类名修改为selection-row

@xinguanhua xinguanhua merged commit 041227d into ElemeFE:dev Aug 14, 2023
@me1129
Copy link

me1129 commented Oct 10, 2023

表格嵌套表格, 某一列是fixed定位时,会出现类似问题, 鼠标滑入 定位不准确, hover-row 设置行也不准确

@me1129
Copy link

me1129 commented Oct 10, 2023

表格嵌套表格, 某一列是fixed定位时,会出现类似问题, 鼠标滑入 定位不准确, hover-row 设置行也不准确,鼠标放在第五行,hover-row却设置第三行
微信图片_20231010164526

@wangdaodao
Copy link
Contributor Author

表格嵌套表格, 某一列是fixed定位时,会出现类似问题, 鼠标滑入 定位不准确, hover-row 设置行也不准确

不太明白你表达的意思,这个功能是在表格复选的时候给复选的行增加高亮选中的样式,而你说的是划过,这两个不是一个功能。如果有代码,可以放出来看一下。

@me1129
Copy link

me1129 commented Oct 13, 2023

1, 先二次封装表格组件,然后增加了个插槽
image
2.在这个组件 插槽使用中,嵌套了popover 里面嵌套了新的表格组件
image
3.然后点击打开嵌套表格,鼠标再滑入上一个表格就会出现两个hover效果
image
4,分析原因,应该是无法识别两个表格,我用key设置还是无效,是否是element 缺陷呢

@wangdaodao
Copy link
Contributor Author

1, 先二次封装表格组件,然后增加了个插槽
image
2.在这个组件 插槽使用中,嵌套了popover 里面嵌套了新的表格组件
image
3.然后点击打开嵌套表格,鼠标再滑入上一个表格就会出现两个hover效果
image
4,分析原因,应该是无法识别两个表格,我用key设置还是无效,是否是element 缺陷呢

这个问题应该和这个PR没有关系,我建议你不要二次封装,直接使用原始表格复现你的问题,这样起码可以排除是否是二次封装的问题。如果原始表格还有问题,你可以新建一个bug反馈。

@me1129
Copy link

me1129 commented Oct 16, 2023

刚才用原始表格测试了下还是有问题的

@guohuihot
Copy link

guohuihot commented Jan 25, 2024

别乱加东西,越加越重了,现在都不敢用了

@wangdaodao
Copy link
Contributor Author

别乱加东西,越加越重了,现在都不敢用了

  1. 如果代码有问题,欢迎指正,而不是发表没有意义的评论。
  2. 这不是乱加功能,这些功能你用不到,不能代表别人用不到。
  3. 各个开源项目都在维护功能,如果加了东西不敢用,那就从一开始别用开源的,自己把控功能更好。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants