Project: JDT
Bug 58777 [Viewers] add API to not deactivate CellEditor on FocusLost
Reported: 2004-04-16 03:00:36 | Status: verified fixed

TableViewerImpl installs a FocusListener on CellEditors, which calls applyEditorValue() on focusLost events. This FocusListener must be removed since applying the editor value is not always the wanted behavior on focusLost. Subclasses of CellEditor already handle focus loss in an extensible way: they install their own FocusListener and call a hook method focusLost() (which can be extended or reimplemented by subclasses with special needs). To support content assist in table cells (such as in the Change Method Signature dialog), we need to stay in editing mode even while the proposal list has focus. This is currently not implementable, since TableViewerImpl deactivates the CellEditor even if the CellEditor's focusLost() is reimplemented. The fix is to simply remove the redundant FocusListener in TableViewerImpl.

Files changed in fix:
org.eclipse.jdt.ui/ui/org/eclipse/jdt/internal/ui/dialogs/TableTextCellEditor.java