[TASKMGR] Application Page: Fix "Go To Process" item selection #4448
Conversation
| @@ -944,6 +944,8 @@ void ApplicationPage_OnGotoProcess(void) | |||
| i = ProcGetIndexByProcessId(dwProcessId); | |||
| if (i != -1) | |||
| { | |||
| ListView_SetSelectionMark(hProcessPageListCtrl, i); | |||
There was a problem hiding this comment.
How does this differ with the SetItemState from below? (is there also some sort of visual difference, with and without that line?) -- Note that this is this sort of "explanation" that was to be given into the "Purpose" of the PR.
There was a problem hiding this comment.
Without that line, it selects wrong process in the process page.
ListView_SetItemState only shows process selection but it doesn't change the selected index.
You can reproduce the bug by selecting "Properties" after "Go To Process"
There was a problem hiding this comment.
So you mean, that setitemstate puts the "blue" around the process, but the "selected" item with dotted rectangle is still at the old process?
There was a problem hiding this comment.
I cannot reproduce this problem on unpatched Task Manager in Windows 10. Need to check on XP/2003.
|
The problem maybe in another module, taskmgr works fine on Windows. |
Purpose
Fix "Go To Process" selecting wrong process in the process page