How can I prevent click on drag?

SumNeuron picture SumNeuron · Dec 10, 2018 · Viewed 7.3k times · Source

I am trying to have an element with both a drag and click event. I have read about and tried a combination of event modifiers.

However, no matter what I try, I get a click when drag is stopped.

Note, in the MWE this is done on the component itself, but in my actual app I am using .native modifier to drag a component

How can I drag without click?

Edit Drag Click

Component Draggable:

<template>
  <div
    @pointerdown="handleDown"
    @pointerup="handleUp"
    @pointercancel="handleUp"
    @click="click = !click;"
    :style="style"
    ref="draggableRoot"
    class="draggable"
  >
    drag me!<br />
    am dragged? {{ drag }}<br />
    am clicked? {{ click }}<br />
  </div>
</template>
<script>
export default {
  computed: {
    style() {
      return {
        left: `${this.x}px`,
        top: `${this.y}px`
      };
    }
  },
  data() {
    return {
      x: 100,
      y: 100,
      left: 0,
      top: 0,
      drag: false,
      click: false
    };
  },
  methods: {
    handleMove({ pageX, pageY, clientX, clientY }) {
      if (this.$refs.draggableRoot) {
        this.x = pageX + this.left;
        this.y = pageY + this.top;
        this.drag = true;
      }
    },
    handleDown(event) {
      const { pageX, pageY } = event;
      const { left, top } = this.$refs.draggableRoot.getBoundingClientRect();
      this.left = left - pageX;
      this.top = top - pageY;
      document.addEventListener("pointermove", this.handleMove);
    },
    handleUp() {
      document.removeEventListener("pointermove", this.handleMove);
      this.drag = false;
    }
  }
};
</script>
<style scoped>
.draggable {
  position: fixed;
  border: solid coral 1px;
  height: 100px;
}
</style>

Answer

Krzysztof Atłasik picture Krzysztof Atłasik · Dec 10, 2018

Maybe that would work:

Add setTimeout inside method handleUp:

handleUp() { 
  document.removeEventListener("pointermove", this.handleMove);
  setTimeout(() => this.drag = false) //this would move this assigment at the end of event queue
}

Also add new method handleClick and assing it to event @click:

handleClick() {
  if(!this.drag) { //change click only if not draged
    this.click = !this.click
  }
}

Seems to work!