diff options
author | Jason Barron <jbarron@trolltech.com> | 2009-09-28 09:54:06 (GMT) |
---|---|---|
committer | Jason Barron <jbarron@trolltech.com> | 2009-09-28 10:04:18 (GMT) |
commit | c912b0234ead0e35a969b5600a61c77b60e18aa6 (patch) | |
tree | 402e36ab7fc51708682f248887275aa52825b86d /src/gui | |
parent | f8188b14493eb3d01263be5fef4ea086d3a542a0 (diff) | |
download | Qt-c912b0234ead0e35a969b5600a61c77b60e18aa6.zip Qt-c912b0234ead0e35a969b5600a61c77b60e18aa6.tar.gz Qt-c912b0234ead0e35a969b5600a61c77b60e18aa6.tar.bz2 |
Make sure the correct soft key is triggered.
In cases where there are both softkey actions as well as "normal"
actions, we need to be sure to skip over the none softkey actions since
they should not be a part of the softkey framework and the 'index' will
be out of sync.
Reviewed-by: Sami Merila
Diffstat (limited to 'src/gui')
-rw-r--r-- | src/gui/kernel/qsoftkeymanager.cpp | 12 |
1 files changed, 9 insertions, 3 deletions
diff --git a/src/gui/kernel/qsoftkeymanager.cpp b/src/gui/kernel/qsoftkeymanager.cpp index 45ecb5a..7874622 100644 --- a/src/gui/kernel/qsoftkeymanager.cpp +++ b/src/gui/kernel/qsoftkeymanager.cpp @@ -247,9 +247,15 @@ bool QSoftKeyManager::handleCommand(int command) if (command >= s60CommandStart && QSoftKeyManagerPrivate::softKeySource) { int index = command - s60CommandStart; const QList<QAction*>& softKeys = QSoftKeyManagerPrivate::softKeySource->actions(); - if (index < softKeys.count()) { - softKeys.at(index)->activate(QAction::Trigger); - return true; + for (int i = 0, j = 0; i < softKeys.count(); ++i) { + QAction *action = softKeys.at(i); + if (action->softKeyRole() != QAction::NoSoftKey) { + if (j == index) { + action->activate(QAction::Trigger); + return true; + } + j++; + } } } |