1.1. 问题
在项目页面中使用 popover,设置trigger='click'
时点击外部不会触发自动隐藏,但在 element 官网中是可以正常触发的(官方示例),项目中的Popover上一层还是Popover。
1.2. 查找原因
- 将 popover 写在
app.vue
根组件内,发现可以正常触发自动隐藏。 - 在
app.vue
的 mounted 钩子中加入window.addEventListener('click', () => console.log('window click===>>>>'))
,发现只有上一层Popover外层能够触发。 - 检查Popover组件所有click事件,发现Popover所引用的Vue-popover.js中有一个click事件,这里的 click 事件使用了 stop(阻止冒泡),阻止了 popover 内部点击的事件判断,尝试将 stop 修饰符去掉,发现外部点击事件正常触发。
1.3. 确认代码修改没有副作用
在修复 bug 时,需要注意不会产生额外的 bug,那就需要了解修改的这段代码的含义
var stop = function stop(e) {
return e.stopPropagation();
};this.popperElm.addEventListener('click', stop);
从代码上看,点击 popoper元素本身将会阻止click事件的冒泡,所以导致当前的popover组件并不可以监听到click事件的触发,所以无法正常触发关闭事件.
on(document, 'click', this.handleDocumentClick);
handleDocumentClick(e) {
let reference = this.reference || this.$refs.reference;
const popper = this.popper || this.$refs.popper;
if (!reference && this.$refs.wrapper.children) {
reference = this.referenceElm = this.$refs.wrapper.children[0];
}
if (!this.$el ||
!reference ||
this.$el.contains(e.target) ||
reference.contains(e.target) ||
!popper ||
popper.contains(e.target)) return;
this.showPopper = false;
},
1.4. 深入 element popover 源码分析原因
对 element 组件进行 debug 时,可以直接引入相关组件的源码
然后我们就可以在node_modules
的 element 源码进行 debug 操作( 危险步骤 ,debug 后需要复原)。
popover 在 mounted 钩子内初始化了trigger='click'
的事件绑定,on(document, 'click', this.handleDocumentClick)
这里绑定了 document 很可能就是阻止事件冒泡后不能触发外部点击隐藏的判断逻辑。
这里判断this.$el
是否包含 click 的 target,从而是否触发this.showPopper = false
,当菜单栏阻止事件冒泡后 document 不能监听到 click 事件,才会无法进行外部点击隐藏的判断逻辑。
1.5. 延伸v-clickoutside
element 的 select 组件中用到了 v-clickoutside 自定义指令,作用和 popover 的 handleDocumentClick 差不多(倒不如说 handleDocumentClick 是特殊的 clickoutside)
在上面的问题中,我们单独把 v-clickoutside 抽出来使用确实可以的,这是为什么呢?
答案是 v-clickoutside 使用鼠标事件判断的,所以 click 的 阻止冒泡不会让 clickoutside 无效。
1.6. 总结
解决 bug 的过程中需要做到不产生额外的 bug,并且深入分析问题的原因有助于能力的提高。