Merge lp:~tiagosh/messaging-app/fix-multirecipient-focus into lp:messaging-app

Proposed by Tiago Salem Herrmann
Status: Merged
Approved by: Bill Filler
Approved revision: 530
Merged at revision: 534
Proposed branch: lp:~tiagosh/messaging-app/fix-multirecipient-focus
Merge into: lp:messaging-app
Diff against target: 11 lines (+0/-1)
1 file modified
src/qml/Messages.qml (+0/-1)
To merge this branch: bzr merge lp:~tiagosh/messaging-app/fix-multirecipient-focus
Reviewer Review Type Date Requested Status
PS Jenkins bot continuous-integration Needs Fixing
Ubuntu Phablet Team Pending
Review via email: mp+288641@code.launchpad.net

Commit message

Fix active focus management of multi recipient component.

Description of the change

Fix active focus management of multi recipient component.

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
530. By Tiago Salem Herrmann

focus: true is breaking an internal binding

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/qml/Messages.qml'
2--- src/qml/Messages.qml 2016-02-23 15:07:46 +0000
3+++ src/qml/Messages.qml 2016-03-11 15:16:55 +0000
4@@ -659,7 +659,6 @@
5 top: parent ? parent.top: undefined
6 topMargin: units.gu(1)
7 }
8- focus: true
9
10 Connections {
11 target: mainView.bottomEdge

Subscribers

People subscribed via source and target branches