[Bug 49097] New: Treeview unduly handles WM_SYSKEYDOWN

[WineHQ Bugzilla]

https://bugs.winehq.org/show_bug.cgi?id=49097

            Bug ID: 49097
           Summary: Treeview unduly handles WM_SYSKEYDOWN
           Product: Wine
           Version: unspecified
          Hardware: x86
                OS: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: P2
         Component: comctl32
          Assignee: wine-bugs(a)winehq.org
          Reporter: contact(a)kcsoftwares.com
      Distribution: ---

Treeview unduly handles WM_SYSKEYDOWN

Refer to treeview.c
    case WM_KEYDOWN:
    case WM_SYSKEYDOWN:
    return TREEVIEW_KeyDown(infoPtr, wParam);

No such processing on Listview : 
  case WM_KEYDOWN:
    return LISTVIEW_KeyDown(infoPtr, (INT)wParam, (LONG)lParam);


As a consequence, WM_SYSKEYDOWN not correctly passed to parent Wnd if focus is
on a Treeview control.

Proof of concept : Use ReactOS. Open Regedit or Event Viewer. Put focus on
treeview. Alt+F4 does nothing (WM_SYSKEYDOWN "stuck" at Treeview component
level)

Suggested patch : 
change
    case WM_KEYDOWN:
    case WM_SYSKEYDOWN:
    return TREEVIEW_KeyDown(infoPtr, wParam);
into
    case WM_KEYDOWN:
    return TREEVIEW_KeyDown(infoPtr, wParam);

as no reason to handle WM_SYSKEYDOWN at component level.

-- 
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.