fix: set dimensions and z-index on ghost element

mattlewis92 · Jun 6, 2018 · 9b5a6b0 · 9b5a6b0
1 parent 6e0a1a3
commit 9b5a6b0
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 2 deletions.
diff --git a/src/draggable.directive.ts b/src/draggable.directive.ts
@@ -228,15 +228,21 @@ export class DraggableDirective implements OnInit, OnChanges, OnDestroy {
               this.setElementStyles({
                 position: 'fixed',
                 top: `${rect.top}px`,
-                left: `${rect.left}px`
+                left: `${rect.left}px`,
+                width: `${rect.width}px`,
+                height: `${rect.height}px`,
+                zIndex: '10'
               });
 
               dragEnd$.subscribe(() => {
                 clone.parentElement!.removeChild(clone);
                 this.setElementStyles({
                   position: '',
                   top: '',
-                  left: ''
+                  left: '',
+                  width: '',
+                  height: '',
+                  zIndex: ''
                 });
               });
             }

diff --git a/test/draggable.directive.spec.ts b/test/draggable.directive.spec.ts
@@ -578,6 +578,9 @@ describe('draggable directive', () => {
     expect(draggableElement.style.position).to.equal('fixed');
     expect(draggableElement.style.top).to.be.ok;
     expect(draggableElement.style.left).to.be.ok;
+    expect(draggableElement.style.width).to.be.ok;
+    expect(draggableElement.style.height).to.be.ok;
+    expect(draggableElement.style.zIndex).to.equal('10');
     expect(draggableElement.previousSibling).to.be.ok;
     expect(
       (draggableElement.previousElementSibling as HTMLElement).style.visibility
@@ -591,6 +594,9 @@ describe('draggable directive', () => {
     expect(draggableElement.style.position).not.to.be.ok;
     expect(draggableElement.style.top).not.to.be.ok;
     expect(draggableElement.style.left).not.to.be.ok;
+    expect(draggableElement.style.width).not.to.be.ok;
+    expect(draggableElement.style.height).not.to.be.ok;
+    expect(draggableElement.style.zIndex).not.to.be.ok;
     expect(
       (draggableElement.previousElementSibling as HTMLElement).hasAttribute(
         'mwldraggable'