ubuntu-sdk-bugs team mailing list archive

[Launchpad Bug Tracker <1537782@xxxxxxxxxxxxxxxxxx>]

This bug was fixed in the package webbrowser-app -
0.23+16.04.20160408.1-0ubuntu1

---------------
webbrowser-app (0.23+16.04.20160408.1-0ubuntu1) xenial; urgency=medium

  [ CI Train Bot ]
  * Resync trunk.

  [ Loïc Molinari ]
  * Made AddressBar height scalable with regards to the grid units
    system.

  [ Olivier Tilloy ]
  * Add dep8 tests and instructions to run them in qemu or on a phone.
    Original work by Leo Arias and Vincent Ladeuil. added: debian/tests/
    debian/tests/control debian/tests/touch-session-autopilot
  * Catch ESC key event one level up to ensure that it’s not incorrectly
    bubbled up to the outer component. (LP: #1557016)
  * Customize the contents of the media permission dialog to avoid
    truncated text. (LP: #1554220)
  * Do not write the session to a temporary file when no target file is
    defined.
  * Fix a failing unit test with Qt 5.6. (LP: #1565507)
  * Fix broken webapp container autopilot tests. (LP: #1557019)
  * Fix issues with item selection in the downloads page: do not allow
    selecting multiple files when only one is expected do not allow
    entering delete mode (with a long press on an item) while in picker
    mode (LP: #1534112, #1561575)
  * Fix unit tests when run under an sbuild chroot. (LP: #1567294)
  * Import QQuickShortcut from Qt 5.5 to properly handle window-level
    keyboard shortcuts. We cannot bump the dependency on Qt to 5.5 as
    the stable overlay PPA for devices currently has Qt 5.4.1. (LP:
    #1542224, #1545802, #1537782)
  * Make the autopilot tests more reliable when dragging the bottom
    edge. (LP: #1560109)
  * Remove workaround for bug #1526940 that was fixed in the latest
    release of the UITK. (LP: #1526940)
  * Rename debian packages to follow new QML module naming convention.
    (LP: #1342031)
  * Simplify the implementation of HistoryViewWide quite a bit, and as a
    side effect fix a unit test failure when run against the staging
    branch of the UITK. (LP: #1567337)

 -- Olivier Tilloy <olivier.tilloy@xxxxxxxxxxxxx>  Fri, 08 Apr 2016
17:07:04 +0000

** Changed in: webbrowser-app (Ubuntu)
       Status: In Progress => Fix Released

-- 
You received this bug notification because you are a member of Ubuntu
SDK bug tracking, which is subscribed to qtdeclarative-opensource-src in
Ubuntu.
https://bugs.launchpad.net/bugs/1537782

Title:
  Need to use (Q)Shortcuts instead of Keys.onPressed for shortcuts

Status in Canonical System Image:
  Confirmed
Status in qtdeclarative-opensource-src package in Ubuntu:
  Invalid
Status in webbrowser-app package in Ubuntu:
  Fix Released
Status in webbrowser-app package in Ubuntu RTM:
  Fix Released

Bug description:
  On a phone with a bluetooth keyboard connected, with the following
  standalone example, if I press Ctrl+T I expect the message "Ctrl+T
  pressed" to be printed on the console, but instead the Ctrl key is
  ignored and "t" is inserted in the text field.

  import QtQuick 2.4
  Item {
    TextInput {
      anchors.centerIn: parent
      width: parent.width - 20
      height: 100
      Component.onCompleted: forceActiveFocus()
    }
    Keys.onPressed: {
      console.log("key pressed:", event.key, event.modifiers)
      if (event.key == Qt.Key_T && event.modifiers == Qt.ControlModifier) {
        console.log("Ctrl+T pressed")
        event.accepted = true
      }
    }
  }

  =========================================================================

  See the discussion on https://codereview.qt-project.org/#/c/147556/,
  Qt's behaviour is correct here.

  QShortcut [1], or QML's Shortcut [2] from qt-5.5 should be used for
  this instead.

  [1] http://doc.qt.io/qt-5/qshortcut.html
  [2] http://doc.qt.io/qt-5/qml-qtquick-shortcut.html

To manage notifications about this bug go to:
https://bugs.launchpad.net/canonical-devices-system-image/+bug/1537782/+subscriptions