Skip to content

Commit

Permalink
fix column resizing of grid having vertical scrollbar
Browse files Browse the repository at this point in the history
  • Loading branch information
OlegKi committed Apr 1, 2015
1 parent 41f3706 commit 5108e53
Show file tree
Hide file tree
Showing 4 changed files with 349 additions and 349 deletions.
2 changes: 1 addition & 1 deletion js/grid.base.js
Original file line number Diff line number Diff line change
Expand Up @@ -5145,7 +5145,7 @@ jgrid.extend({
if(p.footerrow) {
footers[lvc].style.width = cw+"px";
}
if (p.tblwidth < p.width) {
if (p.tblwidth + (hs ? scw: 0) < p.width) { // prabably bDiv.offsetWidth - bDiv.clientWidth is better as scw
// decrease the width if required
setWidthOfAllDivs(p.tblwidth);
}
Expand Down

3 comments on commit 5108e53

@jimmccarthy
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice, I just ran into this today. Thanks for the fix.

@OlegKi
Copy link
Member Author

@OlegKi OlegKi commented on 5108e53 Apr 1, 2015

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You are welcome! It's really difficult to test every combination of input options. So some problems will be fixed after the bug report. I made the fix after reading the question. So if you would find more problems, then just post the issue here (it's better to include the demo which reproduce the problem) or write the question on the stackoverflow with the tags "jqgrid" and "free-jqgrid". Typically I create the fix very quickly.

Best regards
Oleg

@jimmccarthy
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks Oleg, I just grabbed the latest and it is working great.

Please sign in to comment.