To comment on the following update, log in, then open the issue: http://www.openoffice.org/issues/show_bug.cgi?id=31275
------- Additional comments from [email protected] Wed Jun 17 12:52:59 +0000 2009 ------- fs->os: The attached patch implements this, by outsourcing the ListBox'es quick search functionality into a dedicated QuickSelectionEngine class, and re-using it in the SvLBox. Two quirks so far: First, entries below collapsed nodes are included in the quick search, which probably is not desired. This should be easily fixable by adjusting FirstSearchEntry and NextSearchEntry in the SvLBox to not include such entries. Second, traveling does to reset the "quick search term". That is, when you type e.g. "sea", then an entry starting with this string is selected. When you press an additional "r" within a given time span, then an entry starting with "sear" is selected. If you wait to long before pressing "r", then the search term has been reset meanwhile, and a new search term consisting of "r" only is used, so the next entry starting with "r" is selected. Now what would be desirable is that if you move the cursor before typing "r", then the search term is reset, too. However, this does not work currently. The problem is that the SvLBox, which manages the QuickSelectionEngine instances, does not known about the cursors of its derived classes. More precise, it is not notified when the current entry changes. Two possible solutions come to my mind: a) Implement an OnCurEntryChanged (or so) in the SvLBox, to be called appropriately by all derived classes. b) use the (abstract) GetCurEntry in SvLBox::HandleKeyInput, and if the current entry changed since the last call to QuickSelectionEngine::HandleKeyEvent, then call QuickSelectionEngine::Reset. --------------------------------------------------------------------- Please do not reply to this automatically generated notification from Issue Tracker. Please log onto the website and enter your comments. http://qa.openoffice.org/issue_handling/project_issues.html#notification --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
