fix(b-form-file): ensure drag/drop handles required prop correctly (c…

…loses #3673)
bootstrap-vue · Jul 11, 2019 · d74ab11 · d74ab11
1 parent 988ac2f
commit d74ab11
Showing 1 changed file with 10 additions and 7 deletions.
diff --git a/src/components/form-file/form-file.js b/src/components/form-file/form-file.js
@@ -199,23 +199,26 @@ export const BFormFile = /*#__PURE__*/ Vue.extend({
       this.selectedFile = this.multiple ? [] : null
     },
     onDragover(evt) /* istanbul ignore next: difficult to test in JSDOM */ {
-      evt.preventDefault()
-      evt.stopPropagation()
-      if (this.noDrop || !this.custom) {
+      if (this.noDrop) {
+        evt.preventDefault()
+        evt.stopPropagation()
         return
       }
       this.dragging = true
       evt.dataTransfer.dropEffect = 'copy'
     },
     onDragleave(evt) /* istanbul ignore next: difficult to test in JSDOM */ {
-      evt.preventDefault()
-      evt.stopPropagation()
+      if (this.noDrop) {
+        evt.preventDefault()
+        evt.stopPropagation()
+        return
+      }
       this.dragging = false
     },
     onDrop(evt) /* istanbul ignore next: difficult to test in JSDOM */ {
-      evt.preventDefault()
-      evt.stopPropagation()
       if (this.noDrop) {
+        evt.preventDefault()
+        evt.stopPropagation()
         return
       }
       this.dragging = false