Fix order of arguments in wxOSX/Carbon wxListCtrl::ScrollList().
wxMacDataBrowserTableViewControl::SetScrollPosition() takes "top" and "left" arguments so swap "dx" and "dy" parameters being passed to it. It might be better to fix SetScrollPosition() to follow the more natural (at least from wx point of view) convention and as it's used in only a few places this wouldn't be difficult to do. But let's keep the changes to this code to the minimum for now. Closes #12339. git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@65952 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775
This commit is contained in:
@@ -2319,7 +2319,9 @@ bool wxListCtrl::ScrollList(int dx, int dy)
|
||||
|
||||
if (m_dbImpl)
|
||||
{
|
||||
m_dbImpl->SetScrollPosition(dx, dy);
|
||||
// Notice that the parameter order is correct here: first argument is
|
||||
// the "top" displacement, second one is the "left" one.
|
||||
m_dbImpl->SetScrollPosition(dy, dx);
|
||||
}
|
||||
return true;
|
||||
}
|
||||
|
Reference in New Issue
Block a user