Skip to content

Commit

Permalink
fix(core): resizeCanvas should include preheader height, fixes #493 (
Browse files Browse the repository at this point in the history
…#497)

* fix(core): `resizeCanvas` should include preheader height, fixes #493
- ref #493

* refactor(core): remove unnecessary preheader height from calculation
  • Loading branch information
ghiscoding authored May 17, 2020
1 parent 03e794a commit af6151f
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion slick.grid.js
Original file line number Diff line number Diff line change
Expand Up @@ -3452,6 +3452,7 @@ if (typeof Slick === "undefined") {
paneBottomH = 0;
viewportTopH = 0;
viewportBottomH = 0;
var preHeaderHeight = options.showPreHeaderPanel ? options.preHeaderPanelHeight : 0;

getViewportWidth();
getViewportHeight();
Expand All @@ -3470,7 +3471,7 @@ if (typeof Slick === "undefined") {
}

// The top pane includes the top panel and the header row
paneTopH += topPanelH + headerRowH + footerRowH;
paneTopH += topPanelH + preHeaderHeight + headerRowH + footerRowH;

if (hasFrozenColumns() && options.autoHeight) {
paneTopH += scrollbarDimensions.height;
Expand Down

0 comments on commit af6151f

Please sign in to comment.