Fix ListCtrl::HitTest() on wxMSW not accounting for column header (PR #26948)
PB
When the coordinates were on the column header, HitTest() unexpectedly returned the index of the topmost visible item instead of wxNOT_FOUND.
See #22239.
This was fixed thanks to @gix, who provided the solution in #22239.
I am not closing #22239, since it also describes a similar problem on macOS, which, of course, may have been somehow fixed since but I cannot test it.
You can view, comment on, or merge this pull request online at:
https://github.com/wxWidgets/wxWidgets/pull/26948
Commit Summary
- 0316c44 Fix ListCtrl::HitTest() on wxMSW not accounting for column header
File Changes
(1 file)
- M src/msw/listctrl.cpp (10)
Patch Links:
- https://github.com/wxWidgets/wxWidgets/pull/26948.patch
- https://github.com/wxWidgets/wxWidgets/pull/26948.diff
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications, keep track of coding agent tasks and review pull requests on the go with GitHub Mobile for iOS and Android. Download it today!
You are receiving this because you are subscribed to this thread.
VZ
vadz left a comment (wxWidgets/wxWidgets#26948)Thanks for fixing and testing this!
I'll close the other issue as it's 99% about wxMSW and if the problem still exists in wxOSX, I'll let someone open a new issue about it.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications, keep track of coding agent tasks and review pull requests on the go with GitHub Mobile for iOS and Android. Download it today!
You are receiving this because you are subscribed to this thread.
VZ
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications, keep track of coding agent tasks and review pull requests on the go with GitHub Mobile for iOS and Android. Download it today!
You are receiving this because you are subscribed to this thread.