notes-app

The input focus gets lost when the OSK appears during autopilot test run

Bug #1204084 reported by Omer Akram
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
notes-app
Fix Released
High
Omer Akram
notes-app (Ubuntu)
Fix Released
Undecided
Unassigned

Bug Description

In some cases when the OSK appears the focus from the input box gets lost, to workaround that I added a 1second delay on touch devices to re-click the input area to gain focus, which is not ideal.

to test you need to remove the timeout in: notes_app.tests.test_parts.TestFocus.test_parts_focus and run the test on touch device.

Tags: autopilot

Related branches

lp:~om26er/notes-app/remove_autopilot_workaround
Olivier Tilloy: Approve
PS Jenkins bot: Approve (continuous-integration)
Diff: 209 lines (+102/-38)
5 files modified
KeyboardRectangle.qml (+81/-0)
NotesApp.qml (+1/-35)
tests/autopilot/notes_app/emulators/main_window.py (+3/-0)
tests/autopilot/notes_app/tests/__init__.py (+14/-0)
tests/autopilot/notes_app/tests/test_parts.py (+3/-3)
lp:ubuntu/saucy/notes-app
Revision history for this message
Bill Filler (bfiller) wrote :

we need to update the tests to do the same thing as the webbrowser-app-autopilot which will remove the need for sleep. That is, ensure the osk is visible and then explicitly move input into the note. I will attach a reference to the code that does so in the OSK.

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

This MR contains the code from the webbrowser. The same can be done in notes:
https://code.launchpad.net/~osomon/webbrowser-app/keyboard-rectangle-states/+merge/175963

Changed in notes-app:
assignee: nobody → Omer Akram (om26er)
importance: Undecided → High
Bill Filler (bfiller)
tags: added: autopilot
Bill Filler (bfiller)
Changed in notes-app:
status: New → In Progress
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

Fix committed into lp:notes-app at revision 165, scheduled for release in notes-app, milestone ubuntu-13.04-month-5

Changed in notes-app:
status: In Progress → 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: New → Fix Released
Olivier Tilloy (osomon)
Changed in notes-app:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

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