Differential D23716 Diff 67590 src/kitemviews/private/kitemlistkeyboardsearchmanager.cpp

Changeset View

Standalone View

View Options

src/kitemviews/private/kitemlistkeyboardsearchmanager.cpp

Show All 16 Lines
17 * You should have received a copy of the GNU General Public License * 17​ * You should have received a copy of the GNU General Public License *
18 * along with this program; if not, write to the * 18​ * along with this program; if not, write to the *
19 * Free Software Foundation, Inc., * 19​ * Free Software Foundation, Inc., *
20 * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA * 20​ * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA *
21 ***************************************************************************/ 21​ ***************************************************************************/
22 22​
23#include "kitemlistkeyboardsearchmanager.h" 23​#include "kitemlistkeyboardsearchmanager.h"
24 24​
25 ​
26KItemListKeyboardSearchManager::KItemListKeyboardSearchManager(QObject* parent) : 25​KItemListKeyboardSearchManager::KItemListKeyboardSearchManager(QObject* parent) :
27 QObject(parent), 26​ QObject(parent),
27​ m_isSearchRestarted(false),
28 m_timeout(1000) 28​ m_timeout(1000)
29{ 29​{
30 m_keyboardInputTime.invalidate(); 30​ m_keyboardInputTime.invalidate();
31} 31​}
32 32​
33KItemListKeyboardSearchManager::~KItemListKeyboardSearchManager() 33​KItemListKeyboardSearchManager::~KItemListKeyboardSearchManager()
34{ 34​{
35} 35​}
36 36​
37bool KItemListKeyboardSearchManager::shouldClearSearchIfInputTimeReached() 37​bool KItemListKeyboardSearchManager::shouldClearSearchIfInputTimeReached()
38{ 38​{
39 const bool keyboardTimeWasValid = m_keyboardInputTime.isValid(); 39​ const bool keyboardTimeWasValid = m_keyboardInputTime.isValid();
40 const qint64 keyboardInputTimeElapsed = m_keyboardInputTime.restart(); 40​ const qint64 keyboardInputTimeElapsed = m_keyboardInputTime.restart();
41 return (keyboardInputTimeElapsed > m_timeout) || !keyboardTimeWasValid; 41​ return (keyboardInputTimeElapsed > m_timeout) || !keyboardTimeWasValid;
42} 42​}
43 43​
44void KItemListKeyboardSearchManager::addKeys(const QString& keys) 44​void KItemListKeyboardSearchManager::addKeys(const QString& keys)
45{ 45​{
46 if (shouldClearSearchIfInputTimeReached()) { 46​ if (shouldClearSearchIfInputTimeReached()) {
47 m_searchedString.clear(); 47​ m_searchedString.clear();
elvisangelaccioUnsubmitted
Done

This is an unrelated refactoring that should go in its own commit. Feel free to commit this single change without review.

elvisangelaccio: This is an unrelated refactoring that should go in its own commit. Feel free to commit this…
48 } 48​ }
49 49​
50 const bool newSearch = m_searchedString.isEmpty(); 50​ const bool newSearch = m_searchedString.isEmpty();
51 51​
52 // Do not start a new search if the user pressed Space. Only add 52​ // Do not start a new search if the user pressed Space. Only add
53 // it to the search string if a search is in progress already. 53​ // it to the search string if a search is in progress already.
54 if (newSearch && keys == QLatin1Char(' ')) { 54​ if (newSearch && keys == QLatin1Char(' ')) {
55 return; 55​ return;
56 } 56​ }
57 57​
58 if (!keys.isEmpty()) { 58​ if (!keys.isEmpty()) {
59 m_searchedString.append(keys); 59​ m_searchedString.append(keys);
60 60​
61 // Special case: 61​ // Special case:
62 // If the same key is pressed repeatedly, the next item matching that key should be highlighted 62​ // If the same key is pressed repeatedly, the next item matching that key should be highlighted
63 const QChar firstKey = m_searchedString.length() > 0 ? m_searchedString.at(0) : QChar(); 63​ const QChar firstKey = m_searchedString.length() > 0 ? m_searchedString.at(0) : QChar();
64 const bool sameKey = m_searchedString.length() > 1 && m_searchedString.count(firstKey) == m_searchedString.length(); 64​ const bool sameKey = m_searchedString.length() > 1 && m_searchedString.count(firstKey) == m_searchedString.length();
65 65​
66 // Searching for a matching item should start from the next item if either 66​ // Searching for a matching item should start from the next item if either
67 // 1. a new search is started, or 67​ // 1. a new search is started and a search has not been restarted or
68 // 2. a 'repeated key' search is done. 68​ // 2. a 'repeated key' search is done.
69 const bool searchFromNextItem = newSearch || sameKey; 69​ const bool searchFromNextItem = (!m_isSearchRestarted && newSearch) || sameKey;
70​
71​ // to remember not to searchFromNextItem if selection was deselected
72​ // loosing keyboard search context basically
73​ m_isSearchRestarted = false;
70 74​
71 emit changeCurrentItem(sameKey ? firstKey : m_searchedString, searchFromNextItem); 75​ emit changeCurrentItem(sameKey ? firstKey : m_searchedString, searchFromNextItem);
72 } 76​ }
73 m_keyboardInputTime.start(); 77​ m_keyboardInputTime.start();
74} 78​}
75 79​
76void KItemListKeyboardSearchManager::setTimeout(qint64 milliseconds) 80​void KItemListKeyboardSearchManager::setTimeout(qint64 milliseconds)
77{ 81​{
78 m_timeout = milliseconds; 82​ m_timeout = milliseconds;
79} 83​}
80 84​
81qint64 KItemListKeyboardSearchManager::timeout() const 85​qint64 KItemListKeyboardSearchManager::timeout() const
82{ 86​{
83 return m_timeout; 87​ return m_timeout;
84} 88​}
85 89​
86void KItemListKeyboardSearchManager::cancelSearch() 90​void KItemListKeyboardSearchManager::cancelSearch()
87{ 91​{
92​ m_isSearchRestarted = true;
88 m_searchedString.clear(); 93​ m_searchedString.clear();
89} 94​}
90 95​
91void KItemListKeyboardSearchManager::slotCurrentChanged(int current, int previous) 96​void KItemListKeyboardSearchManager::slotCurrentChanged(int current, int previous)
92{ 97​{
93 Q_UNUSED(previous); 98​ Q_UNUSED(previous);
94 99​
95 if (current < 0) { 100​ if (current < 0) {
96 // The current item has been removed. We should cancel the search. 101​ // The current item has been removed. We should cancel the search.
97 cancelSearch(); 102​ cancelSearch();
98 } 103​ }
99} 104​}
105​
106​void KItemListKeyboardSearchManager::slotSelectionChanged(const KItemSet& current, const KItemSet& previous)
107​{
108​ if (!previous.isEmpty() && current.isEmpty() && previous.count() > 0 && current.count() == 0) {
109​ // The selection has been emptied. We should cancel the search.
110​ cancelSearch();
111​ }
112​}