diff options
author | Janne Anttila <janne.anttila@digia.com> | 2010-04-08 10:55:21 (GMT) |
---|---|---|
committer | Janne Anttila <janne.anttila@digia.com> | 2010-04-09 08:00:03 (GMT) |
commit | e86eabc78e222d9c46a38940085025dea518aefa (patch) | |
tree | 9c5c7a6fa12253956a7f110bcc2c443ac2c32355 | |
parent | 0ef1365a07ca63b8243d8c051ebc7a8b7a7b5d5e (diff) | |
download | Qt-e86eabc78e222d9c46a38940085025dea518aefa.zip Qt-e86eabc78e222d9c46a38940085025dea518aefa.tar.gz Qt-e86eabc78e222d9c46a38940085025dea518aefa.tar.bz2 |
Generate triggered signal even the action launches menu in Symbian.
Triggered signal is useful for detecting native 'Options' menu
launches in Symbian. QMenu::aboutToShow event is currently also not
generated, but that is part of another bug report. And
QMenu::aboutToShow would not even be generated for 'Options' menu
itself but only for its sub/cascade menus.
Task-number: QTBUG-9669
Reviewed-by: Sami Merila
-rw-r--r-- | src/gui/kernel/qsoftkeymanager_s60.cpp | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/src/gui/kernel/qsoftkeymanager_s60.cpp b/src/gui/kernel/qsoftkeymanager_s60.cpp index 2a9ac42..c0761f0 100644 --- a/src/gui/kernel/qsoftkeymanager_s60.cpp +++ b/src/gui/kernel/qsoftkeymanager_s60.cpp @@ -408,14 +408,14 @@ bool QSoftKeyManagerPrivateS60::handleCommand(int command) } qt_symbian_next_menu_from_action(actionContainer); QT_TRAP_THROWING(tryDisplayMenuBarL()); - } else { - Q_ASSERT(action->softKeyRole() != QAction::NoSoftKey); - QWidget *actionParent = action->parentWidget(); - Q_ASSERT_X(actionParent, Q_FUNC_INFO, "No parent set for softkey action!"); - if (actionParent->isEnabled()) { - action->activate(QAction::Trigger); - return true; - } + } + + Q_ASSERT(action->softKeyRole() != QAction::NoSoftKey); + QWidget *actionParent = action->parentWidget(); + Q_ASSERT_X(actionParent, Q_FUNC_INFO, "No parent set for softkey action!"); + if (actionParent->isEnabled()) { + action->activate(QAction::Trigger); + return true; } } return false; |