search

ayiaq1 / el-tree-select

基于element-ui2.x扩展下拉树
224 stars 93 forks source link

有时会报 e.className.indexOf is not a function #28

Closed samyhd closed 4 years ago

samyhd commented 5 years ago

如下图： image

我看代码处如下： image

望完善，谢谢分享。

ayiaq1 commented 5 years ago

贴一下你的完整代码呢 你修改了class名称?

samyhd commented 5 years ago

我没有改，很简单的测试应用，就下面两块地方涉及到

image

image

偶尔发生的，我也比较难重现

ayiaq1 commented 5 years ago

这个组件在至少20个项目里面使用，没有出现过这个问题，最好你给在线地址或者追踪报错原因

samyhd commented 5 years ago

你那边是否可以加上保护，在className明确是一个字符时才调用 indexOf ？

ayiaq1 commented 5 years ago

既然有报错必定有原因，否则功能会有问题

ayiaq1 commented 5 years ago

追踪没有结果，close

xdxiaodong commented 4 years ago

这个问题确实存在，我也遇到了相同的问题，<el-popover>里的<svg>触发click事件或者直接点击，就会报错

ayiaq1 commented 4 years ago

https://ayiaq1.github.io/el-tree-select/storybook-static/?path=/story/%E6%B8%B2%E6%9F%93--%E9%80%9A%E8%BF%87props%E4%BC%A0%E5%8F%82%E6%95%B0 在线测试下 点击箭头和图标 没有触发该问题，贴一下代码，以及使用得版本号

xdxiaodong commented 4 years ago

版本3.1.4，这个问题是我在做图标选择组件时发现的，涉及到的代码比较多，在出现问题的时候我把el-tree-select屏蔽掉问题即消失，这个问题应该是其他组件触发了el-tree-select的_popoverHideFun而_popoverHideFun内判断不严谨导致的报错，<el-tree-select />任何参数不设置也会100%报错

ayiaq1 commented 4 years ago

晚一点加一个是否存在的判断，但是因为没有具体得代码无法直接追踪。

xdxiaodong commented 4 years ago

Snipaste_2019-09-27_17-35-00

ayiaq1 commented 4 years ago

3.1.5增加了兼容性判断，最好提供一下模板代码便于调试

xdxiaodong commented 4 years ago

我刚才升级到3.1.5问题还是存在，我写个测试demo

xdxiaodong commented 4 years ago

测试demo写好了，地址：https://github.com/xdxiaodong/el-tree-select-demo-bug.git

ayiaq1 commented 4 years ago

非常感谢，3.1.6 已解决

xdxiaodong commented 4 years ago

3.1.6 chrome控制台报错的问题已经解决，firefox和edge报错的问题依然存在，他俩的问题跟chrome的报错问题差不多，用我之前发的demo可以测试。报错的触发方法跟chrome一样。
edge
0: Unable to get property 'some' of undefined or null reference el-tree-select.umd.min.js (1,1) firefox 

TypeError: t.path is undefined
_popoverHideFun el-tree-select.umd.min.js:1
_popoverHideFun self-hosted:1003
ayiaq1 commented 4 years ago

3.1.7 处理MouseEvent多浏览器兼容性判断