From 9f7d96bad31e4e199d18014d038110073ebe301c Mon Sep 17 00:00:00 2001 From: Gene Lee Date: Mon, 3 Jun 2019 16:47:07 -0700 Subject: [PATCH] Fixed bug: Notebooks Vertical Scrollbar is Unnecessary for Some Grid Outputs (#5847) --- src/sql/workbench/parts/notebook/outputs/tableRenderers.ts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/sql/workbench/parts/notebook/outputs/tableRenderers.ts b/src/sql/workbench/parts/notebook/outputs/tableRenderers.ts index cf8d1a2589..e19d463f23 100644 --- a/src/sql/workbench/parts/notebook/outputs/tableRenderers.ts +++ b/src/sql/workbench/parts/notebook/outputs/tableRenderers.ts @@ -67,7 +67,7 @@ export function renderDataResource( detailTable.registerPlugin(new AdditionalKeyBindings()); let numRows = detailTable.grid.getDataLength(); // Need to include column headers and scrollbar, so that's why 1 needs to be added - let rowsHeight = (numRows + 1) * RESULTS_GRID_DEFAULTS.rowHeight + BOTTOM_PADDING_AND_SCROLLBAR; + let rowsHeight = (numRows + 1) * RESULTS_GRID_DEFAULTS.rowHeight + BOTTOM_PADDING_AND_SCROLLBAR + numRows; // if no rows are in the grid, set height to 100% of the container's height if (numRows === 0) { tableContainer.style.height = '100%';