Skip to content

Commit

Permalink
Workaround for issue #294. I think the real problem is that _Scroller…
Browse files Browse the repository at this point in the history
… updates the wrong pages in its resize() function. Maybe it should even call needPage() for all pages when the grid's autoHeight property is set. But I don't understand the scroller code good enough to fix this. This workaround just moves the call to grid's resize() method a bit down to when all pages have been updated in updateRowCount(). (#297)
  • Loading branch information
janschatz authored and dylans committed Aug 16, 2018
1 parent 1bf1044 commit 52f7d70
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion grid/_Grid.js
Original file line number Diff line number Diff line change
Expand Up @@ -988,10 +988,10 @@ define([
if(this.layout.cells.length){
this.scroller.updateRowCount(inRowCount);
}
this._resize();
if(this.layout.cells.length){
this.setScrollTop(this.scrollTop);
}
this._resize();
}
},

Expand Down

0 comments on commit 52f7d70

Please sign in to comment.