Fix DND handle size on markdown cells

roblourens · roblourens · commit d109558e3d57 · 2020-06-26T11:13:57.000-05:00
diff --git a/src/vs/workbench/contrib/notebook/browser/media/notebook.css b/src/vs/workbench/contrib/notebook/browser/media/notebook.css
@@ -396,6 +396,12 @@
 	opacity: 1;
 }
 
+.monaco-workbench .notebookOverlay .monaco-list .monaco-list-row .cell-focus-indicator-side {
+	/** Overidden for code cells */
+	top: 0px;
+	bottom: 0px;
+}
+
 .monaco-workbench .notebookOverlay .monaco-list .monaco-list-row .cell-focus-indicator-top,
 .monaco-workbench .notebookOverlay .monaco-list .monaco-list-row .cell-focus-indicator-bottom {
 	width: 100%;
diff --git a/src/vs/workbench/contrib/notebook/browser/view/renderers/cellRenderer.ts b/src/vs/workbench/contrib/notebook/browser/view/renderers/cellRenderer.ts
@@ -268,19 +268,17 @@ abstract class AbstractCellRenderer {
 				this.notebookEditor.focus();
 			}
 
-			if (templateData.focusIndicator) {
-				if (actions.primary.length || actions.secondary.length) {
-					templateData.container.classList.add('cell-has-toolbar-actions');
+			if (actions.primary.length || actions.secondary.length) {
+				templateData.container.classList.add('cell-has-toolbar-actions');
+				if (isCodeCellRenderTemplate(templateData)) {
 					templateData.focusIndicator.style.top = `${EDITOR_TOOLBAR_HEIGHT + EDITOR_TOP_MARGIN}px`;
-					if (isCodeCellRenderTemplate(templateData)) {
-						templateData.focusIndicatorRight.style.top = `${EDITOR_TOOLBAR_HEIGHT + EDITOR_TOP_MARGIN}px`;
-					}
-				} else {
-					templateData.container.classList.remove('cell-has-toolbar-actions');
+					templateData.focusIndicatorRight.style.top = `${EDITOR_TOOLBAR_HEIGHT + EDITOR_TOP_MARGIN}px`;
+				}
+			} else {
+				templateData.container.classList.remove('cell-has-toolbar-actions');
+				if (isCodeCellRenderTemplate(templateData)) {
 					templateData.focusIndicator.style.top = `${EDITOR_TOP_MARGIN}px`;
-					if (isCodeCellRenderTemplate(templateData)) {
-						templateData.focusIndicatorRight.style.top = `${EDITOR_TOP_MARGIN}px`;
-					}
+					templateData.focusIndicatorRight.style.top = `${EDITOR_TOP_MARGIN}px`;
 				}
 			}
 		};
