Skip to content

Commit

Permalink
[BROWSEUI] Fix pos of taskbar toolbar right-click menu (reactos#5421)
Browse files Browse the repository at this point in the history
Don't call ClientToScreen in handling NM_RCLICK message because NMMOUSE.pt is already in screen coordinates.
CORE-18966
  • Loading branch information
katahiromz authored Jul 10, 2023
1 parent 831288f commit b4e946f
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions dll/win32/browseui/shellbars/CISFBand.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -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;
Expand All @@ -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;
Expand Down

0 comments on commit b4e946f

Please sign in to comment.