Amebis/wxWidgets
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix code determining the rows to refresh in wxGrid::SetColSize()

Browse Source 
The changes in 15de1a4cf4 (Optimize refresh when resizing grid rows or
columns, 2020-02-18) incorrectly translated XToPos() used in
SetRowSize() to YToRow(), which compiled due to an implicit conversion
of m_gridWin pointer to bool, but did the wrong thing, as the wrong grid
window could be used for a frozen grid.

Fix this by switching to internalYToRow() which actually does what the
original code tried to do.

Co-authored-by: Artur Wieczorek <artwik@wp.pl>
This commit is contained in:
Vadim Zeitlin
2020-07-05 14:59:21 +02:00
parent 97924a3b81
commit a9b419af51
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
src/generic/grid.cpp
View File

@@ -9869,8 +9869,8 @@ void wxGrid::DoSetColSize( int col, int width )
CalcUnscrolledPosition(0, rect.GetTop(), NULL, &top);
CalcUnscrolledPosition(0, rect.GetBottom(), NULL, &bottom);
const int rowTop = YToRow(top, m_gridWin);
const int rowBottom = YToRow(bottom, m_gridWin);
const int rowTop = internalYToRow(top, m_gridWin);
const int rowBottom = internalYToRow(bottom, m_gridWin);
for ( int row = rowTop; row <= rowBottom; ++row )
{
int numRows, numCols;