Merge lp:~mterry/unity8/expose-lockscreen-passphrase into lp:unity8

Proposed by Michael Terry
Status: Merged
Approved by: Michał Sawicz
Approved revision: 1370
Merged at revision: 1392
Proposed branch: lp:~mterry/unity8/expose-lockscreen-passphrase
Merge into: lp:unity8
Diff against target: 51 lines (+12/-0)
4 files modified
debian/changelog (+6/-0)
qml/Components/Lockscreen.qml (+2/-0)
qml/Components/PassphraseLockscreen.qml (+2/-0)
qml/Components/PinLockscreen.qml (+2/-0)
To merge this branch: bzr merge lp:~mterry/unity8/expose-lockscreen-passphrase
Reviewer Review Type Date Requested Status
Michał Sawicz Approve
PS Jenkins bot (community) continuous-integration Needs Fixing
Review via email: mp+239364@code.launchpad.net

Commit message

Expose a 'passphrase' field of Components.Lockscreen to allow consumers (like the wizard) to query the password content before the user finishes. Needed to provide a proper Continue button in the welcome wizard.

I know that passphrase isn't the *best* field name, considering it works for a passcode too. But we already expose that phrase for both in the accepted() signal. So for consistency, I kept it the same.

And I bumped the unity8 version, so that ubuntu-system-settings can depend on this new field.

Description of the change

Expose a 'passphrase' field of Components.Lockscreen to allow consumers (like the wizard) to query the password content before the user finishes. Needed to provide a proper Continue button in the welcome wizard.

I know that passphrase isn't the *best* field name, considering it works for a passcode too. But we already expose that phrase for both in the accepted() signal. So for consistency, I kept it the same.

And I bumped the unity8 version, so that ubuntu-system-settings can depend on this new field.

== Checklist ==

 * Are there any related MPs required for this MP to build/function as expected? Please list.
 No

 * Did you perform an exploratory manual test run of your code change and any related functionality?
 Yes

 * Did you make sure that your branch does not contain spurious tags?
 Yes

 * If you changed the packaging (debian), did you subscribe the ubuntu-unity team to this MP?
 I am in that team

 * If you changed the UI, has there been a design review?
 NA

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Michał Sawicz (saviq) wrote :

 * Did you perform an exploratory manual test run of the code change and any related functionality?
No functional change.
 * Did CI run pass? If not, please explain why.
Not AP :/
 * Did you make sure that the branch does not contain spurious tags?
Y

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/changelog'
2--- debian/changelog 2014-10-13 15:43:04 +0000
3+++ debian/changelog 2014-10-23 13:26:46 +0000
4@@ -1,3 +1,9 @@
5+unity8 (8.01) UNRELEASED; urgency=medium
6+
7+ * Provide 'passphrase' as a field of Components.Lockscreen
8+
9+ -- Michael Terry <mterry@ubuntu.com> Wed, 22 Oct 2014 16:05:26 -0400
10+
11 unity8 (8.00+14.10.20141013.2-0ubuntu1) utopic; urgency=low
12
13 [ Michael Terry ]
14
15=== modified file 'qml/Components/Lockscreen.qml'
16--- qml/Components/Lockscreen.qml 2014-10-07 08:19:36 +0000
17+++ qml/Components/Lockscreen.qml 2014-10-23 13:26:46 +0000
18@@ -56,6 +56,8 @@
19
20 property url background: ""
21
22+ readonly property string passphrase: (pinPadLoader.item && pinPadLoader.item.passphrase) ? pinPadLoader.item.passphrase : ""
23+
24 signal entered(string passphrase)
25 signal cancel()
26 signal emergencyCall()
27
28=== modified file 'qml/Components/PassphraseLockscreen.qml'
29--- qml/Components/PassphraseLockscreen.qml 2014-09-25 13:53:52 +0000
30+++ qml/Components/PassphraseLockscreen.qml 2014-10-23 13:26:46 +0000
31@@ -28,6 +28,8 @@
32 property string errorText
33 property bool entryEnabled: true
34
35+ readonly property string passphrase: pinentryField.text
36+
37 signal entered(string passphrase)
38 signal cancel()
39
40
41=== modified file 'qml/Components/PinLockscreen.qml'
42--- qml/Components/PinLockscreen.qml 2014-10-08 14:31:53 +0000
43+++ qml/Components/PinLockscreen.qml 2014-10-23 13:26:46 +0000
44@@ -35,6 +35,8 @@
45 property int maxPinLength: -1
46 property bool showCancelButton: true
47
48+ readonly property string passphrase: pinentryField.text
49+
50 signal entered(string passphrase)
51 signal cancel()
52

Subscribers

People subscribed via source and target branches