Ubuntu
address-book-app package

Contacts Cancel/Save buttons don't work with OSK showing when running Mir

Bug #1234600 reported by Richard Huddie
12
This bug affects 2 people
Affects Status Importance Assigned to Milestone
ubuntu-keyboard
Fix Released
Undecided
Daniel d'Andrada
unity-mir
Fix Released
High
Daniel d'Andrada
unity-mir phone-v1-freeze
address-book-app (Ubuntu)
Confirmed
High
Unassigned
Saucy
Confirmed
High
Unassigned
ubuntu-keyboard (Ubuntu)
Fix Released
Undecided
Unassigned
Saucy
Fix Released
Undecided
Unassigned
unity-mir (Ubuntu)
Fix Released
Undecided
Unassigned
Saucy
Fix Released
Undecided
Unassigned

Bug Description

mako 20131003

PRE-REQUISITE:
Enable Mir:
adb shell touch /home/phablet/.display-mir
adb reboot

TEST CASE:
1. Load the Conatcts app and create a new contact
2. Type in a name for the contact using OSK
3. With OSK still showing try and press either the cancel or save buttons, shown above the OSK

EXPECTED RESULT:
The cancel/save buttons should work

ACTUAL RESULT:
The cancel/save buttons do not work when the OSK is showing. When the OSK is hidden, the cancel/save buttons will function again.

This problem only occurs with Mir enabled.

ProblemType: Bug
DistroRelease: Ubuntu 13.10
Package: address-book-app 0.2+13.10.20131001.4-0ubuntu1
Uname: Linux 3.4.0-3-mako armv7l
ApportVersion: 2.12.5-0ubuntu1
Architecture: armhf
Date: Thu Oct 3 09:40:14 2013
InstallationDate: Installed on 2013-10-03 (0 days ago)
InstallationMedia: Ubuntu Saucy Salamander (development branch) - armhf (20131003)
MarkForUpload: True
ProcEnviron:
 TERM=linux
 PATH=(custom, no user)
SourcePackage: address-book-app
UpgradeStatus: No upgrade log present (probably fresh install)

Related branches

lp:~dandrader/unity-mir/fix_lp1234600
Michał Sawicz: Approve
PS Jenkins bot (community): Approve (continuous-integration)
Diff: 277 lines (+177/-9)
5 files modified
src/modules/Unity/Application/Application.pro (+3/-1)
src/modules/Unity/Application/OSKController.qml (+8/-8)
src/modules/Unity/Application/plugin.cpp (+2/-0)
src/modules/Unity/Application/ubuntukeyboardinfo.cpp (+103/-0)
src/modules/Unity/Application/ubuntukeyboardinfo.h (+61/-0)
lp:~dandrader/ubuntu-keyboard/fix_lp1234600
Thomas Voß (community): Approve
Michał Sawicz: Approve
PS Jenkins bot: Approve (continuous-integration)
Diff: 151 lines (+91/-4)
2 files modified
src/plugin/ubuntuapplicationapiwrapper.cpp (+70/-1)
src/plugin/ubuntuapplicationapiwrapper.h (+21/-3)
lp:ubuntu/saucy-proposed/unity-mir
lp:ubuntu/saucy-proposed/ubuntu-keyboard
Revision history for this message
Richard Huddie (rhuddie) wrote :
Richard Huddie (rhuddie)
tags: added: rls-s-incoming
Revision history for this message
Jean-Baptiste Lallement (jibel) wrote :

confirmed with build 78

Changed in address-book-app (Ubuntu Saucy):
status: New → Confirmed
Revision history for this message
kevin gunn (kgunn72) wrote :

can you make sure this still happens in images > 79 & determine if its our bug

Changed in unity-mir:
status: New → Triaged
importance: Undecided → High
assignee: nobody → Daniel d'Andrada (dandrader)
milestone: none → phone-v1-freeze
Revision history for this message
Gerry Boland (gerboland) wrote :

Our bug, InputArea for OSK must be overlapping those buttons, easy fix

Changed in unity-mir:
status: Triaged → Confirmed
Daniel d'Andrada (dandrader)
Changed in unity-mir:
status: Confirmed → In Progress
Revision history for this message
Daniel d'Andrada (dandrader) wrote :

It's not just a matter of updating our hardcoded OSK size in unity-mir.
That extra space is needed for the word prediction ribbon (that shows word candidates). So we really need to remove that hack and get size updated from maliit.

Revision history for this message
Daniel d'Andrada (dandrader) wrote :

s/updated/updates

Daniel d'Andrada (dandrader)
no longer affects: address-book-app
Changed in ubuntu-keyboard:
assignee: nobody → Daniel d'Andrada (dandrader)
status: New → In Progress
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

Fix committed into lp:unity-mir at revision 111, scheduled for release in unity-mir, milestone phone-v1-freeze

Changed in unity-mir:
status: In Progress → Fix Committed
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

Fix committed into lp:ubuntu-keyboard at revision 68, scheduled for release in ubuntu-keyboard, milestone Unknown

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

This bug was fixed in the package ubuntu-keyboard - 0.99.trunk.phablet2+13.10.20131009.2-0ubuntu1

---------------
ubuntu-keyboard (0.99.trunk.phablet2+13.10.20131009.2-0ubuntu1) saucy; urgency=low

  [ Daniel d'Andrada ]
  * Publish keyboard info on a socket so that unity-mir can have it Have
    a communication channel between ubuntu-keyboard and unity-mir where
    the former hands out all the information about itself that unity-mir
    might be interested into. That all should go away once a proper
    architecure gets implemented. Fixes LP#1234600. (LP: #1234600,
    #1237472)

  [ Thomas Moenicke ]
  * 3-key on symbols layout does not have extended keys .
  * fixes #1236501 wrong layout when symbols. (LP: #1236501)
  * fixing spacebar background.
  * various fixes, prioritized by Bill .

  [ Ubuntu daily release ]
  * New rebuild forced

  [ Christopher Lee ]
  * Remove workarounds and fixed uncaught exception at import time.

  [ Guenter Schwann ]
  * Send return ("\r") as text in return key event fixes LP: #1230035.
    (LP: #1230035, #1236388)
  * Propagating mouse events does not work as expected when dragging
    Going back to make the keyboard a child item of the swipe mouse
    area. Fixes LP: #1236474. (LP: #1236474)
  * New Magnifier that fixes the old Popper element fixes LP: #1236478
    fixes LP: #1236477. (LP: #1236477, #1236478)

  [ Ubuntu daily release ]
  * Automatic snapshot from revision 70
 -- Ubuntu daily release <email address hidden> Wed, 09 Oct 2013 23:35:18 +0000

Changed in ubuntu-keyboard (Ubuntu Saucy):
status: New → Fix Released
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package unity-mir - 0.1+13.10.20131009.3-0ubuntu2

---------------
unity-mir (0.1+13.10.20131009.3-0ubuntu2) saucy; urgency=low

  * No change rebuild to drop upstart-app-launch dep; sigh.
 -- Loic Minier <email address hidden> Thu, 10 Oct 2013 04:12:30 +0200

Changed in unity-mir (Ubuntu Saucy):
status: New → Fix Released
Michał Sawicz (saviq)
Changed in unity-mir:
status: Fix Committed → Fix Released
Changed in ubuntu-keyboard:
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.