Make it possible to search for files whose name contains a space with the keyboard search

Review Request #113071 - Created Oct. 2, 2013 and submitted

Information
Frank Reininghaus
kde-baseapps
KDE/4.11
324479
Reviewers
dolphin
The user can jump quickly to files by entering the first few characters of the file name. However, we only add pressed keys to the search string if they have no other meaning. This means that files containing a Space in their name can currently not be searched because Ctrl+Space toggles the selection state of the current item, and Space alone selects the current item.

I propose to add Space to the search string if 

(a) the key press did not have any other effect, i.e., if Ctrl was not pressed, and the current item is selected already, and
(b) a keyboard search has been started already (to prevent unexpected effects when pressing Space accidentally - I think that it's rather uncommon to have files whose names start with a Space - and to make the unit test simpler).

I modified the unit test of KItemListController, which did not test keyboard search yet. This uncovered a small problem in KItemListController::slotChangeCurrentItem() when NoSelection mode is used. It's not really relevant for anything that is executed inside Dolphin, but I still fixed it to make the unit test happy.
I can search for files with a Space in the name now. Old and new unit tests pass, and I haven't seen any regressions so far.
Emmanuel Pescosta
Commit Hook
Frank Reininghaus
Review request changed

Status: Closed (submitted)

Loading...