ddomingues / X-SlickGrid

An extension of SlickGrid providing Frozen Columns and Rows, Tree Columns, etc.
http://ddomingues.com/X-SlickGrid/
MIT License
48 stars 19 forks source link

Scrollbar issues with FrozenRows #9

Open mohoch1 opened 9 years ago

mohoch1 commented 9 years ago

Hi. First of all, we would like to thank you for the great improvements you added to Slickgird.

We have an issue with scrollbars when setting a frozen row. It seems that the vertical scrollbar is always treated as presented, as its width is deducted from the total width, even if there is no need for scrollbars at all.

This can be viewed in your frozen row demo , reduce the total number of rows to 12, which fills up exactly all the height of the grid, and you will see that the vertical scrollbar's track still appears.

image

As we saw in the slick.grid.js source code, the updateCanvasWidth function will always deduct the width of the scrollbar whenever a frozen row is present (line 696 and 761).

In our use case, the scrollbar should not be displayed when all rows are presented. How can this be resolved?

Thanks!