Fix microsoft#99920

roblourens · roblourens · commit 026968b0cbea · 2020-06-11T20:55:26.000-05:00
diff --git a/src/vs/workbench/contrib/notebook/browser/view/renderers/cellRenderer.ts b/src/vs/workbench/contrib/notebook/browser/view/renderers/cellRenderer.ts
@@ -48,7 +48,6 @@ import { CodeCellViewModel } from 'vs/workbench/contrib/notebook/browser/viewMod
 import { MarkdownCellViewModel } from 'vs/workbench/contrib/notebook/browser/viewModel/markdownCellViewModel';
 import { CellViewModel } from 'vs/workbench/contrib/notebook/browser/viewModel/notebookViewModel';
 import { CellKind, NotebookCellRunState, NotebookCellMetadata } from 'vs/workbench/contrib/notebook/common/notebookCommon';
-import { containsDragType } from 'vs/workbench/browser/dnd';
 import { CellContextKeyManager } from 'vs/workbench/contrib/notebook/browser/view/renderers/cellContextKeys';
 
 const $ = DOM.$;
@@ -463,8 +462,6 @@ interface CellDragEvent {
 }
 
 export class CellDragAndDropController extends Disposable {
-	private static DATA_TYPE = 'vscode_notebook_cell';
-
 	// TODO@roblourens - should probably use dataTransfer here, but any dataTransfer set makes the editor think I am dropping a file, need
 	// to figure out how to prevent that
 	private currentDraggedCell: ICellViewModel | undefined;
@@ -575,7 +572,7 @@ export class CellDragAndDropController extends Disposable {
 			return;
 		}
 
-		if (!this.currentDraggedCell || !containsDragType(event.browserEvent, CellDragAndDropController.DATA_TYPE)) {
+		if (!this.currentDraggedCell) {
 			event.browserEvent.dataTransfer.dropEffect = 'none';
 			return;
 		}
@@ -652,8 +649,6 @@ export class CellDragAndDropController extends Disposable {
 				return;
 			}
 
-			event.dataTransfer.setData(CellDragAndDropController.DATA_TYPE, 'true');
-
 			this.currentDraggedCell = templateData.currentRenderedCell!;
 			this.currentDraggedCell.dragging = true;
 
