Intravision/reactos
1
0
Fork 0
mirror of https://github.com/reactos/reactos.git synced 2025-08-05 17:52:56 +00:00
Code Issues Projects Releases Packages Wiki Activity

[BROWSEUI] Fix pos of taskbar toolbar right-click menu (#5421)

Browse source 
Don't call ClientToScreen in handling NM_RCLICK message because NMMOUSE.pt is already in screen coordinates.
CORE-18966
This commit is contained in:
Katayama Hirofumi MZ 2023-07-11 05:28:50 +09:00 • committed by GitHub
parent 831288fe69
commit b4e946f5b0
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 1 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

3
dll/win32/browseui/shellbars/CISFBand.cpp
View file

@ -353,7 +353,7 @@ HRESULT CISFBand::CreateSimpleToolbar(HWND hWndParent)
case NM_RCLICK: case NM_RCLICK:
{ {
HRESULT hr; HRESULT hr;
POINT pt = ((LPNMMOUSE)lParam)->pt; POINT pt = ((LPNMMOUSE)lParam)->pt; // Already in screen coordinates
CComPtr<IContextMenu> picm; CComPtr<IContextMenu> picm;
HMENU fmenu = CreatePopupMenu(); HMENU fmenu = CreatePopupMenu();
TBBUTTON tb; TBBUTTON tb;
@ -363,7 +363,6 @@ HRESULT CISFBand::CreateSimpleToolbar(HWND hWndParent)
if (chk) if (chk)
{ {
ClientToScreen(&pt);
hr = m_pISF->GetUIObjectOf(m_hWnd, 1, &pidl, IID_NULL_PPV_ARG(IContextMenu, &picm)); hr = m_pISF->GetUIObjectOf(m_hWnd, 1, &pidl, IID_NULL_PPV_ARG(IContextMenu, &picm));
if (FAILED_UNEXPECTEDLY(hr)) if (FAILED_UNEXPECTEDLY(hr))
return hr; return hr;