[PATCH 0/1] MR10205: comctl32/treeview: Fixed wrong check for currently edited item

[Piotr Pawłowski (＠DEATH)]

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=59455

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/10205

[PATCH 1/1] comctl32/treeview: Fixed wrong check for currently edited item Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=59455

[Piotr Pawłowski]

From: Piotr Pawłowski <[e-mail]>

---
 dlls/comctl32/treeview.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/dlls/comctl32/treeview.c b/dlls/comctl32/treeview.c
index 6cc49fa02ab..8b38f098553 100644
--- a/dlls/comctl32/treeview.c
+++ b/dlls/comctl32/treeview.c
@@ -2660,7 +2660,7 @@ TREEVIEW_DrawItem(const TREEVIEW_INFO *infoPtr, HDC hdc, TREEVIEW_ITEM *item)
      */
 
     /* Don't paint item's text if it's being edited */
-    if (!infoPtr->hwndEdit || (infoPtr->selectedItem != item))
+    if (!infoPtr->hwndEdit || (infoPtr->editItem != item))
     {
 	if (item->pszText)
 	{
-- 
GitLab

https://gitlab.winehq.org/wine/wine/-/merge_requests/10205

[Nikolay Sivov (＠nsivov)]

I'm not sure if we need this condition at all, it should work if we painted text all the time I think. Does it break child Edit box rendering?

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/10205#note_130720

[Piotr Pawłowski (＠DEATH)]

Edit box resizes with edited content - may shrink below the size of previous item text if you delete stuff.

The check is needed.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/10205#note_130721

[PATCH 0/1] MR10205: comctl32/treeview: Fixed wrong check for currently edited item - approved

[Piotr Pawłowski (＠DEATH)]

This merge request was approved by Piotr Pawłowski.
-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/10205