ckaczor/azuredatastudio
1
0
Fork 0
mirror of https://github.com/ckaczor/azuredatastudio.git synced 2026-01-27 01:25:36 -05:00
Code Issues Packages Projects Releases Wiki Activity

Minimum grid height set when grid returns 0 rows (#4716)

Browse Source
This commit is contained in:
Chris LaFreniere
2019-03-27 10:57:48 -07:00
committed by GitHub
parent e1e9c08242
commit eac3420583
2 changed files with 13 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
src/sql/parts/notebook/notebook.css
View File

@@ -85,4 +85,8 @@
.notebookEditor .notebook-cellTable .ui-widget-content.slick-row {
border-left: 1px silver dotted;
}
.notebookEditor .notebook-cellTable .slick-viewport {
min-height: 39px;
}

13
src/sql/parts/notebook/outputs/tableRenderers.ts
View File

@@ -60,11 +60,16 @@ export function renderDataResource(
});
detailTable.registerPlugin(rowNumberColumn);
let numRows = detailTable.grid.getDataLength();
// Need to include column headers and scrollbar, so that's why 1 needs to be added
let rowsHeight = (detailTable.grid.getDataLength() + 1) * ROW_HEIGHT + BOTTOM_PADDING_AND_SCROLLBAR;
// Set the height dynamically if the grid's height is < 500px high; otherwise, set height to 500px
tableContainer.style.height = rowsHeight >= 500 ? '500px' : rowsHeight.toString() + 'px';
let rowsHeight = (numRows + 1) * ROW_HEIGHT + BOTTOM_PADDING_AND_SCROLLBAR;
// if no rows are in the grid, set height to 100% of the container's height
if (numRows === 0) {
tableContainer.style.height = '100%';
} else {
// Set the height dynamically if the grid's height is < 500px high; otherwise, set height to 500px
tableContainer.style.height = rowsHeight >= 500 ? '500px' : rowsHeight.toString() + 'px';
}
attachTableStyler(detailTable, options.themeService);
host.appendChild(tableContainer);