Ticket #11619 (closed defect: fixed)

Opened 3 years ago

Last modified 3 years ago

Reset start selection variable when we delete item

Reported by: TNikolay Owned by:
Priority: normal Milestone:
Component: wxMSW Version: 2.9-svn
Keywords: wxTreeCtrl selection Cc:
Blocked By: Patch: yes
Blocking:

Description

We should reset m_htSelStart when we delete item. Probably patch expand better :)

Attachments

tree-unset-selection-when-delete.patch download (0.6 KB) - added by TNikolay 3 years ago.

Change History

Changed 3 years ago by TNikolay

Changed 3 years ago by neis

  • patch set

Changed 3 years ago by TNikolay

This patch fixes the following problem in wxTreeCtrl with multiple selection:

1. select a few items

2. delete the.

m_htSelStart  will be point to the latest selected item, but we already have deleted it. So if user press shift + arrow up at this point - we get incorrect selected range.

Changed 3 years ago by vadz

  • status changed from new to confirmed

Thanks for the patch, it seems correct to me but I think we should also do the same for m_htClickedItem, shouldn't we?

I'll apply the obvious generalization of this patch for both of them soon if there are no objections.

Changed 3 years ago by VZ

  • status changed from confirmed to closed
  • resolution set to fixed

(In [63170]) Invalidate stored wxTreeItemIds when the corresponding item is deleted in wxMSW.

m_htSelStart and m_htClickedItem stored in wxTreeCtrl must be invalidated when
the item they point to is deleted (which also happens when all the items are
deleted), otherwise the selection doesn't behave correctly after doing it.

Closes #11619.

Note: See TracTickets for help on using tickets.