diff options
author | Mitch Curtis <mitch.curtis@qt.io> | 2017-12-20 13:48:07 +0100 |
---|---|---|
committer | Mitch Curtis <mitch.curtis@qt.io> | 2017-12-20 15:00:45 +0000 |
commit | cbdbbe79ec141e66f3160a1332e49518f9682517 (patch) | |
tree | 7e6a4b0c149fc7c105c508fe481edffd1029b620 /src/quick/items/qquickitem.cpp | |
parent | 3bbf303403550e4e6f48b8778982d248651449ee (diff) |
Doc: correct Keys::shortcutOverride code snippet
c4eefa4a added a snippet where an onEscapePressed handler had:
event.accepted = true
which is unnecessary, as the documentation says that handlers for
specific key events set event.accepted to true by default.
Change-Id: I1a40e6e82240a517ba5059a1d5d2217cc7968302
Reviewed-by: Shawn Rutledge <shawn.rutledge@qt.io>
Diffstat (limited to 'src/quick/items/qquickitem.cpp')
-rw-r--r-- | src/quick/items/qquickitem.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/quick/items/qquickitem.cpp b/src/quick/items/qquickitem.cpp index 4a44b1531a..9adb993a2a 100644 --- a/src/quick/items/qquickitem.cpp +++ b/src/quick/items/qquickitem.cpp @@ -970,7 +970,7 @@ bool QQuickKeysAttached::isConnected(const char *signalName) const Keys.onEscapePressed: { console.log("escapeItem is handling escape"); - event.accepted = true; + // event.accepted is set to true by default for the specific key handlers } } |