Bug 161805 [Viewers] JFace TreeViewer collapses node when sibbling removed
3.2 1. Use the Eclipse Wizard to produce a plugin with a tree viewer view. 2. Paste the modified SampleView.java - this looks like the left part of the attached screenshot 3. Click on the top node Parent1 and type - 4. TreeViewer.remove is called to remove Parent 1. TreeViewer.refresh is called - this looks like the right part of the attached screenshot Wrong: node Parent 4 is now collapsed. Expectation Parent 4 is still expanded. Is there any workaround? Am I misusing your API? Note. The snippet is almost identical to the code generated by PDE for plugin / view / Tree Viewer. Here is the extra code in createPartControl: viewer.getTree().addKeyListener(new KeyListener() { public void keyReleased(KeyEvent e) { // TODO Auto-generated method stub } public void keyPressed(KeyEvent e) { if (e.character == '-') { TreeParent parent = (TreeParent) ((IStructuredSelection)viewer.getSelection()).toArray()[0]; parent.getParent().removeChild(parent); viewer.refresh(); } if (e.character == '+') { TreeParent parent = (TreeParent) ((IStructuredSelection)viewer.getSelection()).toArray()[0]; parent.addChild(new TreeParent(new Date().toString())); } } });
bundles/org.eclipse.jface/src/org/eclipse/jface/viewers/AbstractTreeViewer.java
}
		return false;
	}
	/**
	 * Returns the index where the item should be inserted. It uses sorter to
	 * determine the correct position, if sorter is not assigned, returns the
	 * index of the element after the last.
	 * 
	 * @param items
	 *            the items to search
	 * @param sorter
	 *            The sorter to use.
	 * @param lastInsertion
	 *            the start index to start search for position from this allows
	 *            optimising search for multiple elements that are sorted
	 *            themself.
	 * @param element
	 *            element to find position for.
	 * @param parentPath
	 *            the tree path for the element's parent or <code>null</code>
	 *            if the element is a root element or the sorter is not a
	 *            {@link TreePathViewerSorter}
	 * @return the index to use when inserting the element.
	 * 
	 */
	private int insertionPosition(Item[] items, ViewerComparator comparator,
			int lastInsertion, Object element, TreePath parentPath) {
		int size = items.length;
		if (comparator == null) {
			return size;