@uppy/drop-target - npm Package Compare versions

Comparing version 2.0.2 to 2.0.3

CHANGELOG.md

 # @uppy/drop-target
 
+## 2.0.3
+
+Released: 2024-02-19
+Included in: Uppy v3.22.0
+
+- @uppy/drop-target: refactor to typescript (artur paikin / #4863)
+
 ## 2.0.1
@@ -4,0 +11,0 @@

lib/index.js

 import BasePlugin from '@uppy/core/lib/BasePlugin.js';
 import getDroppedFiles from '@uppy/utils/lib/getDroppedFiles';
 import toArray from '@uppy/utils/lib/toArray';
+// eslint-disable-next-line @typescript-eslint/ban-ts-comment
+// @ts-ignore We don't want TS to generate types for the package.json
 const packageJson = {
-  "version": "2.0.2"
+  "version": "2.0.3"
 };
+// Default options
+const defaultOpts = {
+  target: null
+};
 function isFileTransfer(event) {
-  var _event$dataTransfer$t, _event$dataTransfer$t2;
-  return (_event$dataTransfer$t = (_event$dataTransfer$t2 = event.dataTransfer.types) == null ? void 0 : _event$dataTransfer$t2.some(type => type === 'Files')) != null ? _event$dataTransfer$t : false;
+  var _event$dataTransfer$t, _event$dataTransfer;
+  return (_event$dataTransfer$t = (_event$dataTransfer = event.dataTransfer) == null || (_event$dataTransfer = _event$dataTransfer.types) == null ? void 0 : _event$dataTransfer.some(type => type === 'Files')) != null ? _event$dataTransfer$t : false;
 }
@@ -18,3 +24,6 @@
   constructor(uppy, opts) {
-    super(uppy, opts);
+    super(uppy, {
+      ...defaultOpts,
+      ...opts
+    });
     this.addFiles = files => {
@@ -39,3 +48,3 @@ const descriptors = files.map(file => ({
     this.handleDrop = async event => {
-      var _this$opts$onDrop, _this$opts;
+      var _event$currentTarget, _this$opts$onDrop, _this$opts;
       if (!isFileTransfer(event)) {
@@ -46,6 +55,7 @@ return;
       event.stopPropagation();
-      clearTimeout(this.removeDragOverClassTimeout);
+      clearTimeout(this.removeDragOverClassTimeout)
 
       // Remove dragover class
-      event.currentTarget.classList.remove('uppy-is-drag-over');
+      ;
+      (_event$currentTarget = event.currentTarget) == null || _event$currentTarget.classList.remove('uppy-is-drag-over');
       this.setPluginState({
@@ -58,4 +68,4 @@ isDraggingOver: false
         if (plugin.type === 'acquirer') {
-          // Every Plugin with .type acquirer can define handleRootDrop(event)
-          plugin.handleRootDrop == null ? void 0 : plugin.handleRootDrop(event);
+          // @ts-expect-error Every Plugin with .type acquirer can define handleRootDrop(event)
+          plugin.handleRootDrop == null || plugin.handleRootDrop(event);
         }
@@ -83,3 +93,3 @@ });
       }
-      (_this$opts$onDrop = (_this$opts = this.opts).onDrop) == null ? void 0 : _this$opts$onDrop.call(_this$opts, event);
+      (_this$opts$onDrop = (_this$opts = this.opts).onDrop) == null || _this$opts$onDrop.call(_this$opts, event);
     };
@@ -104,3 +114,3 @@ this.handleDragOver = event => {
       });
-      (_this$opts$onDragOver = (_this$opts2 = this.opts).onDragOver) == null ? void 0 : _this$opts$onDragOver.call(_this$opts2, event);
+      (_this$opts$onDragOver = (_this$opts2 = this.opts).onDragOver) == null || _this$opts$onDragOver.call(_this$opts2, event);
     };
@@ -121,2 +131,3 @@ this.handleDragLeave = event => {
       this.removeDragOverClassTimeout = setTimeout(() => {
+        ;
         currentTarget.classList.remove('uppy-is-drag-over');
@@ -127,3 +138,3 @@ this.setPluginState({
       }, 50);
-      (_this$opts$onDragLeav = (_this$opts3 = this.opts).onDragLeave) == null ? void 0 : _this$opts$onDragLeav.call(_this$opts3, event);
+      (_this$opts$onDragLeav = (_this$opts3 = this.opts).onDragLeave) == null || _this$opts$onDragLeav.call(_this$opts3, event);
     };
@@ -139,3 +150,3 @@ this.addListeners = () => {
       }
-      if (!this.nodes && !this.nodes.length > 0) {
+      if (!this.nodes || this.nodes.length === 0) {
         throw new Error(`"${target}" does not match any HTML elements`);
@@ -160,15 +171,4 @@ }
     this.id = this.opts.id || 'DropTarget';
+    // @ts-expect-error TODO: remove in major
     this.title = 'Drop Target';
-
-    // Default options
-    const defaultOpts = {
-      target: null
-    };
-
-    // Merge default options with the ones set by user
-    this.opts = {
-      ...defaultOpts,
-      ...opts
-    };
-    this.removeDragOverClassTimeout = null;
   }
@@ -175,0 +175,0 @@ install() {

package.json

 {
   "name": "@uppy/drop-target",
   "description": "Lets your users drag and drop files on a DOM element",
-  "version": "2.0.2",
+  "version": "2.0.3",
   "license": "MIT",
@@ -28,6 +28,6 @@ "main": "lib/index.js",
   "dependencies": {
-    "@uppy/utils": "^5.5.2"
+    "@uppy/utils": "^5.7.1"
   },
   "peerDependencies": {
-    "@uppy/core": "^3.6.0"
+    "@uppy/core": "^3.9.0"
   },
@@ -34,0 +34,0 @@ "publishConfig": {

New alerts

Major refactor

Supply chain risk

Package has recently undergone a major refactor. It may be unstable or indicate significant internal changes. Use caution when updating to versions that include significant changes.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

27593

increased by10.14%

Lines of code

361

increased by6.8%

Worsened metrics

Number of medium supply chain risk alerts

1

increased byInfinity%

Dependency changes

• Updated@uppy/utils@^5.7.1