removed droppable attribute

BonnierNews · Feb 14, 2023 · d83f0a9 · d83f0a9
1 parent 79746df
commit d83f0a9
Show file tree

Hide file tree

Showing 3 changed files with 1 addition and 24 deletions.
diff --git a/app/assets/public/dnd.html b/app/assets/public/dnd.html
@@ -10,7 +10,7 @@
   </head>
   <body>
     <div draggable="true">draggable</div>
-    <div droppable="false">droppable</div>
+    <div>droppable</div>
     <script>
       (function IIFE() {
         window.scriptsAreExecutedInBody = true;

diff --git a/lib/HTMLElement.js b/lib/HTMLElement.js
@@ -38,13 +38,4 @@ export default class HTMLElement extends Element {
     this.getAttribute("draggable", !!value);
     return value;
   }
-
-  get droppable() {
-    return !!this.getAttribute("droppable");
-  }
-
-  set droppable(value) {
-    this.getAttribute("droppable", !!value);
-    return value;
-  }
 }
diff --git a/test/drag-and-drop-test.js b/test/drag-and-drop-test.js
@@ -21,20 +21,6 @@ describe("drag and drop", () => {
     expect(divs[1].draggable = true).to.equal(true);
   });
 
-  it("can read droppable property", () => {
-    const divs = browser.document.getElementsByTagName("div");
-
-    expect(divs[0].droppable).to.equal(false);
-    expect(divs[1].droppable).to.equal(true);
-  });
-
-  it("can set droppable property", () => {
-    const divs = browser.document.getElementsByTagName("div");
-
-    expect(divs[0].droppable = true).to.equal(true);
-    expect(divs[1].droppable = false).to.equal(false);
-  });
-
   describe("events", () => {
     it("dispatches start event when dragging", (done) => {
       const [ draggable ] = browser.document.getElementsByTagName("div");