https://git.reactos.org/?p=reactos.git;a=commitdiff;h=b4e946f5b0147480d0df34654b55f9a7705787ff

commit b4e946f5b0147480d0df34654b55f9a7705787ff
Author:     Katayama Hirofumi MZ <[email protected]>
AuthorDate: Tue Jul 11 05:28:50 2023 +0900
Commit:     GitHub <[email protected]>
CommitDate: Tue Jul 11 05:28:50 2023 +0900

    [BROWSEUI] Fix pos of taskbar toolbar right-click menu (#5421)
    
    Don't call ClientToScreen in handling NM_RCLICK message because NMMOUSE.pt 
is already in screen coordinates.
    CORE-18966
---
 dll/win32/browseui/shellbars/CISFBand.cpp | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/dll/win32/browseui/shellbars/CISFBand.cpp 
b/dll/win32/browseui/shellbars/CISFBand.cpp
index 2d8c7d4500b..c0266aeca9c 100644
--- a/dll/win32/browseui/shellbars/CISFBand.cpp
+++ b/dll/win32/browseui/shellbars/CISFBand.cpp
@@ -353,7 +353,7 @@ HRESULT CISFBand::CreateSimpleToolbar(HWND hWndParent)
                     case NM_RCLICK:
                     {
                         HRESULT hr;
-                        POINT pt = ((LPNMMOUSE)lParam)->pt;
+                        POINT pt = ((LPNMMOUSE)lParam)->pt; // Already in 
screen coordinates
                         CComPtr<IContextMenu> picm;
                         HMENU fmenu = CreatePopupMenu();
                         TBBUTTON tb;
@@ -363,7 +363,6 @@ HRESULT CISFBand::CreateSimpleToolbar(HWND hWndParent)
 
                         if (chk)
                         {
-                            ClientToScreen(&pt);
                             hr = m_pISF->GetUIObjectOf(m_hWnd, 1, &pidl, 
IID_NULL_PPV_ARG(IContextMenu, &picm));
                             if (FAILED_UNEXPECTEDLY(hr))
                                 return hr;

Reply via email to