fix: do not change property

os-js · Jan 26, 2024 · b392800 · b392800
1 parent 26614e9
commit b392800
Showing 1 changed file with 3 additions and 9 deletions.
diff --git a/src/window.js b/src/window.js
@@ -454,16 +454,10 @@ export default class Window extends EventEmitter {
 
     // DnD functionality
     if (this.attributes.droppable) {
-      /*
-        `this.attributes.droppable` should probably always be an object
-        and never be set to `true`.
-        The following code is only to update the setting for legacy applications.
-      */
-      if (this.attributes.droppable === true) {
-        this.attributes.droppable = {};
-      }
+      const {dataTransferProperty = 'files'} = this.attributes.droppable === true
+        ? {}
+        : this.attributes.droppable;
 
-      const {dataTransferProperty = 'files'} = this.attributes.droppable;
       const d = droppable(this.$element, {
         ondragenter: (...args) => this.emit('dragenter', ...args, this),
         ondragover: (...args) => this.emit('dragover', ...args, this),