[MainView] contentOrientation is missing which makes the keyboard rotate unrelated to the content

Bug #1187321 reported by Michael Zanetti
20
This bug affects 3 people
Affects Status Importance Assigned to Milestone
Ubuntu UI Toolkit
Fix Released
Critical
Florian Boucault
touch-preview-images
Fix Released
Critical
Florian Boucault
notes-app (Ubuntu)
Fix Released
Undecided
Unassigned
ubuntu-ui-toolkit (Ubuntu)
Fix Released
Undecided
Unassigned

Bug Description

http://doc-snapshot.qt-project.org/qt5-stable/qtquick/qml-qtquick-window2-window.html#contentOrientation-prop

This property should be implemented in the SDK so that Qt can forward it to the OSK. Right now, the OSK has no idea about the content orientation and rotates unrelated to the content.

Tags: autopilot

Related branches

Changed in ubuntu-ui-toolkit:
status: New → Confirmed
assignee: nobody → Florian Boucault (fboucault)
importance: Undecided → High
Changed in qtubuntu:
importance: Undecided → High
Revision history for this message
Bill Filler (bfiller) wrote :

This is needed to ensure that OSK and app stay in sync. The OSK should be querying this property by checking whether or not the app is rotated rather than looking at the raw sensor data.

We are experiencing problems with autopilot tests. If we get into this state where the app is not rotate but the keyboard is, then tests fail.

Revision history for this message
Bill Filler (bfiller) wrote :

Bug https://bugs.launchpad.net/touch-preview-images/+bug/1201109 tracks the changes that need to be made to the OSK once this is fixed.

Changed in ubuntu-ui-toolkit:
importance: High → Critical
tags: added: autopilot
Revision history for this message
Florian Boucault (fboucault) wrote :

The API in the bug's description (QWindow::contentOrientation) is new in Qt 5.1 (which we are not using yet).
If we could use it right now, we would have to:
1) set it appropriately in the toolkit's OrientationHelper.qml
2) implement QPlatformWindow::handleContentOrientationChange in QtUbuntu so that it writes that value somewhere the keyboard can read it; for example, the xcb equivalent of QtUbuntu sets a property on the XWindow. It could be that we want the shell/window manager/compositor (= unity) to know about it too.

In the meantime, a workaround could be possible but the cost is quite high (introducing a temporary API in the toolkit that would do now what QtUbuntu would do later).

Changed in qtubuntu:
status: New → Confirmed
Revision history for this message
Florian Boucault (fboucault) wrote :

Actually it feels like QtUbuntu to implement 2) should rely on a third step:
3) expose per window/surface content orientation in ubuntu platform api

Changed in qtubuntu:
importance: High → Critical
Revision history for this message
Bill Filler (bfiller) wrote :

per discussion, the plan is to use the contentOrientation property on QtWindow
http://qt-project.org/doc/qt-5.0/qtgui/qwindow.html#contentOrientation-prop

The maliit qt plugin will need to change to stop reading from Screen. orientation and instead use this property. The sdk will be updated to properly set this property.

Changed in ubuntu-ui-toolkit:
status: Confirmed → In Progress
no longer affects: qtubuntu
Changed in touch-preview-images:
importance: Undecided → Critical
status: New → In Progress
assignee: nobody → Florian Boucault (fboucault)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

Fix committed into lp:ubuntu-ui-toolkit at revision None, scheduled for release in ubuntu-ui-toolkit, milestone Unknown

Changed in ubuntu-ui-toolkit:
status: In Progress → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package ubuntu-ui-toolkit - 0.1.46+13.10.20130729-0ubuntu1

---------------
ubuntu-ui-toolkit (0.1.46+13.10.20130729-0ubuntu1) saucy; urgency=low

  [ Kaleo ]
  * OrientationHelper: report the current orientation of the application
    via QWindow::contentOrientation. (LP: #1187321)
  * Reverted revision 658 introducing reporting of content orientation.
    Was causing crashes on x86. (LP: #1205615)

  [ Thomas Moenicke ]
  * OrientationHelper: report the current orientation of the application
    via QWindow::contentOrientation. (LP: #1187321)

  [ Michał Sawicz ]
  * Depend on devscripts for licensecheck.

  [ Ubuntu daily release ]
  * Automatic snapshot from revision 661
 -- Ubuntu daily release <email address hidden> Mon, 29 Jul 2013 04:02:12 +0000

Changed in ubuntu-ui-toolkit (Ubuntu):
status: New → Fix Released
Changed in ubuntu-ui-toolkit:
status: Fix Committed → In Progress
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

Fix committed into lp:ubuntu-ui-toolkit at revision 663, scheduled for release in ubuntu-ui-toolkit, milestone Unknown

Changed in ubuntu-ui-toolkit:
status: In Progress → Fix Committed
Ugo Riboni (uriboni)
Changed in notes-app (Ubuntu):
status: New → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package notes-app - 1.4+13.10.20130731-0ubuntu1

---------------
notes-app (1.4+13.10.20130731-0ubuntu1) saucy; urgency=low

  [ Sergio Schvezov ]
  * Adding test xml output and coverage targets.

  [ Omer Akram ]
  * added the functionality to make sure the keyboard appears before
    changing focus to another field. (LP: #1204084)

  [ Ugo Riboni ]
  * Rename the main window to something that doesn't clash with the
    "window" name injected by the SDK to expose the main view. (LP:
    #1187321)

  [ Ubuntu daily release ]
  * Automatic snapshot from revision 167
 -- Ubuntu daily release <email address hidden> Wed, 31 Jul 2013 05:02:49 +0000

Changed in notes-app (Ubuntu):
status: Fix Committed → Fix Released
Changed in ubuntu-ui-toolkit:
status: Fix Committed → Fix Released
Changed in touch-preview-images:
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.