From 44c82a16b3e2fc2d40601fd70f8253dce40dda86 Mon Sep 17 00:00:00 2001 From: Felix Ernst Date: Mon, 2 Jan 2023 16:01:49 +0100 Subject: Make space shortcut for selection mode view-local instead of global Before this commit, the "Space" keyboard shortcut was bound to triggering selection mode by default. After this commit, pressing "Space" will only trigger selection mode when the file view area has keyboard focus. Pros: + Other buttons in the UI can be triggered with Space once again just like it is expected from an accessibility point of view. + "Type-ahead" searching works once more when typing the space char for file names containing such a space char. Cons: - "Space" can no longer be used to add the currently underlined item to the selection. Instead "Ctrl+Space" needs to be used. (However, this is the current status anyway unless a user has manually unbound "Space" as a shortcut from Selection Mode.) - The Selection Mode action will no longer show "Space" as its shortcut in menus. Overall, I see solutions to all of these problems, but they seem over-engineered for the issues they are trying to solve, so I believe this somewhat small commit is the best solution for now. BUG: 458282 BUG: 458281 CCBUG: 463048 FIXED-IN: 23.04 --- src/kitemviews/private/kitemlistkeyboardsearchmanager.cpp | 5 +++++ 1 file changed, 5 insertions(+) (limited to 'src/kitemviews/private/kitemlistkeyboardsearchmanager.cpp') diff --git a/src/kitemviews/private/kitemlistkeyboardsearchmanager.cpp b/src/kitemviews/private/kitemlistkeyboardsearchmanager.cpp index 0e6280ede..e646a6249 100644 --- a/src/kitemviews/private/kitemlistkeyboardsearchmanager.cpp +++ b/src/kitemviews/private/kitemlistkeyboardsearchmanager.cpp @@ -63,6 +63,11 @@ void KItemListKeyboardSearchManager::addKeys(const QString& keys) m_keyboardInputTime.start(); } +bool KItemListKeyboardSearchManager::addKeyBeginsNewSearch() const +{ + return m_keyboardInputTime.hasExpired(m_timeout) || m_searchedString.isEmpty(); +} + void KItemListKeyboardSearchManager::setTimeout(qint64 milliseconds) { m_timeout = milliseconds; -- cgit v1.3