@uppy/drop-target: refactor to ESM (#3648)

.eslintrc.js

@@ -201,6 +201,7 @@ module.exports = {
         'packages/@uppy/audio/src/**/*.js',
         'packages/@uppy/box/src/**/*.js',
         'packages/@uppy/drag-drop/src/**/*.js',
+        'packages/@uppy/drop-target/src/**/*.js',
         'packages/@uppy/compressor/src/**/*.js',
         'packages/@uppy/vue/src/**/*.js',
       ],

packages/@uppy/drop-target/package.json

@@ -4,6 +4,7 @@
   "version": "1.1.2",
   "license": "MIT",
   "main": "lib/index.js",
+  "type": "module",
   "types": "types/index.d.ts",
   "keywords": [
     "file uploader",

packages/@uppy/drop-target/src/index.js

@@ -1,13 +1,19 @@
-const BasePlugin = require('@uppy/core/lib/BasePlugin')
-const getDroppedFiles = require('@uppy/utils/lib/getDroppedFiles')
-const toArray = require('@uppy/utils/lib/toArray')
+import BasePlugin from '@uppy/core/lib/BasePlugin'
+import getDroppedFiles from '@uppy/utils/lib/getDroppedFiles'
+import toArray from '@uppy/utils/lib/toArray'
+
+import packageJson from '../package.json'
+
+function isFileTransfer (event) {
+  return event.dataTransfer.types?.some((type) => type === 'Files') ?? false
+}
 
 /**
  * Drop Target plugin
  *
  */
-module.exports = class DropTarget extends BasePlugin {
-  static VERSION = require('../package.json').version
+export default class DropTarget extends BasePlugin {
+  static VERSION = packageJson.version
 
   constructor (uppy, opts) {
     super(uppy, opts)
@@ -45,10 +51,7 @@ module.exports = class DropTarget extends BasePlugin {
     }
   }
 
-  isFileTransfer = (event) => {
-    const transferTypes = event.dataTransfer.types ?? []
-    return transferTypes.some((type) => type === 'Files')
-  }
+  isFileTransfer = isFileTransfer // TODO: expose this as a static instead (or don't expose it at all) in the next major.
 
   handleDrop = async (event) => {
     if (!this.isFileTransfer(event)) {
@@ -104,7 +107,7 @@ module.exports = class DropTarget extends BasePlugin {
     // Add a small (+) icon on drop
     // (and prevent browsers from interpreting this as files being _moved_ into the browser,
     // https://github.com/transloadit/uppy/issues/1978)
-    event.dataTransfer.dropEffect = 'copy'
+    event.dataTransfer.dropEffect = 'copy' // eslint-disable-line no-param-reassign
 
     clearTimeout(this.removeDragOverClassTimeout)
     event.currentTarget.classList.add('uppy-is-drag-over')