From 8ae80de41f98d12fbdb36eb4732195d21d1f4fa8 Mon Sep 17 00:00:00 2001 From: Vadim Zeitlin Date: Sun, 3 Mar 2013 12:19:20 +0000 Subject: [PATCH] Revert "Fix off by 1 error in wxGenericListCtrl::HitTest()." Revert r73563, this breaks the selection in the control apparently. See #15063. git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@73596 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775 --- docs/changes.txt | 1 - src/generic/listctrl.cpp | 4 ---- 2 files changed, 5 deletions(-) diff --git a/docs/changes.txt b/docs/changes.txt index 52b72d3..74939c4 100644 --- a/docs/changes.txt +++ b/docs/changes.txt @@ -621,7 +621,6 @@ All (GUI): - Add wxMask::GetBitmap() for wxMSW, wxGTK and wxOSX - Add wxCheckListBox::GetCheckedItems() (hartwigw). - Add wxAUI_TB_PLAIN_BACKGROUND wxAuiToolBar style (Allann Jones). -- Fix off by 1 error in wxGenericListCtrl::HitTest() (Daniel Hyams). - Make wxGenericDataViewCtrl::SetFont() really work (Laurent Poujoulat). wxGTK: diff --git a/src/generic/listctrl.cpp b/src/generic/listctrl.cpp index caebe00..b311e03 100644 --- a/src/generic/listctrl.cpp +++ b/src/generic/listctrl.cpp @@ -4144,10 +4144,6 @@ long wxListMainWindow::HitTest( int x, int y, int &flags ) const if ( InReportView() ) { - // Account for the header height if it's present. - if ( HasHeader() ) - y -= GetListCtrl()->m_headerWin->GetSize().y + 1; - size_t current = y / GetLineHeight(); if ( current < count ) { -- 2.7.4