一个因@click.stop引发的bug的解决

Tani ·

更新时间:2024-11-14

· 793 次阅读

问题

在项目页面中使用 element popover，设置trigger='click'时点击外部不会触发自动隐藏，但在 element 官网中是可以正常触发的（官方示例），项目中的菜单是自定义写的，所以怀疑是有黑魔法。

查找原因

将 popover 写在app.vue根组件内，发现可以正常触发自动隐藏。在app.vue的 mounted 钩子中加入window.addEventListener('click', () => console.log('window click===>>>>'))，发现只有菜单栏外层能够触发。检查菜单栏组件，发现代码中<div class="main" @click.stop="isShowWhole = false">，这里的 click 事件使用了 stop 修饰符（阻止冒泡），可能阻止了 popover 外部点击的事件判断，尝试将 stop 修饰符去掉，发现外部点击事件正常触发。

确认代码修改没有副作用

在修复 bug 时，需要注意不会产生额外的 bug，那就需要了解修改的这段代码的含义


@click.stop="isShowWhole = false"

从代码上看，点击 class 为 main 的 div 将会触发左边侧边栏缩略显示，加上 stop 修饰符是为了防止事件冒泡，所以能否去掉 stop 需要确认是否有这个必要。


// router.js
let routes = [
  {
   path: '/',
   alias: '/admin',
   component: Menu,
   children: [...Pages],
  },
  {
   path: '*',
   name: '404',
   component: NotFound,
  },
 ];

在路由中可以看到，Menu 是作为根路由进行渲染，除了 404 页面都是它的子路由，所以 stop 修饰符是没有必要加上的，去除后经过测试没有其他影响。

深入 element popover 源码分析原因

对 element 组件进行 debug 时，可以直接引入相关组件的源码


import ElPopover from 'element-ui/packages/popover';
export default {
  components: {
    CheckboxFilter,
    ElPopover
  },
  ...
}

然后我们就可以在node_modules的 element 源码进行 debug 操作（危险步骤，debug 后需要复原）。


// node_modules/element-ui/packages/popover/src/main.vue
mounted() {
  ...
  if (this.trigger === 'click') {
   on(reference, 'click', this.doToggle);
   on(document, 'click', this.handleDocumentClick);
  } else if (this.trigger === 'hover') {
   ...
  } else if (this.trigger === 'focus') {
   ...
  }
}

popover 在 mounted 钩子内初始化了trigger='click'的事件绑定，on(document, 'click', this.handleDocumentClick)这里绑定了 document 很可能就是阻止事件冒泡后不能触发外部点击隐藏的判断逻辑。


// node_modules/element-ui/packages/popover/src/main.vue
handleDocumentClick(e) {
 let reference = this.reference || this.$refs.reference;
 const popper = this.popper || this.$refs.popper;
 if (!reference && this.$slots.reference && this.$slots.reference[0]) {
  reference = this.referenceElm = this.$slots.reference[0].elm;
 }
 if (!this.$el ||
  !reference ||
  this.$el.contains(e.target) ||
  reference.contains(e.target) ||
  !popper ||
  popper.contains(e.target)) return;
 this.showPopper = false;
},

这里判断this.$el是否包含 click 的 target，从而是否触发this.showPopper = false，当菜单栏阻止事件冒泡后 document 不能监听到 click 事件，才会无法进行外部点击隐藏的判断逻辑。

延伸v-clickoutside

element 的 select 组件中用到了 v-clickoutside 自定义指令，作用和 popover 的 handleDocumentClick 差不多（倒不如说 handleDocumentClick 是特殊的 clickoutside）

在上面的问题中，我们单独把 v-clickoutside 抽出来使用确实可以的，这是为什么呢？


// node_modules/element-ui/packages/popover/src/utils/clickoutside.js
!Vue.prototype.$isServer && on(document, 'mousedown', e => (startClick = e));
!Vue.prototype.$isServer && on(document, 'mouseup', e => {
 nodeList.forEach(node => node[ctx].documentHandler(e, startClick));
});

答案是 v-clickoutside 使用鼠标事件判断的，所以 click 的阻止冒泡不会让 clickoutside 无效。

总结

解决 bug 的过程中需要做到不产生额外的 bug，并且深入分析问题的原因有助于能力的提高。

stop click

1024 个赞