[Bug 57948] New: Listview / LVM_GETORIGIN is broken
https://bugs.winehq.org/show_bug.cgi?id=57948 Bug ID: 57948 Summary: Listview / LVM_GETORIGIN is broken Product: Wine Version: 10.3 Hardware: x86-64 OS: Linux Status: UNCONFIRMED Severity: normal Priority: P2 Component: comctl32 Assignee: wine-bugs(a)winehq.org Reporter: kyle.kcsoftwares(a)gmail.com Distribution: --- See test program (see ListViewOrigin.cpp in ListViewOrigin.zip) from https://jira.reactos.org/browse/CORE-17266 which does a simple call to SendMessage(g_hList, LVM_ENSUREVISIBLE, 35, FALSE); POINT pt; ListView_GetOrigin(g_hList, &pt); INT x = GetScrollPos(g_hList, SB_HORZ); INT y = GetScrollPos(g_hList, SB_VERT); if (y != 0 && pt.x == x && pt.y == y) MessageBox(hwnd, TEXT("success 1"), TEXT("success 1"), MB_ICONINFORMATION); else MessageBox(hwnd, TEXT("failure 1"), TEXT("failure 1"), MB_ICONINFORMATION); SendMessage(g_hList, WM_VSCROLL, MAKELONG(SB_TOP, 0), 0); ListView_GetOrigin(g_hList, &pt); x = GetScrollPos(g_hList, SB_HORZ); y = GetScrollPos(g_hList, SB_VERT); if (y == 0 && pt.x == x && pt.y == y) MessageBox(hwnd, TEXT("success 2"), TEXT("success 2"), MB_ICONINFORMATION); else MessageBox(hwnd, TEXT("failure 2"), TEXT("failure 2"), MB_ICONINFORMATION); Shows "success 1" and "success 2" in Windows Shows "failure 1" and "failure 2" in Wine 10.3 -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=57948 KRosUser <kyle.kcsoftwares(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Distribution|--- |Ubuntu Keywords| |download URL| |https://jira.reactos.org/br | |owse/CORE-17266 -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=57948 --- Comment #1 from KRosUser <kyle.kcsoftwares(a)gmail.com> --- Created attachment 78197 --> https://bugs.winehq.org/attachment.cgi?id=78197 Fails on Ubuntu / Wine 10.3 -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=57948 Nikolay Sivov <bunglehead(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|UNCONFIRMED |NEW Ever confirmed|0 |1 --- Comment #2 from Nikolay Sivov <bunglehead(a)gmail.com> --- Thanks, I set a fix for that https://gitlab.winehq.org/wine/wine/-/merge_requests/7541 and another one for SB_TOP. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=57948 --- Comment #3 from Nikolay Sivov <bunglehead(a)gmail.com> --- Should be fixed now, please retest. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=57948 --- Comment #4 from KRosUser <kyle.kcsoftwares(a)gmail.com> --- I could only test when it will be shipped in 10.4 or later. If you run the test application with this modification does it show "success 1" and "success 2" ? -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=57948 --- Comment #5 from Nikolay Sivov <bunglehead(a)gmail.com> --- It does, yes. Let's wait for 10.4, so you can verify as well. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=57948 --- Comment #6 from KRosUser <kyle.kcsoftwares(a)gmail.com> --- I confirm test is OK, thank you ! You can close this ticket -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=57948 Nikolay Sivov <bunglehead(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Resolution|--- |FIXED Fixed by SHA1| |2ea75572d9e42f5cee8934ba109 | |4f1bf88d05d5a Status|NEW |RESOLVED --- Comment #7 from Nikolay Sivov <bunglehead(a)gmail.com> --- Marking fixed 2ea75572d9e42f5cee8934ba1094f1bf88d05d5a. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=57948 Nikolay Sivov <bunglehead(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Summary|Listview / LVM_GETORIGIN is |Listview / LVM_GETORIGIN |broken |returns wrong coordinates -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=57948 Alexandre Julliard <julliard(a)winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED --- Comment #8 from Alexandre Julliard <julliard(a)winehq.org> --- Closing bugs fixed in 10.5. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
participants (1)
-
WineHQ Bugzilla