Aren/haiku
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Various cleanups to how the selection indices are maintained when part of the selection is occluded by a collapse operation, which could cause a crash in some instances. This fixes ticket #3711.

Browse Source 
git-svn-id: file:///srv/svn/repos/haiku/haiku/trunk@30015 a95241bf-73f2-0310-859d-f6bbb57e9c96
This commit is contained in:
Rene Gollent 2009-04-08 00:09:33 +00:00
parent b494a17f1c
commit cd03af05a4
1 changed files with 18 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

31
src/kits/interface/OutlineListView.cpp
View File

@ -483,6 +483,8 @@ BOutlineListView::Collapse(BListItem* item)
int32 startIndex = index;
int32 max = FullListCountItems() - fullIndex - 1;
int32 count = 0;
bool selectionChanged = false;
BListItem** items = (BListItem**)fFullList.Items() + fullIndex + 1;
while (max-- > 0) {
@ -493,9 +495,10 @@ BOutlineListView::Collapse(BListItem* item)
if (item->IsItemVisible()) {
fList.RemoveItem(item);
item->SetItemVisible(false);
if (item->IsSelected())
if (item->IsSelected()) {
selectionChanged = true;
item->Deselect();
}
count++;
}
@ -504,19 +507,21 @@ BOutlineListView::Collapse(BListItem* item)
_RecalcItemTops(startIndex);
// fix selection hints
// TODO: revise for multi selection lists
if (index < fFirstSelected) {
if (index + count < fFirstSelected) {
fFirstSelected -= count;
fLastSelected -= count;
} else {
// select top item
//fFirstSelected = fLastSelected = index;
//item->Select();
Select(index);
}
if (index < fFirstSelected && index + count < fFirstSelected) {
// all items removed were higher than the selection range,
// adjust the indexes to correspond to their new visible positions
fFirstSelected -= count;
fLastSelected -= count;
}
int32 maxIndex = fList.CountItems() - 1;
if (fFirstSelected > maxIndex)
fFirstSelected = maxIndex;
if (fLastSelected > maxIndex)
fLastSelected = maxIndex;
if (selectionChanged)
SelectionChanged();
_FixupScrollBar();
Invalidate();
}